mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-10-20 03:23:01 +02:00
[DAGCombiner] don't dereference an operand that doesn't exist (PR26070)
The bug was introduced with changes for x86-64 fp128: http://reviews.llvm.org/rL254653 I don't know why an x86 change is here, so I'll follow up in: http://reviews.llvm.org/D15134 Should fix: https://llvm.org/bugs/show_bug.cgi?id=26070 llvm-svn: 257200
This commit is contained in:
parent
d23d8b325b
commit
0ba1147ccc
@ -8795,20 +8795,21 @@ SDValue DAGCombiner::visitFSQRT(SDNode *N) {
|
||||
ZeroCmp, Zero, RV);
|
||||
}
|
||||
|
||||
/// copysign(x, fp_extend(y)) -> copysign(x, y)
|
||||
/// copysign(x, fp_round(y)) -> copysign(x, y)
|
||||
static inline bool CanCombineFCOPYSIGN_EXTEND_ROUND(SDNode *N) {
|
||||
// copysign(x, fp_extend(y)) -> copysign(x, y)
|
||||
// copysign(x, fp_round(y)) -> copysign(x, y)
|
||||
// Do not optimize out type conversion of f128 type yet.
|
||||
// For some target like x86_64, configuration is changed
|
||||
// to keep one f128 value in one SSE register, but
|
||||
// instruction selection cannot handle FCOPYSIGN on
|
||||
// SSE registers yet.
|
||||
SDValue N1 = N->getOperand(1);
|
||||
EVT N1VT = N1->getValueType(0);
|
||||
EVT N1Op0VT = N1->getOperand(0)->getValueType(0);
|
||||
return (N1.getOpcode() == ISD::FP_EXTEND ||
|
||||
N1.getOpcode() == ISD::FP_ROUND) &&
|
||||
(N1VT == N1Op0VT || N1Op0VT != MVT::f128);
|
||||
if ((N1.getOpcode() == ISD::FP_EXTEND ||
|
||||
N1.getOpcode() == ISD::FP_ROUND)) {
|
||||
// Do not optimize out type conversion of f128 type yet.
|
||||
// For some targets like x86_64, configuration is changed to keep one f128
|
||||
// value in one SSE register, but instruction selection cannot handle
|
||||
// FCOPYSIGN on SSE registers yet.
|
||||
EVT N1VT = N1->getValueType(0);
|
||||
EVT N1Op0VT = N1->getOperand(0)->getValueType(0);
|
||||
return (N1VT == N1Op0VT || N1Op0VT != MVT::f128);
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
SDValue DAGCombiner::visitFCOPYSIGN(SDNode *N) {
|
||||
|
@ -18,3 +18,19 @@ define x86_fp80 @foo(x86_fp80 %a) {
|
||||
}
|
||||
|
||||
declare x86_fp80 @copysignl(x86_fp80, x86_fp80) nounwind readnone
|
||||
|
||||
; This would crash:
|
||||
; https://llvm.org/bugs/show_bug.cgi?id=26070
|
||||
|
||||
define float @pr26070() {
|
||||
%c = call float @copysignf(float 1.0, float undef) readnone
|
||||
ret float %c
|
||||
|
||||
; CHECK-LABEL: pr26070:
|
||||
; CHECK: andps
|
||||
; CHECK-NEXT: orps
|
||||
; CHECK-NEXT: retq
|
||||
}
|
||||
|
||||
declare float @copysignf(float, float)
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user