[PATCH] D124063: [LegacyPM] Rename and deprecate populateModulePassManager
Nikita Popov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 20 00:08:26 PDT 2022
nikic added a comment.
Please do not change names in the FFI API at least. It's okay to drop it entirely, but renaming is unnecessarily hostile for FFI APIs.
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:95
This seems like an easy usage to replace?
Comment at: llvm/include/llvm-c/Transforms/PassManagerBuilder.h:72
+__attribute__((deprecated("Migrate to new pass manager for optimization pipeline. This header will be removed in LLVM 16.")))
Let's not promise any specific timeline. IMHO we should drop this as soon as there are no in-tree users anymore.
Comment at: llvm/unittests/ExecutionEngine/MCJIT/MCJITCAPITest.cpp:313
This also looks like an easy usage to remove.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits