[llvm] r353888 - [gn build] Separate debug and optimization settings
David Major via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 12 14:24:45 PST 2019
Author: dmajor
Date: Tue Feb 12 14:24:45 2019
New Revision: 353888
URL: http://llvm.org/viewvc/llvm-project?rev=353888&view=rev
Log:
[gn build] Separate debug and optimization settings
This patch adds an `is_optimized` variable, orthogonal to `is_debug`, to allow for a gn analogue to `RelWithDebInfo` builds.
As part of this we'll want to explicitly enable GC+ICF, for the sake of `is_debug && is_optimized` builds. The flags normally default to true except that if you pass `/DEBUG` they default to false.
Differential Revision: https://reviews.llvm.org/D58075
Modified:
llvm/trunk/utils/gn/build/BUILD.gn
llvm/trunk/utils/gn/build/buildflags.gni
Modified: llvm/trunk/utils/gn/build/BUILD.gn
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/gn/build/BUILD.gn?rev=353888&r1=353887&r2=353888&view=diff
==============================================================================
--- llvm/trunk/utils/gn/build/BUILD.gn (original)
+++ llvm/trunk/utils/gn/build/BUILD.gn Tue Feb 12 14:24:45 2019
@@ -23,7 +23,8 @@ config("compiler_defaults") {
if (host_os != "win") {
if (is_debug) {
cflags += [ "-g" ]
- } else {
+ }
+ if (is_optimized) {
cflags += [ "-O3" ]
}
cflags += [ "-fdiagnostics-color" ]
@@ -39,11 +40,16 @@ config("compiler_defaults") {
"/FS",
]
ldflags += [ "/DEBUG" ]
- } else {
+ }
+ if (is_optimized) {
cflags += [
"/O2",
"/Zc:inline",
]
+ ldflags += [
+ "/OPT:REF",
+ "/OPT:ICF",
+ ]
}
defines += [
"_CRT_SECURE_NO_DEPRECATE",
Modified: llvm/trunk/utils/gn/build/buildflags.gni
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/gn/build/buildflags.gni?rev=353888&r1=353887&r2=353888&view=diff
==============================================================================
--- llvm/trunk/utils/gn/build/buildflags.gni (original)
+++ llvm/trunk/utils/gn/build/buildflags.gni Tue Feb 12 14:24:45 2019
@@ -1,10 +1,13 @@
declare_args() {
- # Whether to build with debug information and without optimizations.
+ # Whether to build with debug information.
is_debug = false
}
# args that depend on other args must live in a later declare_args() block.
declare_args() {
+ # Whether to build with optimizations.
+ is_optimized = !is_debug
+
# Whether to enable assertions.
llvm_enable_assertions = true
}
More information about the llvm-commits
mailing list