[region-isolation] Now that we support all instructions, remove Unhandled... all future instructions added must be handled!

Author: gottesmm

lib/SILOptimizer/Analysis/RegionAnalysis.cpp

@@ -1172,10 +1172,6 @@ enum class TranslationSemantics {
   /// without updating this code correctly. This is most likely driver error and
   /// should be caught in testing when we assert.
   AssertingIfNonSendable,
-
-  /// An instruction that we do not handle yet. Just for now during bring
-  /// up. Will be removed.
-  Unhandled,
 };
 
 } // namespace
@@ -1218,9 +1214,6 @@ llvm::raw_ostream &operator<<(llvm::raw_ostream &os,
   case TranslationSemantics::AssertingIfNonSendable:
     os << "asserting_if_nonsendable";
     return os;
-  case TranslationSemantics::Unhandled:
-    os << "unhandled";
-    return os;
   }
 
   llvm_unreachable("Covered switch isn't covered?!");
@@ -2120,9 +2113,6 @@ class PartitionOpTranslator {
           "transfer-non-sendable: Found banned instruction?!");
       return;
 
-    case TranslationSemantics::Unhandled:
-      LLVM_DEBUG(llvm::dbgs() << "Unhandled inst: " << *inst);
-      return;
     case TranslationSemantics::AssertingIfNonSendable:
       // Do not error if all of our operands are sendable.
       if (llvm::none_of(inst->getOperandValues(), [&](SILValue value) {