[llvm-commits] CVS: llvm/lib/Target/PowerPC/PPC32ISelPattern.cpp
Chris Lattner
lattner at cs.uiuc.edu
Sun Aug 21 12:03:39 PDT 2005
Changes in directory llvm/lib/Target/PowerPC:
PPC32ISelPattern.cpp updated: 1.157 -> 1.158
---
Log message:
Simplify the logic for BRTWOWAY_CC handling. The isel code already
simplifies BRTWOWAY into BR if one of the results is a fall-through.
Unless I'm missing something, there is no reason to duplicate this
in the target-specific code.
---
Diffs of the changes: (+7 -16)
PPC32ISelPattern.cpp | 23 +++++++----------------
1 files changed, 7 insertions(+), 16 deletions(-)
Index: llvm/lib/Target/PowerPC/PPC32ISelPattern.cpp
diff -u llvm/lib/Target/PowerPC/PPC32ISelPattern.cpp:1.157 llvm/lib/Target/PowerPC/PPC32ISelPattern.cpp:1.158
--- llvm/lib/Target/PowerPC/PPC32ISelPattern.cpp:1.157 Sun Aug 21 12:41:11 2005
+++ llvm/lib/Target/PowerPC/PPC32ISelPattern.cpp Sun Aug 21 14:03:28 2005
@@ -707,10 +707,6 @@
unsigned CCReg = SelectCC(N.getOperand(2), N.getOperand(3), CC);
unsigned Opc = getBCCForSetCC(CC);
- // Iterate to the next basic block
- ilist<MachineBasicBlock>::iterator It = BB;
- ++It;
-
// If this is a two way branch, then grab the fallthrough basic block argument
// and build a PowerPC branch pseudo-op, suitable for long branch conversion
// if necessary by the branch selection pass. Otherwise, emit a standard
@@ -718,19 +714,14 @@
if (N.getOpcode() == ISD::BRTWOWAY_CC) {
MachineBasicBlock *Fallthrough =
cast<BasicBlockSDNode>(N.getOperand(5))->getBasicBlock();
- if (Dest != It) {
- BuildMI(BB, PPC::COND_BRANCH, 4).addReg(CCReg).addImm(Opc)
- .addMBB(Dest).addMBB(Fallthrough);
- if (Fallthrough != It)
- BuildMI(BB, PPC::B, 1).addMBB(Fallthrough);
- } else {
- if (Fallthrough != It) {
- Opc = PPC32InstrInfo::invertPPCBranchOpcode(Opc);
- BuildMI(BB, PPC::COND_BRANCH, 4).addReg(CCReg).addImm(Opc)
- .addMBB(Fallthrough).addMBB(Dest);
- }
- }
+ BuildMI(BB, PPC::COND_BRANCH, 4).addReg(CCReg).addImm(Opc)
+ .addMBB(Dest).addMBB(Fallthrough);
+ BuildMI(BB, PPC::B, 1).addMBB(Fallthrough);
} else {
+ // Iterate to the next basic block
+ ilist<MachineBasicBlock>::iterator It = BB;
+ ++It;
+
// If the fallthrough path is off the end of the function, which would be
// undefined behavior, set it to be the same as the current block because
// we have nothing better to set it to, and leaving it alone will cause the
More information about the llvm-commits
mailing list