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

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

I was trying to determine what else the current check is protecting like vectors or floating point. It looks like ValidLookupTableConstant may largely filter out vectors. Though the recursion in ValidLookupTableConstant for ConstantExpr seems like it might be incomplete. It only visits operand 0. Though there are probably no operations that takes a scalar as a first operand and returns a vector result.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107233