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

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev added inline comments.


================
Comment at: lib/Transforms/Scalar/SimpleLoopUnswitch.cpp:2300-2301
+  // of candidates is small.
+  unsigned CandidatesPower = std::max(
+      (int)UnswitchCandidates.size() - (int)UnswitchCandidatesBottomCap, 0);
+
----------------
chandlerc wrote:
> Just return here?
Nope, power 0 means 1 for this part of a multiplier.
Still can get nontrivial siblings multiplier part.


Repository:
  rL LLVM

https://reviews.llvm.org/D54223