[llvm] [SimplifyCFG] Replace unreachable switch lookup table holes with poison (PR #94990)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 17 03:48:08 PDT 2024


================
@@ -6322,6 +6333,9 @@ SwitchLookupTable::SwitchLookupTable(
       if (!ConstVal) {
         // This is an undef. We could deal with it, but undefs in lookup tables
         // are very seldom. It's probably not worth the additional complexity.
+        // TODO: In switches with holes and an unreachable default branch, this
----------------
DaMatrix wrote:

I wasn't planning on fixing it in this PR, since it seemed like too big of a change. I've got another PR lined up which fixes this TODO, which I was planning to open as soon as this gets merged. Would you prefer I made that part of this PR?

https://github.com/llvm/llvm-project/pull/94990


More information about the llvm-commits mailing list