Skip to content

Commit 86e2f16

Browse files
committed
Tidy up a comment, fix a typo, remove a comment that's obsolete.
llvm-svn: 359852
1 parent 3947968 commit 86e2f16

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

llvm/lib/Passes/PassBuilder.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1235,7 +1235,6 @@ PassBuilder::buildLTODefaultPipeline(OptimizationLevel Level, bool DebugLogging,
12351235

12361236
// FIXME: once we fix LoopPass Manager, add LICM here.
12371237
// FIXME: once we provide support for enabling MLSM, add it here.
1238-
// FIXME: once we provide support for enabling NewGVN, add it here.
12391238
if (RunNewGVN)
12401239
MainFPM.addPass(NewGVNPass());
12411240
else
@@ -1248,7 +1247,7 @@ PassBuilder::buildLTODefaultPipeline(OptimizationLevel Level, bool DebugLogging,
12481247
MainFPM.addPass(DSEPass());
12491248

12501249
// FIXME: at this point, we run a bunch of loop passes:
1251-
// indVarSimplify, loopDeletion, loopInterchange, loopUnrool,
1250+
// indVarSimplify, loopDeletion, loopInterchange, loopUnroll,
12521251
// loopVectorize. Enable them once the remaining issue with LPM
12531252
// are sorted out.
12541253

@@ -1298,7 +1297,7 @@ PassBuilder::buildLTODefaultPipeline(OptimizationLevel Level, bool DebugLogging,
12981297
// Now that we have optimized the program, discard unreachable functions.
12991298
MPM.addPass(GlobalDCEPass());
13001299

1301-
// FIXME: Enable MergeFuncs, conditionally, after ported, maybe.
1300+
// FIXME: Maybe enable MergeFuncs conditionally after it's ported.
13021301
return MPM;
13031302
}
13041303

0 commit comments

Comments
 (0)