[PATCH] D156612: [SimplifyCFG] Find the smallest table considering overflow in `switchToLookupTable`

Tue Sep 19 00:57:13 PDT 2023

DianQK added a comment.

In D156612#4647819 <https://reviews.llvm.org/D156612#4647819>, @khei4 wrote:

> I'm still not sure about the correctness of this. It seems like only setting the BeginCaseVal and EndCaseVal to the appropriate values is enough to handle overflowing switch cases. Test transformations themselves look correct!

I seem to understand what you mean. Yes, this patch is looking for the appropriate BeginCaseVal and EndCaseVal.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156612