[PATCH] D88438: BreakCriticalEdges: do not split the critical edge from a CallBr indirect successor

[Nick Desaulniers via Phabricator via llvm-commits]

nickdesaulniers added inline comments.


================
Comment at: llvm/test/Transforms/LoopStrengthReduce/callbr-critical-edge-splitting.ll:1
+; RUN: opt < %s -o /dev/null -loop-reduce
+
----------------
MaskRay wrote:
> Just use the new PM way: `-passes='loop(loop-reduce)'`
If I do that for both tests:
1. this one passes before this patch has been applied. (bad)(lol, wut)
2. the other test still fails. (good)

Is there something funny with NPM that I need to specify additional passes in addition to `loop(loop-reduce)`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88438