Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35014 )
Change subject: Move and rename ARCH_STAGE_HAS_xxx_SECTION rules ......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35014/4/src/include/rules.h File src/include/rules.h:
https://review.coreboot.org/c/coreboot/+/35014/4/src/include/rules.h@270 PS4, Line 270: /* No .data or .bss sections. Cache as RAM is handled separately. */ Append the comment to refer to the file which handles that? arch/x86/car.ld I know it wasn't there before, but we could provide a hint in the comments. And we may want to drop a hint somewhere that lib/program.ld isn't used for many of these programs on x86.