diff options
| author | Nate Begeman <natebegeman@mac.com> | 2005-08-14 18:38:32 +0000 |
|---|---|---|
| committer | Nate Begeman <natebegeman@mac.com> | 2005-08-14 18:38:32 +0000 |
| commit | 2d56e72f09337a1cab3ba199a9467c159296f3fa (patch) | |
| tree | 4271c2d383b96ef4ae582b8b51214a3fc349c0c7 /lib/Target/PowerPC | |
| parent | 889f2c1acf930275e277d1438838a28c7ccc65df (diff) | |
| download | external_llvm-2d56e72f09337a1cab3ba199a9467c159296f3fa.zip external_llvm-2d56e72f09337a1cab3ba199a9467c159296f3fa.tar.gz external_llvm-2d56e72f09337a1cab3ba199a9467c159296f3fa.tar.bz2 | |
Fix last night's PPC32 regressions by
1. Not selecting the false value of a select_cc in the false arm, which
isn't legal for nested selects.
2. Actually returning the node we created and Legalized in the FP_TO_UINT
Expander.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@22789 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/PowerPC')
| -rw-r--r-- | lib/Target/PowerPC/PPCISelPattern.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/Target/PowerPC/PPCISelPattern.cpp b/lib/Target/PowerPC/PPCISelPattern.cpp index 80a405e..9831111 100644 --- a/lib/Target/PowerPC/PPCISelPattern.cpp +++ b/lib/Target/PowerPC/PPCISelPattern.cpp @@ -2162,9 +2162,8 @@ unsigned ISel::SelectExpr(SDOperand N, bool Recording) { // If the False value only has one use, we can generate better code by // selecting it in the fallthrough basic block rather than here, which // increases register pressure. - bool FalseHasOneUse = N.getOperand(3).Val->hasOneUse(); unsigned TrueValue = SelectExpr(N.getOperand(2)); - unsigned FalseValue = FalseHasOneUse ? 0 : SelectExpr(N.getOperand(3)); + unsigned FalseValue = SelectExpr(N.getOperand(3)); unsigned CCReg = SelectCC(N.getOperand(0), N.getOperand(1), CC); Opc = getBCCForSetCC(CC); @@ -2195,7 +2194,6 @@ unsigned ISel::SelectExpr(SDOperand N, bool Recording) { // %FalseValue = ... // # fallthrough to sinkMBB BB = copy0MBB; - if (FalseHasOneUse) FalseValue = SelectExpr(N.getOperand(3)); // Update machine-CFG edges BB->addSuccessor(sinkMBB); |
