diff options
| author | Venkatraman Govindaraju <venkatra@cs.wisc.edu> | 2011-12-03 21:24:48 +0000 |
|---|---|---|
| committer | Venkatraman Govindaraju <venkatra@cs.wisc.edu> | 2011-12-03 21:24:48 +0000 |
| commit | 80b1ae92922202c197078038c4229045cb1e295f (patch) | |
| tree | b557d016591441b756c955a08d233f47a77b77ec | |
| parent | a86bfc1071f7e05631e0af81e2c6249f552c9ec5 (diff) | |
Sparc CodeGen: Fix AnalyzeBranch for PR 10282. Removing addSuccessor() since
AnalyzeBranch doesn't change the successor, just the order.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@145779 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Target/Sparc/SparcInstrInfo.cpp | 2 | ||||
| -rw-r--r-- | test/CodeGen/SPARC/2011-12-03-TailDuplication.ll | 25 |
2 files changed, 26 insertions, 1 deletions
diff --git a/lib/Target/Sparc/SparcInstrInfo.cpp b/lib/Target/Sparc/SparcInstrInfo.cpp index 7a6bf50fa7..1c48c04739 100644 --- a/lib/Target/Sparc/SparcInstrInfo.cpp +++ b/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/test/CodeGen/SPARC/2011-12-03-TailDuplication.ll b/test/CodeGen/SPARC/2011-12-03-TailDuplication.ll new file mode 100644 index 0000000000..aa7de1618e --- /dev/null +++ b/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 +} |