diff options
| author | Mon P Wang <wangmp@apple.com> | 2009-03-14 00:25:19 +0000 |
|---|---|---|
| committer | Mon P Wang <wangmp@apple.com> | 2009-03-14 00:25:19 +0000 |
| commit | 0b7a786842fd6601b99b1073e5393ebc928c1e6c (patch) | |
| tree | 24a68edfba7fc0be6a5e1dc70e967140934d220c /lib | |
| parent | d00d2feab2ddb3d0c89f6ca20c0a04216ddc77e9 (diff) | |
| download | external_llvm-0b7a786842fd6601b99b1073e5393ebc928c1e6c.zip external_llvm-0b7a786842fd6601b99b1073e5393ebc928c1e6c.tar.gz external_llvm-0b7a786842fd6601b99b1073e5393ebc928c1e6c.tar.bz2 | |
Avoid doing the transformation c ? 1.0 : 2.0 as load { 2.0, 1.0 } + c*4
if FPConstant is legal because if the FPConstant doesn't need to be stored
in a constant pool, the transformation is unlikely to be profitable.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@66994 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index 7cacbd3..3e517cf 100644 --- a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -5654,10 +5654,13 @@ SDValue DAGCombiner::SimplifySelectCC(DebugLoc DL, SDValue N0, SDValue N1, // it replaces two constant pool loads with one. We only do this if the FP // type is known to be legal, because if it isn't, then we are before legalize // types an we want the other legalization to happen first (e.g. to avoid - // messing with soft float). + // messing with soft float) and if the ConstantFP is not legal, because if + // it is legal, we may not need to store the FP constant in a constant pool. if (ConstantFPSDNode *TV = dyn_cast<ConstantFPSDNode>(N2)) if (ConstantFPSDNode *FV = dyn_cast<ConstantFPSDNode>(N3)) { if (TLI.isTypeLegal(N2.getValueType()) && + (TLI.getOperationAction(ISD::ConstantFP, N2.getValueType()) != + TargetLowering::Legal) && // If both constants have multiple uses, then we won't need to do an // extra load, they are likely around in registers for other users. (TV->hasOneUse() || FV->hasOneUse())) { |