[llvm-exegesis][mips] Fix -Wunused-function after D72858

MaskRay · MaskRay · commit ed9cc6404e7f · 2020-01-18T13:57:19.000-08:00
diff --git a/llvm/tools/llvm-exegesis/lib/Mips/Target.cpp b/llvm/tools/llvm-exegesis/lib/Mips/Target.cpp
@@ -14,12 +14,12 @@
 namespace llvm {
 namespace exegesis {
 
+#ifndef NDEBUG
 // Returns an error if we cannot handle the memory references in this
 // instruction.
 static Error isInvalidMemoryInstr(const Instruction &Instr) {
   switch (Instr.Description.TSFlags & MipsII::FormMask) {
   default:
-    return Error::success();
     llvm_unreachable("Unknown FormMask value");
   // These have no memory access.
   case MipsII::Pseudo:
@@ -36,6 +36,7 @@ static Error isInvalidMemoryInstr(const Instruction &Instr) {
     return make_error<Failure>("unsupported opcode: non uniform memory access");
   }
 }
+#endif
 
 // Helper to fill a memory operand with a value.
 static void setMemOp(InstructionTemplate &IT, int OpIdx,
