[coreboot-gerrit] Change in coreboot[master]: Fix build with CLANG, avoid GCC only CFLAGS

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/21202


Change subject: Fix build with CLANG, avoid GCC only CFLAGS
......................................................................

Fix build with CLANG, avoid GCC only CFLAGS

Commit 7c8d331fbb "Fine-tune compiler flags" added CFLAGS that are not
existing on CLANG hence breaking building coreboot with clang.

Change-Id: Ie0250e285b0c5a9f8ee2eb99401aeca875d2789a
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M Makefile.inc
1 file changed, 2 insertions(+), 0 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/21202/1

diff --git a/Makefile.inc b/Makefile.inc
index 679e0ac..ca9711d 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -363,7 +363,9 @@
 CFLAGS_common += -Wstrict-aliasing -Wshadow -Wdate-time
 CFLAGS_common += -fno-common -ffreestanding -fno-builtin -fomit-frame-pointer
 CFLAGS_common += -ffunction-sections -fdata-sections -fno-pie
+ifeq ($(CONFIG_COMPILER_GCC),y)
 CFLAGS_common += -fno-delete-null-pointer-checks -fconserve-stack
+endif
 
 ADAFLAGS_common += -gnatg -gnatp
 ADAFLAGS_common += -Wuninitialized -Wall -Werror

-- 
To view, visit https://review.coreboot.org/21202
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie0250e285b0c5a9f8ee2eb99401aeca875d2789a
Gerrit-Change-Number: 21202
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170825/833c71c6/attachment.html>