[PATCH] D25212: Add support to tune jump tables

Evandro Menezes via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 3 16:14:43 PDT 2016


evandro marked an inline comment as done.
evandro added inline comments.


> hans wrote in SelectionDAGBuilder.cpp:8550
> I really don't like this. The purpose of this algorithm is to find jump tables. If you don't want to find jump tables, don't run it. If you think it's generating too large or sparse jump tables, tweak those parameters.

I'd like to have the option to choose more of fewer jump tables when the number of partitions is the same.  It's not clear to me why only one possibility was cast in the code originally, but it doesn't seem to me to be always the best choice.

Repository:
  rL LLVM

https://reviews.llvm.org/D25212





More information about the llvm-commits mailing list