summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVenkatraman Govindaraju <venkatra@cs.wisc.edu>2011-12-03 21:24:48 +0000
committerVenkatraman Govindaraju <venkatra@cs.wisc.edu>2011-12-03 21:24:48 +0000
commit6dae604f500124b91a3aad255b467a84c43ede6b (patch)
tree8bbcf7514da15eed86a4ae247f5db6203f966bc9
parent9b0cd149aa98763622abedda0a2573c772b2e9c0 (diff)
downloadbcm5719-llvm-6dae604f500124b91a3aad255b467a84c43ede6b.tar.gz
bcm5719-llvm-6dae604f500124b91a3aad255b467a84c43ede6b.zip
Sparc CodeGen: Fix AnalyzeBranch for PR 10282. Removing addSuccessor() since
AnalyzeBranch doesn't change the successor, just the order. llvm-svn: 145779
-rw-r--r--llvm/lib/Target/Sparc/SparcInstrInfo.cpp2
-rw-r--r--llvm/test/CodeGen/SPARC/2011-12-03-TailDuplication.ll25
2 files changed, 26 insertions, 1 deletions
diff --git a/llvm/lib/Target/Sparc/SparcInstrInfo.cpp b/llvm/lib/Target/Sparc/SparcInstrInfo.cpp
index 7a6bf50fa7d..1c48c047397 100644
--- a/llvm/lib/Target/Sparc/SparcInstrInfo.cpp
+++ b/llvm/lib/Target/Sparc/SparcInstrInfo.cpp
@@ -195,7 +195,7 @@ bool SparcInstrInfo::AnalyzeBranch(MachineBasicBlock &MBB,
.addMBB(UnCondBrIter->getOperand(0).getMBB()).addImm(BranchCode);
BuildMI(MBB, UnCondBrIter, MBB.findDebugLoc(I), get(SP::BA))
.addMBB(TargetBB);
- MBB.addSuccessor(TargetBB);
+
OldInst->eraseFromParent();
UnCondBrIter->eraseFromParent();
diff --git a/llvm/test/CodeGen/SPARC/2011-12-03-TailDuplication.ll b/llvm/test/CodeGen/SPARC/2011-12-03-TailDuplication.ll
new file mode 100644
index 00000000000..aa7de1618eb
--- /dev/null
+++ b/llvm/test/CodeGen/SPARC/2011-12-03-TailDuplication.ll
@@ -0,0 +1,25 @@
+; RUN: llc -march=sparc <%s
+
+define void @foo(i32 %a) nounwind {
+entry:
+ br i1 undef, label %return, label %else.0
+
+else.0:
+ br i1 undef, label %if.end.0, label %return
+
+if.end.0:
+ br i1 undef, label %if.then.1, label %else.1
+
+else.1:
+ %0 = bitcast i8* undef to i8**
+ br label %else.1.2
+
+if.then.1:
+ br i1 undef, label %return, label %return
+
+else.1.2:
+ br i1 undef, label %return, label %return
+
+return:
+ ret void
+}
OpenPOWER on IntegriCloud