[llvm-commits] [llvm] r129602 - in /llvm/trunk: lib/Target/ARM/ARMBaseInstrInfo.cpp test/CodeGen/ARM/2011-04-15-AndVFlagPeepholeBug.ll
Cameron Zwarich
zwarich at apple.com
Fri Apr 15 13:45:00 PDT 2011
Author: zwarich
Date: Fri Apr 15 15:45:00 2011
New Revision: 129602
URL: http://llvm.org/viewvc/llvm-project?rev=129602&view=rev
Log:
The AND instruction leaves the V flag unmodified, so it falls victim to the same
problem as all of the other instructions we fold with CMPs.
Added:
llvm/trunk/test/CodeGen/ARM/2011-04-15-AndVFlagPeepholeBug.ll
Modified:
llvm/trunk/lib/Target/ARM/ARMBaseInstrInfo.cpp
Modified: llvm/trunk/lib/Target/ARM/ARMBaseInstrInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMBaseInstrInfo.cpp?rev=129602&r1=129601&r2=129602&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMBaseInstrInfo.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMBaseInstrInfo.cpp Fri Apr 15 15:45:00 2011
@@ -1638,7 +1638,11 @@
case ARM::t2SUBrr:
case ARM::t2SUBri:
case ARM::t2SBCrr:
- case ARM::t2SBCri: {
+ case ARM::t2SBCri:
+ case ARM::ANDrr:
+ case ARM::ANDri:
+ case ARM::t2ANDrr:
+ case ARM::t2ANDri: {
// Scan forward for the use of CPSR, if it's a conditional code requires
// checking of V bit, then this is not safe to do. If we can't find the
// CPSR use (i.e. used in another block), then it's not safe to perform
@@ -1677,18 +1681,13 @@
if (!isSafe)
return false;
- // fallthrough
- }
- case ARM::ANDrr:
- case ARM::ANDri:
- case ARM::t2ANDrr:
- case ARM::t2ANDri:
// Toggle the optional operand to CPSR.
MI->getOperand(5).setReg(ARM::CPSR);
MI->getOperand(5).setIsDef(true);
CmpInstr->eraseFromParent();
return true;
}
+ }
return false;
}
Added: llvm/trunk/test/CodeGen/ARM/2011-04-15-AndVFlagPeepholeBug.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/ARM/2011-04-15-AndVFlagPeepholeBug.ll?rev=129602&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/ARM/2011-04-15-AndVFlagPeepholeBug.ll (added)
+++ llvm/trunk/test/CodeGen/ARM/2011-04-15-AndVFlagPeepholeBug.ll Fri Apr 15 15:45:00 2011
@@ -0,0 +1,22 @@
+; RUN: llc < %s -mtriple=thumbv7-apple-darwin10 | FileCheck %s
+
+; CHECK: _f
+; CHECK-NOT: ands
+; CHECK: cmp
+; CHECK: blxle _g
+
+define i32 @f(i32 %a, i32 %b) nounwind ssp {
+entry:
+ %and = and i32 %b, %a
+ %cmp = icmp slt i32 %and, 1
+ br i1 %cmp, label %if.then, label %if.end
+
+if.then: ; preds = %entry
+ tail call void (...)* @g(i32 %a, i32 %b) nounwind
+ br label %if.end
+
+if.end: ; preds = %if.then, %entry
+ ret i32 %and
+}
+
+declare void @g(...)
More information about the llvm-commits
mailing list