[PATCH] D76389: [NewPM] Run the Speculative Execution Pass only if the target has divergent branches

Leonard Chan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 6 14:10:38 PDT 2020


leonardchan marked an inline comment as done.
leonardchan added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilder.cpp:435
   if (Level.getSpeedupLevel() > 1) {
-    FPM.addPass(SpeculativeExecutionPass());
+    FPM.addPass(SpeculativeExecutionIfHasBranchDivergencePass());
 
----------------
arsenm wrote:
> I don't expect this to be a separate pass, just added based on a divergent target
Updated such that the current pass is used, although it's a bit more difficult to test since the same pass name will appear when dumping the passes.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76389/new/

https://reviews.llvm.org/D76389





More information about the llvm-commits mailing list