maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Check we aren't trying to convert PPC long double. 

This fixes the testsuite failure on ppcf128-4.ll.

llvm-svn: 49994
This commit is contained in:
Dale Johannesen 2008-04-20 18:23:46 +00:00
parent f77618e6e8
commit aac27592f0
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
llvm/include/llvm/CodeGen/SelectionDAGNodes.h
View File

@ -1500,7 +1500,10 @@ public:
/// We leave the version with the double argument here because it's just so
/// convenient to write "2.0" and the like. Without this function we'd
/// have to duplicate its logic everywhere it's called.
bool isExactlyValue(double V) const {
bool isExactlyValue(double V) const {
// convert is not supported on this type
if (&Value.getSemantics() == &APFloat::PPCDoubleDouble)
return false;
APFloat Tmp(V);
Tmp.convert(Value.getSemantics(), APFloat::rmNearestTiesToEven);
return isExactlyValue(Tmp);

9
llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
View File

@ -74,12 +74,9 @@ bool ConstantFPSDNode::isValueValidForType(MVT::ValueType VT,
const APFloat& Val) {
assert(MVT::isFloatingPoint(VT) && "Can only convert between FP types");
// Anything can be extended to ppc long double.
if (VT == MVT::ppcf128)
return true;
// PPC long double cannot be shrunk to anything though.
if (&Val.getSemantics() == &APFloat::PPCDoubleDouble)
// PPC long double cannot be converted to any other type.
if (VT == MVT::ppcf128 ||
&Val.getSemantics() == &APFloat::PPCDoubleDouble)
return false;
// convert modifies in place, so make a copy.