[PATCH] D104099: [NewPM] Remove SpeculateAroundPHIs pass from pipeline
    Roman Lebedev via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Jun 11 03:42:06 PDT 2021
    
    
  
lebedev.ri added inline comments.
================
Comment at: llvm/lib/Passes/PassBuilder.cpp:1449
-  // inserting redundancies into the program. This even includes SimplifyCFG.
-  OptimizePM.addPass(SpeculateAroundPHIsPass());
-
----------------
nikic wrote:
> As it has been in-tree for a while, I think it would be good to add a cl::opt option for people who have come to rely on it being present in some way.
I'm not convinced that the pass can be returned into it's current position in pipeline,
that is why the diff is what it is now.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104099/new/
https://reviews.llvm.org/D104099
    
    
More information about the cfe-commits
mailing list