[PATCH] D96479: [NFC][LTO] Directly create default AA pipeline
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 10 21:43:15 PST 2021
aeubanks created this revision.
Herald added subscribers: steven_wu, hiraditya, inglorion.
aeubanks requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
I believe the comment was supposed to only apply to runNewPMCustomPasses().
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D96479
Files:
llvm/lib/LTO/LTOBackend.cpp
Index: llvm/lib/LTO/LTOBackend.cpp
===================================================================
--- llvm/lib/LTO/LTOBackend.cpp
+++ llvm/lib/LTO/LTOBackend.cpp
@@ -224,11 +224,6 @@
StandardInstrumentations SI(Conf.DebugPassManager);
SI.registerCallbacks(PIC);
PassBuilder PB(Conf.DebugPassManager, TM, Conf.PTO, PGOOpt, &PIC);
- AAManager AA;
-
- // Parse a custom AA pipeline if asked to.
- if (auto Err = PB.parseAAPipeline(AA, "default"))
- report_fatal_error("Error parsing default AA pipeline");
RegisterPassPlugins(Conf.PassPlugins, PB);
@@ -243,6 +238,7 @@
TLII->disableAllFunctions();
FAM.registerPass([&] { return TargetLibraryAnalysis(*TLII); });
+ AAManager AA = PB.buildDefaultAAPipeline();
// Register the AA manager first so that our version is the one used.
FAM.registerPass([&] { return std::move(AA); });
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D96479.322904.patch
Type: text/x-patch
Size: 870 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210211/c3c2b523/attachment.bin>
More information about the llvm-commits
mailing list