Pass the error string directly to llvm_unreachable instead of the residual

(0 && "error").  Rough consensus seems to be that g++ *should* be diagnosing
this because the pointer makes it not an ICE in c++03.  Everyone agrees that
the current standard is silly and null-pointer-ness should not be based on
ICE-ness.  Excellent fight scene in Act II, denouement weak, two stars.

llvm-svn: 91644

llvm/lib/Target/MSP430/AsmPrinter/MSP430MCInstLower.cpp

@@ -34,7 +34,7 @@ GetGlobalAddressSymbol(const MachineOperand &MO) const {
   Mang.getNameWithPrefix(Name, GV, false);
 
   switch (MO.getTargetFlags()) {
-  default: llvm_unreachable(0 && "Unknown target flag on GV operand");
+  default: llvm_unreachable("Unknown target flag on GV operand");
   case 0: break;
   }