[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