[PATCH] D60295: [CodeGen] Replace -max-jump-table-size with -max-jump-table-targets

[Evandro Menezes via Phabricator via llvm-commits]

evandro marked 4 inline comments as done.
evandro added inline comments.


================
Comment at: llvm/lib/CodeGen/SwitchLoweringUtils.cpp:54
+
+  // Accumulated number of cases in each cluster and those prior to it.
+  SmallVector<unsigned, 8> AllCases(Last + 1);
----------------
hans wrote:
> The point of the AllCases vector was to avoid having to iterate from First to Last each time to compute the number of cases.
> 
> Now that we're iterating from First to Last anyway to count the number of targets, there's no point to this optimization really, and we might as well count the number of cases while counting the number of targets. That would make the code simpler.
Of course!


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

https://reviews.llvm.org/D60295