Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51177 )
Change subject: libpayload: Enable LTO by default ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1: nvramcui fails to build due to problems with PDCurses:
``` LPGCC nvramcui.bin curses/PDCurses/pdcurses/refresh.c: In function 'wrefresh': curses/pdcurses-backend/pdcdisp.c:217:4: error: 'bg' may be used uninitialized in this function [-Werror=maybe-uninitialized] curses/pdcurses-backend/pdcdisp.c:214:18: note: 'bg' was declared here curses/pdcurses-backend/pdcdisp.c:217:4: error: 'fg' may be used uninitialized in this function [-Werror=maybe-uninitialized] curses/pdcurses-backend/pdcdisp.c:214:14: note: 'fg' was declared here lto1: all warnings being treated as errors lto-wrapper: fatal error: i386-elf-gcc returned 1 exit status compilation terminated. /opt/xgcc/lib/gcc/i386-elf/8.3.0/../../../../i386-elf/bin/ld.bfd: error: lto-wrapper failed collect2: error: ld returned 1 exit status make[2]: *** [../libpayload/Makefile.payload:91: build/nvramcui.bin] Error 1 make[2]: Leaving directory '/home/coreboot/node-root/workspace/coreboot-gerrit/payloads/nvramcui' ```