mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-12-30 07:33:23 +00:00
Fix a (u)comiss intrinsic lowering bug. It was using anyext which can return junk in higher bits. Patch by Nate Begeman.
llvm-svn: 54903
This commit is contained in:
parent
0a5e079b8e
commit
6534c78383
@ -1515,6 +1515,8 @@ SDNode *X86DAGToDAGISel::Select(SDValue N) {
|
||||
SDValue N0 = Node->getOperand(0);
|
||||
// Get the subregsiter index for the type to extend.
|
||||
MVT N0VT = N0.getValueType();
|
||||
// FIXME: In x86-32, 8-bit value may be in AH, etc. which don't have
|
||||
// super-registers.
|
||||
unsigned Idx = (N0VT == MVT::i32) ? X86::SUBREG_32BIT :
|
||||
(N0VT == MVT::i16) ? X86::SUBREG_16BIT :
|
||||
(Subtarget->is64Bit()) ? X86::SUBREG_8BIT : 0;
|
||||
|
@ -5419,8 +5419,8 @@ X86TargetLowering::LowerINTRINSIC_WO_CHAIN(SDValue Op, SelectionDAG &DAG) {
|
||||
|
||||
SDValue Cond = DAG.getNode(Opc, MVT::i32, LHS, RHS);
|
||||
SDValue SetCC = DAG.getNode(X86ISD::SETCC, MVT::i8,
|
||||
DAG.getConstant(X86CC, MVT::i8), Cond);
|
||||
return DAG.getNode(ISD::ANY_EXTEND, MVT::i32, SetCC);
|
||||
DAG.getConstant(X86CC, MVT::i8), Cond);
|
||||
return DAG.getNode(ISD::ZERO_EXTEND, MVT::i32, SetCC);
|
||||
}
|
||||
|
||||
// Fix vector shift instructions where the last operand is a non-immediate
|
||||
|
9
test/CodeGen/X86/2008-08-17-UComiCodeGenBug.ll
Normal file
9
test/CodeGen/X86/2008-08-17-UComiCodeGenBug.ll
Normal file
@ -0,0 +1,9 @@
|
||||
; RUN: llvm-as < %s | llc -mtriple=x86_64-apple-darwin | grep movzbl
|
||||
|
||||
define i32 @foo(<4 x float> %a, <4 x float> %b) nounwind {
|
||||
entry:
|
||||
tail call i32 @llvm.x86.sse.ucomige.ss( <4 x float> %a, <4 x float> %b ) nounwind readnone
|
||||
ret i32 %0
|
||||
}
|
||||
|
||||
declare i32 @llvm.x86.sse.ucomige.ss(<4 x float>, <4 x float>) nounwind readnone
|
@ -1,99 +0,0 @@
|
||||
; RUN: llvm-as < %s | opt -loop-reduce | llvm-dis | grep "phi double" | count 1
|
||||
|
||||
define void @foobar(i32 %n) nounwind {
|
||||
entry:
|
||||
icmp eq i32 %n, 0 ; <i1>:0 [#uses=2]
|
||||
br i1 %0, label %return, label %bb.nph
|
||||
|
||||
bb.nph: ; preds = %entry
|
||||
%umax = select i1 %0, i32 1, i32 %n ; <i32> [#uses=1]
|
||||
br label %bb
|
||||
|
||||
bb: ; preds = %bb, %bb.nph
|
||||
%i.03 = phi i32 [ 0, %bb.nph ], [ %indvar.next, %bb ] ; <i32> [#uses=3]
|
||||
tail call void @bar( i32 %i.03 ) nounwind
|
||||
uitofp i32 %i.03 to double ; <double>:1 [#uses=1]
|
||||
tail call void @foo( double %1 ) nounwind
|
||||
%indvar.next = add i32 %i.03, 1 ; <i32> [#uses=2]
|
||||
%exitcond = icmp eq i32 %indvar.next, %umax ; <i1> [#uses=1]
|
||||
br i1 %exitcond, label %return, label %bb
|
||||
|
||||
return: ; preds = %bb, %entry
|
||||
ret void
|
||||
}
|
||||
|
||||
; Unable to eliminate cast because the mantissa bits for double are not enough
|
||||
; to hold all of i64 IV bits.
|
||||
define void @foobar2(i64 %n) nounwind {
|
||||
entry:
|
||||
icmp eq i64 %n, 0 ; <i1>:0 [#uses=2]
|
||||
br i1 %0, label %return, label %bb.nph
|
||||
|
||||
bb.nph: ; preds = %entry
|
||||
%umax = select i1 %0, i64 1, i64 %n ; <i64> [#uses=1]
|
||||
br label %bb
|
||||
|
||||
bb: ; preds = %bb, %bb.nph
|
||||
%i.03 = phi i64 [ 0, %bb.nph ], [ %indvar.next, %bb ] ; <i64> [#uses=3]
|
||||
trunc i64 %i.03 to i32 ; <i32>:1 [#uses=1]
|
||||
tail call void @bar( i32 %1 ) nounwind
|
||||
uitofp i64 %i.03 to double ; <double>:2 [#uses=1]
|
||||
tail call void @foo( double %2 ) nounwind
|
||||
%indvar.next = add i64 %i.03, 1 ; <i64> [#uses=2]
|
||||
%exitcond = icmp eq i64 %indvar.next, %umax ; <i1> [#uses=1]
|
||||
br i1 %exitcond, label %return, label %bb
|
||||
|
||||
return: ; preds = %bb, %entry
|
||||
ret void
|
||||
}
|
||||
|
||||
; Unable to eliminate cast due to potentional overflow.
|
||||
define void @foobar3() nounwind {
|
||||
entry:
|
||||
tail call i32 (...)* @nn( ) nounwind ; <i32>:0 [#uses=1]
|
||||
icmp eq i32 %0, 0 ; <i1>:1 [#uses=1]
|
||||
br i1 %1, label %return, label %bb
|
||||
|
||||
bb: ; preds = %bb, %entry
|
||||
%i.03 = phi i32 [ 0, %entry ], [ %3, %bb ] ; <i32> [#uses=3]
|
||||
tail call void @bar( i32 %i.03 ) nounwind
|
||||
uitofp i32 %i.03 to double ; <double>:2 [#uses=1]
|
||||
tail call void @foo( double %2 ) nounwind
|
||||
add i32 %i.03, 1 ; <i32>:3 [#uses=2]
|
||||
tail call i32 (...)* @nn( ) nounwind ; <i32>:4 [#uses=1]
|
||||
icmp ugt i32 %4, %3 ; <i1>:5 [#uses=1]
|
||||
br i1 %5, label %bb, label %return
|
||||
|
||||
return: ; preds = %bb, %entry
|
||||
ret void
|
||||
}
|
||||
|
||||
; Unable to eliminate cast due to overflow.
|
||||
define void @foobar4() nounwind {
|
||||
entry:
|
||||
br label %bb.nph
|
||||
|
||||
bb.nph: ; preds = %entry
|
||||
br label %bb
|
||||
|
||||
bb: ; preds = %bb, %bb.nph
|
||||
%i.03 = phi i8 [ 0, %bb.nph ], [ %indvar.next, %bb ] ; <i32> [#uses=3]
|
||||
%tmp2 = sext i8 %i.03 to i32 ; <i32>:0 [#uses=1]
|
||||
tail call void @bar( i32 %tmp2 ) nounwind
|
||||
%tmp3 = uitofp i8 %i.03 to double ; <double>:1 [#uses=1]
|
||||
tail call void @foo( double %tmp3 ) nounwind
|
||||
%indvar.next = add i8 %i.03, 1 ; <i32> [#uses=2]
|
||||
%tmp = sext i8 %indvar.next to i32
|
||||
%exitcond = icmp eq i32 %tmp, 32767 ; <i1> [#uses=1]
|
||||
br i1 %exitcond, label %return, label %bb
|
||||
|
||||
return: ; preds = %bb, %entry
|
||||
ret void
|
||||
}
|
||||
|
||||
declare void @bar(i32)
|
||||
|
||||
declare void @foo(double)
|
||||
|
||||
declare i32 @nn(...)
|
||||
|
Loading…
Reference in New Issue
Block a user