build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48504 )
Change subject: arch/x86/car.ld: Align the stack at buildtime ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/48504/1/src/include/memlayout.h File src/include/memlayout.h:
https://review.coreboot.org/c/coreboot/+/48504/1/src/include/memlayout.h@63 PS1, Line 63: #define CAR_STACK(addr, size) \ Macros with multiple statements should be enclosed in a do - while loop
https://review.coreboot.org/c/coreboot/+/48504/1/src/include/memlayout.h@63 PS1, Line 63: #define CAR_STACK(addr, size) \ macros should not use a trailing semicolon