StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Handle Clang's change of defaults from -fcommon to -fno-common" into main am: 748f8fb624

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2713373

Change-Id: I87cfe235d8ee01409437aac0b5d6d17264ecc38a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Zijun Zhao
2023-09-13 18:00:02 +00:00
committed by Automerger Merge Worker
parent 8233c7fb9c 748f8fb624
commit 59b4a1726c
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cc/config/global.go
View File

@@ -67,10 +67,6 @@ var (
// not emit the table by default on Android since NDK still uses GNU binutils.
"-faddrsig",
// Turn on -fcommon explicitly, since Clang now defaults to -fno-common. The cleanup bug
// tracking this is http://b/151457797.
"-fcommon",
// Help catch common 32/64-bit errors.
"-Werror=int-conversion",
@@ -253,6 +249,8 @@ var (
// (anything for which IsThirdPartyPath() in build/soong/android/paths.go
// returns true - includes external/, most of vendor/ and most of hardware/)
noOverrideExternalGlobalCflags = []string{
// http://b/151457797
"-fcommon",
// http://b/191699019
"-Wno-format-insufficient-args",
// http://b/296321145