Consolidate global cflags
Move all the flags that are used everywhere (or should be used
everywhere) into cc/config/global.go:
-no-canonical-prefixes
-fno-exceptions
-Wno-multichar
-O2
-g
-fno-strict-aliasing
Also remove flags that are already in noOverrideGlobalCflags:
-Werror=pointer-to-int-cast
-Werror=int-to-pointer-cast
x86 and x86_64 were specifying -fstrict-aliasing, but that was
being overriden later by -fno-strict-aliasing, so remove it.
ARM devices still override -fno-strict-aliasing to -fstrict-aliasing
when using ARM (vs. Thumb) instruction set.
Bug: 68855788
Test: m checkbuild
Change-Id: Ia2b5891bdefb60f974ad92b4b84a8548c2dcc7dc
diff --git a/cc/config/arm64_device.go b/cc/config/arm64_device.go
index 390936a..e8b4797 100644
--- a/cc/config/arm64_device.go
+++ b/cc/config/arm64_device.go
@@ -23,9 +23,6 @@
var (
arm64Cflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
- "-fno-strict-aliasing",
"-fstack-protector-strong",
"-ffunction-sections",
"-fdata-sections",
@@ -34,19 +31,14 @@
"-Werror=format-security",
"-D_FORTIFY_SOURCE=2",
"-fno-short-enums",
- "-no-canonical-prefixes",
"-fno-canonical-system-headers",
// Help catch common 32/64-bit errors.
- "-Werror=pointer-to-int-cast",
- "-Werror=int-to-pointer-cast",
"-Werror=implicit-function-declaration",
"-fno-strict-volatile-bitfields",
// TARGET_RELEASE_CFLAGS
- "-DNDEBUG",
- "-O2 -g",
"-Wstrict-aliasing=2",
"-fgcse-after-reload",
"-frerun-cse-after-loop",
diff --git a/cc/config/arm_device.go b/cc/config/arm_device.go
index 7110ccb..8e890ff 100644
--- a/cc/config/arm_device.go
+++ b/cc/config/arm_device.go
@@ -28,8 +28,6 @@
}
armCflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
"-ffunction-sections",
"-fdata-sections",
"-funwind-tables",
@@ -38,15 +36,12 @@
"-Werror=format-security",
"-D_FORTIFY_SOURCE=2",
"-fno-short-enums",
- "-no-canonical-prefixes",
"-fno-canonical-system-headers",
"-fno-builtin-sin",
"-fno-strict-volatile-bitfields",
// TARGET_RELEASE_CFLAGS
- "-DNDEBUG",
- "-g",
"-Wstrict-aliasing=2",
"-fgcse-after-reload",
"-frerun-cse-after-loop",
@@ -71,7 +66,6 @@
}
armArmCflags = []string{
- "-O2",
"-fomit-frame-pointer",
"-fstrict-aliasing",
"-funswitch-loops",
@@ -81,7 +75,6 @@
"-mthumb",
"-Os",
"-fomit-frame-pointer",
- "-fno-strict-aliasing",
}
armArchVariantCflags = map[string][]string{
diff --git a/cc/config/global.go b/cc/config/global.go
index dae218c..5587eae 100644
--- a/cc/config/global.go
+++ b/cc/config/global.go
@@ -34,9 +34,19 @@
"-Winit-self",
"-Wpointer-arith",
- // COMMON_RELEASE_CFLAGS
+ // Make paths in deps files relative
+ "-no-canonical-prefixes",
+
"-DNDEBUG",
"-UDEBUG",
+
+ "-fno-exceptions",
+ "-Wno-multichar",
+
+ "-O2",
+ "-g",
+
+ "-fno-strict-aliasing",
}
commonGlobalConlyflags = []string{}
diff --git a/cc/config/mips64_device.go b/cc/config/mips64_device.go
index e05dbf1..7da693f 100644
--- a/cc/config/mips64_device.go
+++ b/cc/config/mips64_device.go
@@ -22,11 +22,7 @@
var (
mips64Cflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
- "-O2",
"-fomit-frame-pointer",
- "-fno-strict-aliasing",
"-funswitch-loops",
"-Umips",
"-ffunction-sections",
@@ -36,17 +32,12 @@
"-Wa,--noexecstack",
"-Werror=format-security",
"-D_FORTIFY_SOURCE=2",
- "-no-canonical-prefixes",
"-fno-canonical-system-headers",
// Help catch common 32/64-bit errors.
- "-Werror=pointer-to-int-cast",
- "-Werror=int-to-pointer-cast",
"-Werror=implicit-function-declaration",
// TARGET_RELEASE_CFLAGS
- "-DNDEBUG",
- "-g",
"-Wstrict-aliasing=2",
"-fgcse-after-reload",
"-frerun-cse-after-loop",
diff --git a/cc/config/mips_device.go b/cc/config/mips_device.go
index 78e95b6..f8470bf 100644
--- a/cc/config/mips_device.go
+++ b/cc/config/mips_device.go
@@ -22,11 +22,7 @@
var (
mipsCflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
- "-O2",
"-fomit-frame-pointer",
- "-fno-strict-aliasing",
"-funswitch-loops",
"-Umips",
"-ffunction-sections",
@@ -36,12 +32,9 @@
"-Wa,--noexecstack",
"-Werror=format-security",
"-D_FORTIFY_SOURCE=2",
- "-no-canonical-prefixes",
"-fno-canonical-system-headers",
// TARGET_RELEASE_CFLAGS
- "-DNDEBUG",
- "-g",
"-Wstrict-aliasing=2",
"-fgcse-after-reload",
"-frerun-cse-after-loop",
diff --git a/cc/config/x86_64_device.go b/cc/config/x86_64_device.go
index a98001e..8d7a404 100644
--- a/cc/config/x86_64_device.go
+++ b/cc/config/x86_64_device.go
@@ -22,9 +22,6 @@
var (
x86_64Cflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
- "-O2",
"-Wa,--noexecstack",
"-Werror=format-security",
"-D_FORTIFY_SOURCE=2",
@@ -33,22 +30,13 @@
"-finline-functions",
"-finline-limit=300",
"-fno-short-enums",
- "-fstrict-aliasing",
"-funswitch-loops",
"-funwind-tables",
"-fstack-protector-strong",
- "-no-canonical-prefixes",
"-fno-canonical-system-headers",
// Help catch common 32/64-bit errors.
- "-Werror=pointer-to-int-cast",
- "-Werror=int-to-pointer-cast",
"-Werror=implicit-function-declaration",
-
- // TARGET_RELEASE_CFLAGS from build/core/combo/select.mk
- "-O2",
- "-g",
- "-fno-strict-aliasing",
}
x86_64Cppflags = []string{}
diff --git a/cc/config/x86_darwin_host.go b/cc/config/x86_darwin_host.go
index 6d361b5..8d805c9 100644
--- a/cc/config/x86_darwin_host.go
+++ b/cc/config/x86_darwin_host.go
@@ -25,9 +25,6 @@
var (
darwinCflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
-
"-fdiagnostics-color",
"-fPIC",
@@ -38,10 +35,6 @@
"-D__STDC_FORMAT_MACROS",
"-D__STDC_CONSTANT_MACROS",
- // HOST_RELEASE_CFLAGS
- "-O2", // from build/core/combo/select.mk
- "-g", // from build/core/combo/select.mk
- "-fno-strict-aliasing", // from build/core/combo/select.mk
"-isysroot ${macSdkRoot}",
"-mmacosx-version-min=${macMinVersion}",
"-DMACOSX_DEPLOYMENT_TARGET=${macMinVersion}",
diff --git a/cc/config/x86_device.go b/cc/config/x86_device.go
index 53d2265..59f8de7 100644
--- a/cc/config/x86_device.go
+++ b/cc/config/x86_device.go
@@ -22,9 +22,6 @@
var (
x86Cflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
- "-O2",
"-Wa,--noexecstack",
"-Werror=format-security",
"-D_FORTIFY_SOURCE=2",
@@ -33,17 +30,10 @@
"-finline-functions",
"-finline-limit=300",
"-fno-short-enums",
- "-fstrict-aliasing",
"-funswitch-loops",
"-funwind-tables",
"-fstack-protector-strong",
- "-no-canonical-prefixes",
"-fno-canonical-system-headers",
-
- // TARGET_RELEASE_CFLAGS from build/core/combo/select.mk
- "-O2",
- "-g",
- "-fno-strict-aliasing",
}
x86ClangCflags = append(x86Cflags, []string{
diff --git a/cc/config/x86_linux_bionic_host.go b/cc/config/x86_linux_bionic_host.go
index 277361b..645ac26 100644
--- a/cc/config/x86_linux_bionic_host.go
+++ b/cc/config/x86_linux_bionic_host.go
@@ -22,15 +22,11 @@
var (
linuxBionicCflags = ClangFilterUnknownCflags([]string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
-
"-fdiagnostics-color",
"-Wa,--noexecstack",
"-fPIC",
- "-no-canonical-prefixes",
"-U_FORTIFY_SOURCE",
"-D_FORTIFY_SOURCE=2",
@@ -43,14 +39,8 @@
"-fno-short-enums",
"-funswitch-loops",
"-funwind-tables",
- "-no-canonical-prefixes",
"-fno-canonical-system-headers",
- // HOST_RELEASE_CFLAGS
- "-O2", // from build/core/combo/select.mk
- "-g", // from build/core/combo/select.mk
- "-fno-strict-aliasing", // from build/core/combo/select.mk
-
// Tell clang where the gcc toolchain is
"--gcc-toolchain=${LinuxBionicGccRoot}",
diff --git a/cc/config/x86_linux_host.go b/cc/config/x86_linux_host.go
index 88bd514..4f05068 100644
--- a/cc/config/x86_linux_host.go
+++ b/cc/config/x86_linux_host.go
@@ -22,15 +22,11 @@
var (
linuxCflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
-
"-fdiagnostics-color",
"-Wa,--noexecstack",
"-fPIC",
- "-no-canonical-prefixes",
"-U_FORTIFY_SOURCE",
"-D_FORTIFY_SOURCE=2",
@@ -40,11 +36,6 @@
//See bug 12708004.
"-D__STDC_FORMAT_MACROS",
"-D__STDC_CONSTANT_MACROS",
-
- // HOST_RELEASE_CFLAGS
- "-O2", // from build/core/combo/select.mk
- "-g", // from build/core/combo/select.mk
- "-fno-strict-aliasing", // from build/core/combo/select.mk
}
linuxLdflags = []string{
diff --git a/cc/config/x86_windows_host.go b/cc/config/x86_windows_host.go
index 270084e..c9bafe6 100644
--- a/cc/config/x86_windows_host.go
+++ b/cc/config/x86_windows_host.go
@@ -22,9 +22,6 @@
var (
windowsCflags = []string{
- "-fno-exceptions", // from build/core/combo/select.mk
- "-Wno-multichar", // from build/core/combo/select.mk
-
"-DUSE_MINGW",
"-DWIN32_LEAN_AND_MEAN",
"-Wno-unused-parameter",
@@ -43,11 +40,6 @@
"-D_FILE_OFFSET_BITS=64",
"--sysroot ${WindowsGccRoot}/${WindowsGccTriple}",
-
- // HOST_RELEASE_CFLAGS
- "-O2", // from build/core/combo/select.mk
- "-g", // from build/core/combo/select.mk
- "-fno-strict-aliasing", // from build/core/combo/select.mk
}
windowsIncludeFlags = []string{