[PATCH] D145978: [Pipeline] Remove Annotation2Metadata pass in post-link pipelines
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 13 12:34:46 PDT 2023
aeubanks created this revision.
Herald added a subscriber: hiraditya.
Herald added a project: All.
aeubanks requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
The pre-link pipeline already ran the pass and it only needs to be run once.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D145978
Files:
llvm/lib/Passes/PassBuilderPipelines.cpp
Index: llvm/lib/Passes/PassBuilderPipelines.cpp
===================================================================
--- llvm/lib/Passes/PassBuilderPipelines.cpp
+++ llvm/lib/Passes/PassBuilderPipelines.cpp
@@ -1502,9 +1502,6 @@
OptimizationLevel Level, const ModuleSummaryIndex *ImportSummary) {
ModulePassManager MPM;
- // Convert @llvm.global.annotations to !annotation metadata.
- MPM.addPass(Annotation2MetadataPass());
-
if (ImportSummary) {
// These passes import type identifier resolutions for whole-program
// devirtualization and CFI. They must run early because other passes may
@@ -1568,9 +1565,6 @@
ModuleSummaryIndex *ExportSummary) {
ModulePassManager MPM;
- // Convert @llvm.global.annotations to !annotation metadata.
- MPM.addPass(Annotation2MetadataPass());
-
for (auto &C : FullLinkTimeOptimizationEarlyEPCallbacks)
C(MPM, Level);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145978.504805.patch
Type: text/x-patch
Size: 931 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230313/8142bd2f/attachment.bin>
More information about the llvm-commits
mailing list