[PATCH] D104099: [NewPM] Remove SpeculateAroundPHIs pass from pipeline

Roman Lebedev via Phabricator via llvm-commits llvm-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 llvm-commits mailing list