[llvm] r279487 - Fix crash from assert in r279466.

Pete Cooper via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 22 15:27:06 PDT 2016


Author: pete
Date: Mon Aug 22 17:27:05 2016
New Revision: 279487

URL: http://llvm.org/viewvc/llvm-project?rev=279487&view=rev
Log:
Fix crash from assert in r279466.

The assert in r279466 checks that we call the correct version of
Intrinsic::getName.  The version which accepts only an ID should not
be used for intrinsics with overloaded types.  The global-isel
code was calling the wrong version.  The test CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll
will ensure that we call the correct version from now on.

Modified:
    llvm/trunk/lib/CodeGen/MIRPrinter.cpp

Modified: llvm/trunk/lib/CodeGen/MIRPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MIRPrinter.cpp?rev=279487&r1=279486&r2=279487&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/MIRPrinter.cpp (original)
+++ llvm/trunk/lib/CodeGen/MIRPrinter.cpp Mon Aug 22 17:27:05 2016
@@ -883,7 +883,7 @@ void MIPrinter::print(const MachineOpera
   case MachineOperand::MO_IntrinsicID: {
     Intrinsic::ID ID = Op.getIntrinsicID();
     if (ID < Intrinsic::num_intrinsics)
-      OS << "intrinsic(@" << Intrinsic::getName(ID) << ')';
+      OS << "intrinsic(@" << Intrinsic::getName(ID, None) << ')';
     else {
       const MachineFunction &MF = *Op.getParent()->getParent()->getParent();
       const TargetIntrinsicInfo *TII = MF.getTarget().getIntrinsicInfo();




More information about the llvm-commits mailing list