[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