[coreboot-gerrit] Patch merged into coreboot/master: Makefile.inc: Explicitly set GNU11 as C language standard
gerrit at coreboot.org
gerrit at coreboot.org
Sun Jan 22 19:26:28 CET 2017
the following patch was just integrated into master:
commit acbb70b810c6eb17e403c37fa2888b479e5b23a9
Author: Paul Menzel <pmenzel at molgen.mpg.de>
Date: Tue Nov 29 10:30:44 2016 +0100
Makefile.inc: Explicitly set GNU11 as C language standard
Different compiler versions use a different C language standard by
default.
GCC 4.9 uses GNU89 by default [1], while GCC 5.x uses GNU11 [2].
The discussion on the mailing list in thread *[RFC] Setting C99 by
default* [3] resulted in the preference of C11, which results in build
errors.
So explicitly set it to GNU11, which is also what the current coreboot
toolchain with GCC 5.3 is using.
[1] https://gcc.gnu.org/onlinedocs/gcc-4.9.4/gcc/C-Dialect-Options.html
[2] https://gcc.gnu.org/onlinedocs/gcc-5.4.0/gcc/Standards.html
[3] https://www.coreboot.org/pipermail/coreboot/2016-November/082541.html
Change-Id: If1569618f8044925ff72dcf3543480b34d4f90d6
Signed-off-by: Paul Menzel <pmenzel at molgen.mpg.de>
Reviewed-on: https://review.coreboot.org/17636
Reviewed-by: Julius Werner <jwerner at chromium.org>
Reviewed-by: Arthur Heymans <arthur at aheymans.xyz>
Tested-by: build bot (Jenkins)
Tested-by: Raptor Engineering Automated Test Stand <noreply at raptorengineeringinc.com>
See https://review.coreboot.org/17636 for details.
-gerrit
More information about the coreboot-gerrit
mailing list