[PATCH] D54223: [SimpleLoopUnswitch] adding cost multiplier to cap exponential unswitch with

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc accepted this revision.
chandlerc added a comment.

still LGTM.



================
Comment at: lib/Transforms/Scalar/SimpleLoopUnswitch.cpp:2303
+      (isGuard(&TI) ||
+       1 >= llvm::count_if(successors(&TI), [&L](BasicBlock *SuccBB) {
+         return L.contains(SuccBB);
----------------
I think the yoda-comparison makes this harder to read. I'd rather say `count_if(...) <= 1`.


Repository:
  rL LLVM

https://reviews.llvm.org/D54223