[llvm] r359852 - Tidy up a comment, fix a typo, remove a comment that's obsolete.
Eric Christopher via llvm-commits
llvm-commits at lists.llvm.org
Thu May 2 17:15:23 PDT 2019
Author: echristo
Date: Thu May 2 17:15:23 2019
New Revision: 359852
URL: http://llvm.org/viewvc/llvm-project?rev=359852&view=rev
Log:
Tidy up a comment, fix a typo, remove a comment that's obsolete.
Modified:
llvm/trunk/lib/Passes/PassBuilder.cpp
Modified: llvm/trunk/lib/Passes/PassBuilder.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Passes/PassBuilder.cpp?rev=359852&r1=359851&r2=359852&view=diff
==============================================================================
--- llvm/trunk/lib/Passes/PassBuilder.cpp (original)
+++ llvm/trunk/lib/Passes/PassBuilder.cpp Thu May 2 17:15:23 2019
@@ -1235,7 +1235,6 @@ PassBuilder::buildLTODefaultPipeline(Opt
// FIXME: once we fix LoopPass Manager, add LICM here.
// FIXME: once we provide support for enabling MLSM, add it here.
- // FIXME: once we provide support for enabling NewGVN, add it here.
if (RunNewGVN)
MainFPM.addPass(NewGVNPass());
else
@@ -1248,7 +1247,7 @@ PassBuilder::buildLTODefaultPipeline(Opt
MainFPM.addPass(DSEPass());
// FIXME: at this point, we run a bunch of loop passes:
- // indVarSimplify, loopDeletion, loopInterchange, loopUnrool,
+ // indVarSimplify, loopDeletion, loopInterchange, loopUnroll,
// loopVectorize. Enable them once the remaining issue with LPM
// are sorted out.
@@ -1298,7 +1297,7 @@ PassBuilder::buildLTODefaultPipeline(Opt
// Now that we have optimized the program, discard unreachable functions.
MPM.addPass(GlobalDCEPass());
- // FIXME: Enable MergeFuncs, conditionally, after ported, maybe.
+ // FIXME: Maybe enable MergeFuncs conditionally after it's ported.
return MPM;
}
More information about the llvm-commits
mailing list