RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-10-21 03:53:04 +02:00
Code Issues Projects Releases Wiki Activity

[PowerPC] Fix the int2fp(fp2int(x)) DAGCombine to ignore ppc_fp128

Browse Source 
This DAGCombine was creating custom SDAG nodes with an illegal ppc_fp128
operand type because it was triggering on f64/f32 int2fp(fp2int(ppc_fp128 x)),
but shouldn't (it should only apply to f32/f64 types). The result was a crash.

llvm-svn: 245530
This commit is contained in:
Hal Finkel 2015-08-20 01:18:20 +00:00
parent 6cf20390c0
commit c4b6cee81f
2 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/Target/PowerPC/PPCISelLowering.cpp
View File

@ -9991,6 +9991,9 @@ SDValue PPCTargetLowering::combineFPToIntToFP(SDNode *N,
if (Src.getValueType() == MVT::f32) {
Src = DAG.getNode(ISD::FP_EXTEND, dl, MVT::f64, Src);
DCI.AddToWorklist(Src.getNode());
} else if (Src.getValueType() != MVT::f64) {
// Make sure that we don't pick up a ppc_fp128 source value.
return SDValue();
}
unsigned FCTOp =

16
test/CodeGen/PowerPC/fp2int2fp-ppcfp128.ll Normal file
View File

@ -0,0 +1,16 @@
; RUN: llc -mcpu=a2 < %s | FileCheck %s
target datalayout = "E-m:e-i64:64-n32:64"
target triple = "powerpc64-bgq-linux"
define linkonce_odr double @test1() {
entry:
%conv6.i.i = fptosi ppc_fp128 undef to i64
%conv.i = sitofp i64 %conv6.i.i to double
ret double %conv.i
; CHECK-LABEL: @test1
; CHECK: bl __fixtfdi
; CHECK: fcfid
; CHECK: blr
}