diff options
| author | Duncan Sands <baldrick@free.fr> | 2010-07-07 07:48:00 +0000 |
|---|---|---|
| committer | Duncan Sands <baldrick@free.fr> | 2010-07-07 07:48:00 +0000 |
| commit | 8246adc1f0e2d28374da3aeab864aee5ff03f3ff (patch) | |
| tree | 352c03a8c9e1939b3013328a5caffeae94ae1d37 /Makefile.rules | |
| parent | 4f6bdf9042dee0d49b0537b73be93878d0b402b1 (diff) | |
| download | llvm-8246adc1f0e2d28374da3aeab864aee5ff03f3ff.tar.gz llvm-8246adc1f0e2d28374da3aeab864aee5ff03f3ff.tar.bz2 llvm-8246adc1f0e2d28374da3aeab864aee5ff03f3ff.tar.xz | |
Rename "Release" builds as "Release+Asserts"; rename "Release-Asserts"
builds to "Release". The default build is unchanged (optimization on,
assertions on), however it is now called Release+Asserts. The intent
is that future LLVM releases released via llvm.org will be Release builds
in the new sense, i.e. will have assertions disabled (currently they have
assertions enabled, for a more than 20% slowdown). This will bring them
in line with MacOS releases, which ship with assertions disabled. It also
means that "Release" now means the same things in make and cmake builds:
cmake already disables assertions for "Release" builds AFAICS.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@107758 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'Makefile.rules')
| -rw-r--r-- | Makefile.rules | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/Makefile.rules b/Makefile.rules index 14b5ce9719..dc15c921ff 100644 --- a/Makefile.rules +++ b/Makefile.rules @@ -398,12 +398,11 @@ endif # If DISABLE_ASSERTIONS=1 is specified (make command line or configured), # then disable assertions by defining the appropriate preprocessor symbols. -ifdef DISABLE_ASSERTIONS - # Indicate that assertions are turned off using a minus sign - BuildMode := $(BuildMode)-Asserts - CPP.Defines += -DNDEBUG -else +ifndef DISABLE_ASSERTIONS + BuildMode := $(BuildMode)+Asserts CPP.Defines += -D_DEBUG +else + CPP.Defines += -DNDEBUG endif # If ENABLE_EXPENSIVE_CHECKS=1 is specified (make command line or |