[PATCH] D61159: [SimplifyCFG] Run ReduceSwitchRange unconditionally, generalize

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:5529
+  unsigned BestIndex = 0;
+  for (unsigned i = 1;i != Values.size();i++) {
+    if (Values[i] - Values[i-1] > BestDistance) {
----------------
jmolloy wrote:
> Please use capitals for induction variables, calculate the end value once, and use preincrement as per style guide:
> 
>   for (unsigned I = 1, E = Values.size(); I != E; ++I) {
This wasn't done


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

https://reviews.llvm.org/D61159