ARM: move llvm_unreachable use
authorSaleem Abdulrasool <compnerd@compnerd.org>
Wed, 30 Apr 2014 05:12:41 +0000 (05:12 +0000)
committerSaleem Abdulrasool <compnerd@compnerd.org>
Wed, 30 Apr 2014 05:12:41 +0000 (05:12 +0000)
When building with -Werror=covered-switch-default (as on the buildbots), the
build would fail since all cases are covered by the switch.  Move the
llvm_unreachable to the end of the function as an annotation.

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

lib/Target/ARM/ARMExpandPseudoInsts.cpp

index f9b612f5890a0754c796c051760849a1f7049c73..72a9b648dca7caffd7642cde618f7c8062604885 100644 (file)
@@ -620,7 +620,6 @@ static bool IsAnAddressOperand(const MachineOperand &MO) {
   // operand is not a symbol reference, we return that it is a symbol reference.
   // This is important as the load pair may not be split up Windows.
   switch (MO.getType()) {
-  default: llvm_unreachable("unhandled machine operand type");
   case MachineOperand::MO_Register:
   case MachineOperand::MO_Immediate:
   case MachineOperand::MO_CImmediate:
@@ -646,6 +645,7 @@ static bool IsAnAddressOperand(const MachineOperand &MO) {
   case MachineOperand::MO_CFIIndex:
     return false;
   }
+  llvm_unreachable("unhandled machine operand type");
 }
 
 void ARMExpandPseudo::ExpandMOV32BitImm(MachineBasicBlock &MBB,