[llvm] r316694 - [GISel]: Missed checking if it's okay to create a G_CONSTANT of DstTy in the legalizationCombiner

Aditya Nandakumar via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 26 13:13:54 PDT 2017


Author: aditya_nandakumar
Date: Thu Oct 26 13:13:54 2017
New Revision: 316694

URL: http://llvm.org/viewvc/llvm-project?rev=316694&view=rev
Log:
[GISel]: Missed checking if it's okay to create a G_CONSTANT of DstTy in the legalizationCombiner

Modified:
    llvm/trunk/include/llvm/CodeGen/GlobalISel/LegalizerCombiner.h

Modified: llvm/trunk/include/llvm/CodeGen/GlobalISel/LegalizerCombiner.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/GlobalISel/LegalizerCombiner.h?rev=316694&r1=316693&r2=316694&view=diff
==============================================================================
--- llvm/trunk/include/llvm/CodeGen/GlobalISel/LegalizerCombiner.h (original)
+++ llvm/trunk/include/llvm/CodeGen/GlobalISel/LegalizerCombiner.h Thu Oct 26 13:13:54 2017
@@ -88,7 +88,8 @@ public:
       unsigned DstReg = MI.getOperand(0).getReg();
       LLT DstTy = MRI.getType(DstReg);
       if (isInstUnsupported(TargetOpcode::G_SHL, DstTy) ||
-          isInstUnsupported(TargetOpcode::G_ASHR, DstTy))
+          isInstUnsupported(TargetOpcode::G_ASHR, DstTy) ||
+          isInstUnsupported(TargetOpcode::G_CONSTANT, DstTy))
         return false;
       DEBUG(dbgs() << ".. Combine MI: " << MI;);
       Builder.setInstr(MI);




More information about the llvm-commits mailing list