[Outliner] Remove output for offset range check
authorJessica Paquette <jpaquette@apple.com>
Mon, 20 Mar 2017 15:51:45 +0000 (15:51 +0000)
committerJessica Paquette <jpaquette@apple.com>
Mon, 20 Mar 2017 15:51:45 +0000 (15:51 +0000)
Forgot to remove some output before committing last time. (Instruction fixups
don't actually overflow anywhere in the test suite so far, so I missed it).

To prevent the outliner from screaming "Overflow!" in the event that that
does happen, this commit removes that output.

llvm-svn: 298260

llvm/lib/Target/AArch64/AArch64InstrInfo.cpp

index 308f9da..19d5490 100644 (file)
@@ -4348,10 +4348,8 @@ AArch64InstrInfo::getOutliningType(MachineInstr &MI) const {
       // TODO: We should really test what happens if an instruction overflows.
       // This is tricky to test with IR tests, but when the outliner is moved
       // to a MIR test, it really ought to be checked.
-      if (Offset + 16 < MinOffset || Offset + 16 > MaxOffset) {
+      if (Offset + 16 < MinOffset || Offset + 16 > MaxOffset)
         errs() << "Overflow!\n";
-        return MachineOutlinerInstrType::Illegal;
-      }
 
       // It's in range, so we can outline it.
       return MachineOutlinerInstrType::Legal;