Subzero, MIPS32: UnimplementedError removed from most common crashers

src/IceTargetLoweringMIPS32.cpp

Index: src/IceTargetLoweringMIPS32.cpp
diff --git a/src/IceTargetLoweringMIPS32.cpp b/src/IceTargetLoweringMIPS32.cpp
index f17b02d757b4a4eea4f5ddafffca3938f7106f2a..430820eff63985477dc81ebe22995bec2e2c45fe 100644
--- a/src/IceTargetLoweringMIPS32.cpp
+++ b/src/IceTargetLoweringMIPS32.cpp
@@ -480,14 +480,12 @@ void TargetMIPS32::emitVariable(const Variable *Var) const {
   const Type FrameSPTy = IceType_i32;
   if (Var->hasReg()) {
     Str << '$' << getRegName(Var->getRegNum(), Var->getType());
-    return;
   } else {
     int32_t Offset = Var->getStackOffset();
     Str << Offset;

[Jim Stichnoth, 2016/06/22 15:53:56]

The only issue here that I can think of is if Offset is too large to be encoded
in this address mode.  But that will surely get tested elsewhere, so it seems OK
now.

     Str << "($" << getRegName(getFrameOrStackReg(), FrameSPTy);
     Str << ")";
   }
-  UnimplementedError(getFlags());
 }
 
 TargetMIPS32::CallingConv::CallingConv()
@@ -2123,13 +2121,11 @@ void TargetDataMIPS32::lowerGlobals(const VariableDeclarationList &Vars,
 void TargetDataMIPS32::lowerConstants() {
   if (getFlags().getDisableTranslation())
     return;
-  UnimplementedError(getFlags());
 }
 
 void TargetDataMIPS32::lowerJumpTables() {
   if (getFlags().getDisableTranslation())
     return;
-  UnimplementedError(getFlags());
 }
 
 // Helper for legalize() to emit the right code to lower an operand to a