[llvm] [SimplifyCFG] Delete the unnecessary range check for small mask operation (PR #65835)

[Nikita Popov via llvm-commits]

================
@@ -6552,6 +6552,20 @@ static bool SwitchToLookupTable(SwitchInst *SI, IRBuilder<> &Builder,
     // Note: We call removeProdecessor later since we need to be able to get the
     // PHI value for the default case in case we're using a bit mask.
   } else {
+    const APInt *MaskC;
+    if (UseSwitchConditionAsTableIndex &&
+        match(TableIndex, m_And(m_Value(), m_APInt(MaskC)))) {
+      APInt SCVPlus1 = *MaskC + 1;
+      const uint64_t *RawData = MaskC->getRawData();
+      // Expand the size of the table when the MaskC < 64.
+      if (SCVPlus1.isPowerOf2() && SCVPlus1.ule(64) && RawData) {
----------------
nikic wrote:

This should not hardcode 64 bits. Like the rest of the transform, the proper criterion here is whether the new mask size fits in a legal register.

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