mirror of
https://github.com/RPCS3/llvm.git
synced 2025-02-24 04:32:09 +00:00
Don't make PDBs by default in Release mode
Introduce the LLVM_ENABLE_PDB option so that users can request them explicitly instead. Add /OPT:REF and /OPT:ICF back, which /DEBUG disables by default. Differential Revision: https://reviews.llvm.org/D43156 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@325296 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
048f392b85
commit
ff74bbe2b9
@ -353,11 +353,18 @@ if( MSVC )
|
||||
|
||||
append("/Zc:inline" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
|
||||
|
||||
# Generate PDB even in release for profiling.
|
||||
if (uppercase_CMAKE_BUILD_TYPE STREQUAL "RELEASE")
|
||||
# Allow users to request PDBs in release mode. CMake offeres the
|
||||
# RelWithDebInfo configuration, but it uses different optimization settings
|
||||
# (/Ob1 vs /Ob2 or -O2 vs -O3). LLVM provides this flag so that users can get
|
||||
# PDBs without changing codegen.
|
||||
option(LLVM_ENABLE_PDB OFF)
|
||||
if (LLVM_ENABLE_PDB AND uppercase_CMAKE_BUILD_TYPE STREQUAL "RELEASE")
|
||||
append("/Zi" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
|
||||
append("/DEBUG" CMAKE_EXE_LINKER_FLAGS CMAKE_MODULE_LINKER_FLAGS CMAKE_SHARED_LINKER_FLAGS)
|
||||
endif (uppercase_CMAKE_BUILD_TYPE STREQUAL "RELEASE")
|
||||
# /DEBUG disables linker GC and ICF, but we want those in Release mode.
|
||||
append("/DEBUG /OPT:REF /OPT:ICF"
|
||||
CMAKE_EXE_LINKER_FLAGS CMAKE_MODULE_LINKER_FLAGS
|
||||
CMAKE_SHARED_LINKER_FLAGS)
|
||||
endif()
|
||||
|
||||
# /Zc:strictStrings is incompatible with VS12's (Visual Studio 2013's)
|
||||
# debug mode headers. Instead of only enabling them in VS2013's debug mode,
|
||||
|
Loading…
x
Reference in New Issue
Block a user