[CMake] Change -DENABLE_EXPERIMENTAL_NEW_PASS_MANAGER=off to -DLLVM_ENABLE_NEW_PASS_MANAGER=off
LLVM_ENABLE_NEW_PASS_MANAGER is set to ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER, so -DLLVM_ENABLE_NEW_PASS_MANAGER=off has no effect. Change the cache variable to LLVM_ENABLE_NEW_PASS_MANAGER instead. A user opting out the new PM needs to switch from -DENABLE_EXPERIMENTAL_NEW_PASS_MANAGER=off to -DLLVM_ENABLE_NEW_PASS_MANAGER=off. Also give a warning that -DLLVM_ENABLE_NEW_PASS_MANAGER=off is deprecated. Reviewed By: aeubanks, phosek Differential Revision: https://reviews.llvm.org/D108775
This commit is contained in:
parent
1bebc31c61
commit
a42bd1b560
|
@ -41,7 +41,6 @@ set(CLANG_ENABLE_ARCMT OFF CACHE BOOL "")
|
|||
set(CLANG_ENABLE_STATIC_ANALYZER ON CACHE BOOL "")
|
||||
set(CLANG_PLUGIN_SUPPORT OFF CACHE BOOL "")
|
||||
|
||||
set(ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER ON CACHE BOOL "")
|
||||
set(ENABLE_LINKER_BUILD_ID ON CACHE BOOL "")
|
||||
set(ENABLE_X86_RELAX_RELOCATIONS ON CACHE BOOL "")
|
||||
|
||||
|
|
|
@ -32,7 +32,6 @@ set(CLANG_ENABLE_ARCMT OFF CACHE BOOL "")
|
|||
set(CLANG_ENABLE_STATIC_ANALYZER ON CACHE BOOL "")
|
||||
set(CLANG_PLUGIN_SUPPORT OFF CACHE BOOL "")
|
||||
|
||||
set(ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER ON CACHE BOOL "")
|
||||
set(ENABLE_LINKER_BUILD_ID ON CACHE BOOL "")
|
||||
set(ENABLE_X86_RELAX_RELOCATIONS ON CACHE BOOL "")
|
||||
|
||||
|
|
|
@ -706,8 +706,13 @@ else()
|
|||
endif()
|
||||
option(LLVM_ENABLE_PLUGINS "Enable plugin support" ${LLVM_ENABLE_PLUGINS_default})
|
||||
|
||||
set(ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER TRUE CACHE BOOL
|
||||
set(LLVM_ENABLE_NEW_PASS_MANAGER TRUE CACHE BOOL
|
||||
"Enable the new pass manager by default.")
|
||||
if(NOT LLVM_ENABLE_NEW_PASS_MANAGER)
|
||||
message(WARNING "Using the legacy pass manager for the optimization pipeline"
|
||||
" is deprecated. The functionality will degrade over time and"
|
||||
" be removed in a future release.")
|
||||
endif()
|
||||
|
||||
include(HandleLLVMOptions)
|
||||
|
||||
|
@ -846,10 +851,6 @@ if (NOT TENSORFLOW_AOT_PATH STREQUAL "")
|
|||
set_property(GLOBAL APPEND PROPERTY LLVM_EXPORTS tf_xla_runtime)
|
||||
endif()
|
||||
|
||||
# Keep the legacy CMake flag ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER for
|
||||
# compatibility.
|
||||
set(LLVM_ENABLE_NEW_PASS_MANAGER ${ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER})
|
||||
|
||||
# Configure the three LLVM configuration header files.
|
||||
configure_file(
|
||||
${LLVM_MAIN_INCLUDE_DIR}/llvm/Config/config.h.cmake
|
||||
|
|
|
@ -449,7 +449,7 @@ with the legacy PM.
|
|||
|
||||
For the optimization pipeline, the new PM is the default PM. The legacy PM is
|
||||
available for the optimization pipeline either by setting the CMake flag
|
||||
``-DENABLE_EXPERIMENTAL_NEW_PASS_MANAGER=OFF`` when building LLVM, or by
|
||||
``-DLLVM_ENABLE_NEW_PASS_MANAGER=OFF`` when building LLVM, or by
|
||||
various compiler/linker flags, e.g. ``-flegacy-pass-manager`` for ``clang``.
|
||||
|
||||
There will be efforts to deprecate and remove the legacy PM for the
|
||||
|
|
Loading…
Reference in a new issue