|
|
cc9195 |
From e7091299a079fd0405f01276f35838884b4c06b5 Mon Sep 17 00:00:00 2001
|
|
|
cc9195 |
From: Laszlo Ersek <lersek@redhat.com>
|
|
|
cc9195 |
Date: Wed, 29 Aug 2018 17:11:59 +0200
|
|
|
cc9195 |
Subject: [PATCH 2/7] BaseTools/header.makefile: remove "-c" from BUILD_CFLAGS
|
|
|
cc9195 |
|
|
|
cc9195 |
Message-id: <20180829151204.26958-3-lersek@redhat.com>
|
|
|
cc9195 |
Patchwork-id: 81964
|
|
|
cc9195 |
O-Subject: [RHEL8/virt212 edk2 PATCH 2/7] BaseTools/header.makefile: remove "-c"
|
|
|
cc9195 |
from BUILD_CFLAGS
|
|
|
cc9195 |
Bugzilla: 1607906
|
|
|
cc9195 |
Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
cc9195 |
Acked-by: Vitaly Kuznetsov <vkuznets@redhat.com>
|
|
|
cc9195 |
|
|
|
cc9195 |
Option "-c" is a mode selection flag (choosing between compiling and
|
|
|
cc9195 |
linking); it should not be in BUILD_CFLAGS, which applies only to
|
|
|
cc9195 |
compiling anyway. The compilation rule for C source files, in
|
|
|
cc9195 |
"footer.makefile", already includes "-c" -- currently we have double "-c"
|
|
|
cc9195 |
options.
|
|
|
cc9195 |
|
|
|
cc9195 |
This patch doesn't change behavior.
|
|
|
cc9195 |
|
|
|
cc9195 |
Cc: Liming Gao <liming.gao@intel.com>
|
|
|
cc9195 |
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
|
|
|
cc9195 |
Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1540244
|
|
|
cc9195 |
Contributed-under: TianoCore Contribution Agreement 1.1
|
|
|
cc9195 |
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
cc9195 |
Reviewed-by: Liming Gao <liming.gao@intel.com>
|
|
|
cc9195 |
(cherry picked from commit 03252ae287c4a61983b3793ff71baeabe2ff3df7)
|
|
|
cc9195 |
---
|
|
|
cc9195 |
BaseTools/Source/C/Makefiles/header.makefile | 4 ++--
|
|
|
cc9195 |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
|
cc9195 |
|
|
|
cc9195 |
diff --git a/BaseTools/Source/C/Makefiles/header.makefile b/BaseTools/Source/C/Makefiles/header.makefile
|
|
|
cc9195 |
index db43677..08421ba 100644
|
|
|
cc9195 |
--- a/BaseTools/Source/C/Makefiles/header.makefile
|
|
|
cc9195 |
+++ b/BaseTools/Source/C/Makefiles/header.makefile
|
|
|
cc9195 |
@@ -71,9 +71,9 @@ INCLUDE = $(TOOL_INCLUDE) -I $(MAKEROOT) -I $(MAKEROOT)/Include/Common -I $(MAKE
|
|
|
cc9195 |
BUILD_CPPFLAGS = $(INCLUDE) -O2
|
|
|
cc9195 |
ifeq ($(DARWIN),Darwin)
|
|
|
cc9195 |
# assume clang or clang compatible flags on OS X
|
|
|
cc9195 |
-BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-self-assign -Wno-unused-result -nostdlib -c -g
|
|
|
cc9195 |
+BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-self-assign -Wno-unused-result -nostdlib -g
|
|
|
cc9195 |
else
|
|
|
cc9195 |
-BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-stringop-truncation -Wno-restrict -Wno-unused-result -nostdlib -c -g
|
|
|
cc9195 |
+BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -Wno-stringop-truncation -Wno-restrict -Wno-unused-result -nostdlib -g
|
|
|
cc9195 |
endif
|
|
|
cc9195 |
BUILD_LFLAGS =
|
|
|
cc9195 |
BUILD_CXXFLAGS = -Wno-unused-result
|
|
|
cc9195 |
--
|
|
|
cc9195 |
1.8.3.1
|
|
|
cc9195 |
|