[llvm] r372434 - Fix missed case of switching getConstant to getTargetConstant. Try 2.
Sterling Augustine via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 20 15:26:55 PDT 2019
Author: saugustine
Date: Fri Sep 20 15:26:55 2019
New Revision: 372434
URL: http://llvm.org/viewvc/llvm-project?rev=372434&view=rev
Log:
Fix missed case of switching getConstant to getTargetConstant. Try 2.
Summary: This fixes a crasher introduced by r372338.
Reviewers: echristo, arsenm
Subscribers: wdng, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D67850
Added:
llvm/trunk/test/CodeGen/X86/isel-blendi-gettargetconstant.ll
Modified:
llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=372434&r1=372433&r2=372434&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Fri Sep 20 15:26:55 2019
@@ -17608,7 +17608,7 @@ SDValue X86TargetLowering::LowerINSERT_V
// operand form.
N1 = DAG.getNode(ISD::SCALAR_TO_VECTOR, dl, MVT::v4f32, N1);
return DAG.getNode(X86ISD::BLENDI, dl, VT, N0, N1,
- DAG.getConstant(1, dl, MVT::i8));
+ DAG.getTargetConstant(1, dl, MVT::i8));
}
// Create this as a scalar to vector..
N1 = DAG.getNode(ISD::SCALAR_TO_VECTOR, dl, MVT::v4f32, N1);
Added: llvm/trunk/test/CodeGen/X86/isel-blendi-gettargetconstant.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/isel-blendi-gettargetconstant.ll?rev=372434&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/X86/isel-blendi-gettargetconstant.ll (added)
+++ llvm/trunk/test/CodeGen/X86/isel-blendi-gettargetconstant.ll Fri Sep 20 15:26:55 2019
@@ -0,0 +1,19 @@
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc < %s -mtriple=x86_64-linux-gnu -mattr=sse4.2 | FileCheck %s
+
+define void @csrot_(float* %0) {
+; CHECK-LABEL: csrot_:
+; CHECK: # %bb.0:
+; CHECK-NEXT: movss {{.*#+}} xmm0 = mem[0],zero,zero,zero
+; CHECK-NEXT: movss {{.*#+}} xmm1 = mem[0],zero,zero,zero
+; CHECK-NEXT: xorps %xmm0, %xmm1
+; CHECK-NEXT: blendps {{.*#+}} xmm1 = xmm1[0],mem[1,2,3]
+; CHECK-NEXT: movlps %xmm1, (%rax)
+; CHECK-NEXT: retq
+1:
+ %2 = load float, float* %0, align 4
+ %3 = fsub float -0.000000e+00, %2
+ %4 = insertelement <2 x float> <float undef, float -0.000000e+00>, float %3, i32 0
+ store <2 x float> %4, <2 x float>* undef, align 8
+ ret void
+}
More information about the llvm-commits
mailing list