[PATCH] D107233: [SimplifyCFG] Enable switch to lookup table for more types.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

If the type is very large and the constants are small we will generate a very large table which could be a lot worse than the original switch. Of course we could use a smaller type for the lookup table and the extend the constants in such a case, but that's additional work I can't sign up for. So I think it is still reasonable to have some kind of limit in place here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107233