GlobalISel: fall back gracefully when we hit unhandled legalizer default.
authorTim Northover <tnorthover@apple.com>
Tue, 6 Dec 2016 19:02:15 +0000 (19:02 +0000)
committerTim Northover <tnorthover@apple.com>
Tue, 6 Dec 2016 19:02:15 +0000 (19:02 +0000)
llvm-svn: 288840

llvm/lib/CodeGen/GlobalISel/LegalizerInfo.cpp
llvm/test/CodeGen/AArch64/GlobalISel/arm64-fallback.ll

index 2b99a55..e496620 100644 (file)
@@ -93,7 +93,9 @@ LegalizerInfo::getAction(const InstrAspect &Aspect) const {
     if (DefaultAction != DefaultActions.end() && DefaultAction->second == Legal)
       return std::make_pair(Legal, Ty);
 
-    assert(DefaultAction->second == NarrowScalar && "unexpected default");
+    if (DefaultAction == DefaultActions.end() ||
+        DefaultAction->second != NarrowScalar)
+      return std::make_pair(Unsupported, LLT());
     return findLegalAction(Aspect, NarrowScalar);
   }
 
index d164e2e..e4a2bb2 100644 (file)
@@ -74,3 +74,11 @@ define void @odd_type(i42* %addr) {
 define void @sequence_mapping([2 x i64] %in) {
   ret void
 }
+
+  ; Legalizer was asserting when it enountered an unexpected default action.
+; FALLBACK-WITH-REPORT-ERR: warning: Instruction selection used fallback path for legal_default
+; FALLBACK-WITH-REPORT-LABEL: legal_default:
+define void @legal_default(i64 %in) {
+  insertvalue [2 x i64] undef, i64 %in, 0
+  ret void
+}