forked from OSchip/llvm-project
[PPC64] Mark CR0 Live if PPCInstrInfo::optimizeCompareInstr Creates a Use of CR0
Resolve Bug 27046 (https://llvm.org/bugs/show_bug.cgi?id=27046). The PPCInstrInfo::optimizeCompareInstr function could create a new use of CR0, even if CR0 were previously dead. This patch marks CR0 live if a use of CR0 is created. Author: Tom Jablin (tjablin) Reviewers: hfinkel kbarton cycheng http://reviews.llvm.org/D18884 llvm-svn: 266040
This commit is contained in:
parent
c066d9cf55
commit
94f58e79ae
|
@ -1763,6 +1763,10 @@ bool PPCInstrInfo::optimizeCompareInstr(MachineInstr *CmpInstr,
|
|||
get(TargetOpcode::COPY), CRReg)
|
||||
.addReg(PPC::CR0, MIOpC != NewOpC ? RegState::Kill : 0);
|
||||
|
||||
// Even if CR0 register were dead before, it is alive now since the
|
||||
// instruction we just built uses it.
|
||||
MI->clearRegisterDeads(PPC::CR0);
|
||||
|
||||
if (MIOpC != NewOpC) {
|
||||
// We need to be careful here: we're replacing one instruction with
|
||||
// another, and we need to make sure that we get all of the right
|
||||
|
|
|
@ -0,0 +1,23 @@
|
|||
; RUN: llc -print-before=peephole-opts -print-after=peephole-opts -mtriple=powerpc64-unknown-linux-gnu -o /dev/null 2>&1 < %s | FileCheck %s
|
||||
|
||||
define signext i32 @fn1(i32 %baz) {
|
||||
%1 = mul nsw i32 %baz, 208
|
||||
%2 = zext i32 %1 to i64
|
||||
%3 = shl i64 %2, 48
|
||||
%4 = ashr exact i64 %3, 48
|
||||
; CHECK: ANDIo8 {{[^,]+}}, 65520, %CR0<imp-def,dead>;
|
||||
; CHECK: CMPLDI
|
||||
; CHECK: BCC
|
||||
|
||||
; CHECK: ANDIo8 {{[^,]+}}, 65520, %CR0<imp-def>;
|
||||
; CHECK: COPY %CR0
|
||||
; CHECK: BCC
|
||||
%5 = icmp eq i64 %4, 0
|
||||
br i1 %5, label %foo, label %bar
|
||||
|
||||
foo:
|
||||
ret i32 1
|
||||
|
||||
bar:
|
||||
ret i32 0
|
||||
}
|
Loading…
Reference in New Issue