[coreboot-gerrit] [XS] Change in coreboot[main]: soc/intel/pmclib.c: Make sure array_size is not 0

[Nico Huber (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans.

Nico Huber has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/84042?usp=email )

Change subject: soc/intel/pmclib.c: Make sure array_size is not 0
......................................................................

Patch Set 7:

(1 comment)

File src/soc/intel/common/block/pmc/pmclib.c:

https://review.coreboot.org/c/coreboot/+/84042/comment/248233b9_02b88c21?usp... :
PS7, Line 104: 	for (i = num_bits - 1; i >= 0; i--) {
I don't get it, it should never enter the loop body with `num_bits == 0`, right?
On first sight, this looks like a compiler bug? What did it complain about specifically?