[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