From c7013ec3a9e09a5aeb00bd6481c0199eadbe0352 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Fri, 13 May 2005 04:45:13 +0000 Subject: [PATCH] do not call expandop on the same value more than once. This fixes X86/2004-02-22-Casts.llx llvm-svn: 21919 --- lib/CodeGen/SelectionDAG/LegalizeDAG.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp b/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp index 67b26dca9de..3bea8c69ee7 100644 --- a/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp +++ b/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp @@ -2131,10 +2131,6 @@ ExpandIntToFP(bool isSigned, MVT::ValueType DestTy, SDOperand Source) { assert(Source.getValueType() == MVT::i64 && "Only handle expand from i64!"); if (!isSigned) { - // If this is unsigned, and not supported, first perform the conversion to - // signed, then adjust the result if the sign bit is set. - SDOperand SignedConv = ExpandIntToFP(true, DestTy, Source); - assert(Source.getValueType() == MVT::i64 && "This only works for 64-bit -> FP"); // The 64-bit value loaded will be incorrectly if the 'sign bit' of the @@ -2143,6 +2139,11 @@ ExpandIntToFP(bool isSigned, MVT::ValueType DestTy, SDOperand Source) { SDOperand Lo, Hi; ExpandOp(Source, Lo, Hi); + // If this is unsigned, and not supported, first perform the conversion to + // signed, then adjust the result if the sign bit is set. + SDOperand SignedConv = ExpandIntToFP(true, DestTy, + DAG.getNode(ISD::BUILD_PAIR, Source.getValueType(), Lo, Hi)); + SDOperand SignSet = DAG.getSetCC(ISD::SETLT, TLI.getSetCCResultTy(), Hi, DAG.getConstant(0, Hi.getValueType())); SDOperand Zero = getIntPtrConstant(0), Four = getIntPtrConstant(4);