]> granicus.if.org Git - llvm/commitdiff
[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.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@298260 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/AArch64/AArch64InstrInfo.cpp

index 308f9da5cd34a8cd56364ab7c788b9efb3532012..19d54904d4efec7511fabe11732df9096bd2f678 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;