1*9880d681SAndroid Build Coastguard Worker //=- AArch64ConditionOptimizer.cpp - Remove useless comparisons for AArch64 -=//
2*9880d681SAndroid Build Coastguard Worker //
3*9880d681SAndroid Build Coastguard Worker // The LLVM Compiler Infrastructure
4*9880d681SAndroid Build Coastguard Worker //
5*9880d681SAndroid Build Coastguard Worker // This file is distributed under the University of Illinois Open Source
6*9880d681SAndroid Build Coastguard Worker // License. See LICENSE.TXT for details.
7*9880d681SAndroid Build Coastguard Worker //
8*9880d681SAndroid Build Coastguard Worker //===----------------------------------------------------------------------===//
9*9880d681SAndroid Build Coastguard Worker //
10*9880d681SAndroid Build Coastguard Worker // This pass tries to make consecutive compares of values use same operands to
11*9880d681SAndroid Build Coastguard Worker // allow CSE pass to remove duplicated instructions. For this it analyzes
12*9880d681SAndroid Build Coastguard Worker // branches and adjusts comparisons with immediate values by converting:
13*9880d681SAndroid Build Coastguard Worker // * GE -> GT
14*9880d681SAndroid Build Coastguard Worker // * GT -> GE
15*9880d681SAndroid Build Coastguard Worker // * LT -> LE
16*9880d681SAndroid Build Coastguard Worker // * LE -> LT
17*9880d681SAndroid Build Coastguard Worker // and adjusting immediate values appropriately. It basically corrects two
18*9880d681SAndroid Build Coastguard Worker // immediate values towards each other to make them equal.
19*9880d681SAndroid Build Coastguard Worker //
20*9880d681SAndroid Build Coastguard Worker // Consider the following example in C:
21*9880d681SAndroid Build Coastguard Worker //
22*9880d681SAndroid Build Coastguard Worker // if ((a < 5 && ...) || (a > 5 && ...)) {
23*9880d681SAndroid Build Coastguard Worker // ~~~~~ ~~~~~
24*9880d681SAndroid Build Coastguard Worker // ^ ^
25*9880d681SAndroid Build Coastguard Worker // x y
26*9880d681SAndroid Build Coastguard Worker //
27*9880d681SAndroid Build Coastguard Worker // Here both "x" and "y" expressions compare "a" with "5". When "x" evaluates
28*9880d681SAndroid Build Coastguard Worker // to "false", "y" can just check flags set by the first comparison. As a
29*9880d681SAndroid Build Coastguard Worker // result of the canonicalization employed by
30*9880d681SAndroid Build Coastguard Worker // SelectionDAGBuilder::visitSwitchCase, DAGCombine, and other target-specific
31*9880d681SAndroid Build Coastguard Worker // code, assembly ends up in the form that is not CSE friendly:
32*9880d681SAndroid Build Coastguard Worker //
33*9880d681SAndroid Build Coastguard Worker // ...
34*9880d681SAndroid Build Coastguard Worker // cmp w8, #4
35*9880d681SAndroid Build Coastguard Worker // b.gt .LBB0_3
36*9880d681SAndroid Build Coastguard Worker // ...
37*9880d681SAndroid Build Coastguard Worker // .LBB0_3:
38*9880d681SAndroid Build Coastguard Worker // cmp w8, #6
39*9880d681SAndroid Build Coastguard Worker // b.lt .LBB0_6
40*9880d681SAndroid Build Coastguard Worker // ...
41*9880d681SAndroid Build Coastguard Worker //
42*9880d681SAndroid Build Coastguard Worker // Same assembly after the pass:
43*9880d681SAndroid Build Coastguard Worker //
44*9880d681SAndroid Build Coastguard Worker // ...
45*9880d681SAndroid Build Coastguard Worker // cmp w8, #5
46*9880d681SAndroid Build Coastguard Worker // b.ge .LBB0_3
47*9880d681SAndroid Build Coastguard Worker // ...
48*9880d681SAndroid Build Coastguard Worker // .LBB0_3:
49*9880d681SAndroid Build Coastguard Worker // cmp w8, #5 // <-- CSE pass removes this instruction
50*9880d681SAndroid Build Coastguard Worker // b.le .LBB0_6
51*9880d681SAndroid Build Coastguard Worker // ...
52*9880d681SAndroid Build Coastguard Worker //
53*9880d681SAndroid Build Coastguard Worker // Currently only SUBS and ADDS followed by b.?? are supported.
54*9880d681SAndroid Build Coastguard Worker //
55*9880d681SAndroid Build Coastguard Worker // TODO: maybe handle TBNZ/TBZ the same way as CMP when used instead for "a < 0"
56*9880d681SAndroid Build Coastguard Worker // TODO: handle other conditional instructions (e.g. CSET)
57*9880d681SAndroid Build Coastguard Worker // TODO: allow second branching to be anything if it doesn't require adjusting
58*9880d681SAndroid Build Coastguard Worker //
59*9880d681SAndroid Build Coastguard Worker //===----------------------------------------------------------------------===//
60*9880d681SAndroid Build Coastguard Worker
61*9880d681SAndroid Build Coastguard Worker #include "AArch64.h"
62*9880d681SAndroid Build Coastguard Worker #include "MCTargetDesc/AArch64AddressingModes.h"
63*9880d681SAndroid Build Coastguard Worker #include "llvm/ADT/DepthFirstIterator.h"
64*9880d681SAndroid Build Coastguard Worker #include "llvm/ADT/SmallVector.h"
65*9880d681SAndroid Build Coastguard Worker #include "llvm/ADT/Statistic.h"
66*9880d681SAndroid Build Coastguard Worker #include "llvm/CodeGen/LiveIntervalAnalysis.h"
67*9880d681SAndroid Build Coastguard Worker #include "llvm/CodeGen/MachineDominators.h"
68*9880d681SAndroid Build Coastguard Worker #include "llvm/CodeGen/MachineFunction.h"
69*9880d681SAndroid Build Coastguard Worker #include "llvm/CodeGen/MachineFunctionPass.h"
70*9880d681SAndroid Build Coastguard Worker #include "llvm/CodeGen/MachineInstrBuilder.h"
71*9880d681SAndroid Build Coastguard Worker #include "llvm/CodeGen/MachineRegisterInfo.h"
72*9880d681SAndroid Build Coastguard Worker #include "llvm/CodeGen/Passes.h"
73*9880d681SAndroid Build Coastguard Worker #include "llvm/Support/Debug.h"
74*9880d681SAndroid Build Coastguard Worker #include "llvm/Support/raw_ostream.h"
75*9880d681SAndroid Build Coastguard Worker #include "llvm/Target/TargetInstrInfo.h"
76*9880d681SAndroid Build Coastguard Worker #include "llvm/Target/TargetSubtargetInfo.h"
77*9880d681SAndroid Build Coastguard Worker #include <cstdlib>
78*9880d681SAndroid Build Coastguard Worker #include <tuple>
79*9880d681SAndroid Build Coastguard Worker
80*9880d681SAndroid Build Coastguard Worker using namespace llvm;
81*9880d681SAndroid Build Coastguard Worker
82*9880d681SAndroid Build Coastguard Worker #define DEBUG_TYPE "aarch64-condopt"
83*9880d681SAndroid Build Coastguard Worker
84*9880d681SAndroid Build Coastguard Worker STATISTIC(NumConditionsAdjusted, "Number of conditions adjusted");
85*9880d681SAndroid Build Coastguard Worker
86*9880d681SAndroid Build Coastguard Worker namespace {
87*9880d681SAndroid Build Coastguard Worker class AArch64ConditionOptimizer : public MachineFunctionPass {
88*9880d681SAndroid Build Coastguard Worker const TargetInstrInfo *TII;
89*9880d681SAndroid Build Coastguard Worker MachineDominatorTree *DomTree;
90*9880d681SAndroid Build Coastguard Worker const MachineRegisterInfo *MRI;
91*9880d681SAndroid Build Coastguard Worker
92*9880d681SAndroid Build Coastguard Worker public:
93*9880d681SAndroid Build Coastguard Worker // Stores immediate, compare instruction opcode and branch condition (in this
94*9880d681SAndroid Build Coastguard Worker // order) of adjusted comparison.
95*9880d681SAndroid Build Coastguard Worker typedef std::tuple<int, unsigned, AArch64CC::CondCode> CmpInfo;
96*9880d681SAndroid Build Coastguard Worker
97*9880d681SAndroid Build Coastguard Worker static char ID;
AArch64ConditionOptimizer()98*9880d681SAndroid Build Coastguard Worker AArch64ConditionOptimizer() : MachineFunctionPass(ID) {}
99*9880d681SAndroid Build Coastguard Worker void getAnalysisUsage(AnalysisUsage &AU) const override;
100*9880d681SAndroid Build Coastguard Worker MachineInstr *findSuitableCompare(MachineBasicBlock *MBB);
101*9880d681SAndroid Build Coastguard Worker CmpInfo adjustCmp(MachineInstr *CmpMI, AArch64CC::CondCode Cmp);
102*9880d681SAndroid Build Coastguard Worker void modifyCmp(MachineInstr *CmpMI, const CmpInfo &Info);
103*9880d681SAndroid Build Coastguard Worker bool adjustTo(MachineInstr *CmpMI, AArch64CC::CondCode Cmp, MachineInstr *To,
104*9880d681SAndroid Build Coastguard Worker int ToImm);
105*9880d681SAndroid Build Coastguard Worker bool runOnMachineFunction(MachineFunction &MF) override;
getPassName() const106*9880d681SAndroid Build Coastguard Worker const char *getPassName() const override {
107*9880d681SAndroid Build Coastguard Worker return "AArch64 Condition Optimizer";
108*9880d681SAndroid Build Coastguard Worker }
109*9880d681SAndroid Build Coastguard Worker };
110*9880d681SAndroid Build Coastguard Worker } // end anonymous namespace
111*9880d681SAndroid Build Coastguard Worker
112*9880d681SAndroid Build Coastguard Worker char AArch64ConditionOptimizer::ID = 0;
113*9880d681SAndroid Build Coastguard Worker
114*9880d681SAndroid Build Coastguard Worker namespace llvm {
115*9880d681SAndroid Build Coastguard Worker void initializeAArch64ConditionOptimizerPass(PassRegistry &);
116*9880d681SAndroid Build Coastguard Worker }
117*9880d681SAndroid Build Coastguard Worker
118*9880d681SAndroid Build Coastguard Worker INITIALIZE_PASS_BEGIN(AArch64ConditionOptimizer, "aarch64-condopt",
119*9880d681SAndroid Build Coastguard Worker "AArch64 CondOpt Pass", false, false)
INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree)120*9880d681SAndroid Build Coastguard Worker INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree)
121*9880d681SAndroid Build Coastguard Worker INITIALIZE_PASS_END(AArch64ConditionOptimizer, "aarch64-condopt",
122*9880d681SAndroid Build Coastguard Worker "AArch64 CondOpt Pass", false, false)
123*9880d681SAndroid Build Coastguard Worker
124*9880d681SAndroid Build Coastguard Worker FunctionPass *llvm::createAArch64ConditionOptimizerPass() {
125*9880d681SAndroid Build Coastguard Worker return new AArch64ConditionOptimizer();
126*9880d681SAndroid Build Coastguard Worker }
127*9880d681SAndroid Build Coastguard Worker
getAnalysisUsage(AnalysisUsage & AU) const128*9880d681SAndroid Build Coastguard Worker void AArch64ConditionOptimizer::getAnalysisUsage(AnalysisUsage &AU) const {
129*9880d681SAndroid Build Coastguard Worker AU.addRequired<MachineDominatorTree>();
130*9880d681SAndroid Build Coastguard Worker AU.addPreserved<MachineDominatorTree>();
131*9880d681SAndroid Build Coastguard Worker MachineFunctionPass::getAnalysisUsage(AU);
132*9880d681SAndroid Build Coastguard Worker }
133*9880d681SAndroid Build Coastguard Worker
134*9880d681SAndroid Build Coastguard Worker // Finds compare instruction that corresponds to supported types of branching.
135*9880d681SAndroid Build Coastguard Worker // Returns the instruction or nullptr on failures or detecting unsupported
136*9880d681SAndroid Build Coastguard Worker // instructions.
findSuitableCompare(MachineBasicBlock * MBB)137*9880d681SAndroid Build Coastguard Worker MachineInstr *AArch64ConditionOptimizer::findSuitableCompare(
138*9880d681SAndroid Build Coastguard Worker MachineBasicBlock *MBB) {
139*9880d681SAndroid Build Coastguard Worker MachineBasicBlock::iterator I = MBB->getFirstTerminator();
140*9880d681SAndroid Build Coastguard Worker if (I == MBB->end())
141*9880d681SAndroid Build Coastguard Worker return nullptr;
142*9880d681SAndroid Build Coastguard Worker
143*9880d681SAndroid Build Coastguard Worker if (I->getOpcode() != AArch64::Bcc)
144*9880d681SAndroid Build Coastguard Worker return nullptr;
145*9880d681SAndroid Build Coastguard Worker
146*9880d681SAndroid Build Coastguard Worker // Since we may modify cmp of this MBB, make sure NZCV does not live out.
147*9880d681SAndroid Build Coastguard Worker for (auto SuccBB : MBB->successors())
148*9880d681SAndroid Build Coastguard Worker if (SuccBB->isLiveIn(AArch64::NZCV))
149*9880d681SAndroid Build Coastguard Worker return nullptr;
150*9880d681SAndroid Build Coastguard Worker
151*9880d681SAndroid Build Coastguard Worker // Now find the instruction controlling the terminator.
152*9880d681SAndroid Build Coastguard Worker for (MachineBasicBlock::iterator B = MBB->begin(); I != B;) {
153*9880d681SAndroid Build Coastguard Worker --I;
154*9880d681SAndroid Build Coastguard Worker assert(!I->isTerminator() && "Spurious terminator");
155*9880d681SAndroid Build Coastguard Worker // Check if there is any use of NZCV between CMP and Bcc.
156*9880d681SAndroid Build Coastguard Worker if (I->readsRegister(AArch64::NZCV))
157*9880d681SAndroid Build Coastguard Worker return nullptr;
158*9880d681SAndroid Build Coastguard Worker switch (I->getOpcode()) {
159*9880d681SAndroid Build Coastguard Worker // cmp is an alias for subs with a dead destination register.
160*9880d681SAndroid Build Coastguard Worker case AArch64::SUBSWri:
161*9880d681SAndroid Build Coastguard Worker case AArch64::SUBSXri:
162*9880d681SAndroid Build Coastguard Worker // cmn is an alias for adds with a dead destination register.
163*9880d681SAndroid Build Coastguard Worker case AArch64::ADDSWri:
164*9880d681SAndroid Build Coastguard Worker case AArch64::ADDSXri: {
165*9880d681SAndroid Build Coastguard Worker unsigned ShiftAmt = AArch64_AM::getShiftValue(I->getOperand(3).getImm());
166*9880d681SAndroid Build Coastguard Worker if (!I->getOperand(2).isImm()) {
167*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "Immediate of cmp is symbolic, " << *I << '\n');
168*9880d681SAndroid Build Coastguard Worker return nullptr;
169*9880d681SAndroid Build Coastguard Worker } else if (I->getOperand(2).getImm() << ShiftAmt >= 0xfff) {
170*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "Immediate of cmp may be out of range, " << *I << '\n');
171*9880d681SAndroid Build Coastguard Worker return nullptr;
172*9880d681SAndroid Build Coastguard Worker } else if (!MRI->use_empty(I->getOperand(0).getReg())) {
173*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "Destination of cmp is not dead, " << *I << '\n');
174*9880d681SAndroid Build Coastguard Worker return nullptr;
175*9880d681SAndroid Build Coastguard Worker }
176*9880d681SAndroid Build Coastguard Worker return &*I;
177*9880d681SAndroid Build Coastguard Worker }
178*9880d681SAndroid Build Coastguard Worker // Prevent false positive case like:
179*9880d681SAndroid Build Coastguard Worker // cmp w19, #0
180*9880d681SAndroid Build Coastguard Worker // cinc w0, w19, gt
181*9880d681SAndroid Build Coastguard Worker // ...
182*9880d681SAndroid Build Coastguard Worker // fcmp d8, #0.0
183*9880d681SAndroid Build Coastguard Worker // b.gt .LBB0_5
184*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPDri:
185*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPSri:
186*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPESri:
187*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPEDri:
188*9880d681SAndroid Build Coastguard Worker
189*9880d681SAndroid Build Coastguard Worker case AArch64::SUBSWrr:
190*9880d681SAndroid Build Coastguard Worker case AArch64::SUBSXrr:
191*9880d681SAndroid Build Coastguard Worker case AArch64::ADDSWrr:
192*9880d681SAndroid Build Coastguard Worker case AArch64::ADDSXrr:
193*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPSrr:
194*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPDrr:
195*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPESrr:
196*9880d681SAndroid Build Coastguard Worker case AArch64::FCMPEDrr:
197*9880d681SAndroid Build Coastguard Worker // Skip comparison instructions without immediate operands.
198*9880d681SAndroid Build Coastguard Worker return nullptr;
199*9880d681SAndroid Build Coastguard Worker }
200*9880d681SAndroid Build Coastguard Worker }
201*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "Flags not defined in BB#" << MBB->getNumber() << '\n');
202*9880d681SAndroid Build Coastguard Worker return nullptr;
203*9880d681SAndroid Build Coastguard Worker }
204*9880d681SAndroid Build Coastguard Worker
205*9880d681SAndroid Build Coastguard Worker // Changes opcode adds <-> subs considering register operand width.
getComplementOpc(int Opc)206*9880d681SAndroid Build Coastguard Worker static int getComplementOpc(int Opc) {
207*9880d681SAndroid Build Coastguard Worker switch (Opc) {
208*9880d681SAndroid Build Coastguard Worker case AArch64::ADDSWri: return AArch64::SUBSWri;
209*9880d681SAndroid Build Coastguard Worker case AArch64::ADDSXri: return AArch64::SUBSXri;
210*9880d681SAndroid Build Coastguard Worker case AArch64::SUBSWri: return AArch64::ADDSWri;
211*9880d681SAndroid Build Coastguard Worker case AArch64::SUBSXri: return AArch64::ADDSXri;
212*9880d681SAndroid Build Coastguard Worker default:
213*9880d681SAndroid Build Coastguard Worker llvm_unreachable("Unexpected opcode");
214*9880d681SAndroid Build Coastguard Worker }
215*9880d681SAndroid Build Coastguard Worker }
216*9880d681SAndroid Build Coastguard Worker
217*9880d681SAndroid Build Coastguard Worker // Changes form of comparison inclusive <-> exclusive.
getAdjustedCmp(AArch64CC::CondCode Cmp)218*9880d681SAndroid Build Coastguard Worker static AArch64CC::CondCode getAdjustedCmp(AArch64CC::CondCode Cmp) {
219*9880d681SAndroid Build Coastguard Worker switch (Cmp) {
220*9880d681SAndroid Build Coastguard Worker case AArch64CC::GT: return AArch64CC::GE;
221*9880d681SAndroid Build Coastguard Worker case AArch64CC::GE: return AArch64CC::GT;
222*9880d681SAndroid Build Coastguard Worker case AArch64CC::LT: return AArch64CC::LE;
223*9880d681SAndroid Build Coastguard Worker case AArch64CC::LE: return AArch64CC::LT;
224*9880d681SAndroid Build Coastguard Worker default:
225*9880d681SAndroid Build Coastguard Worker llvm_unreachable("Unexpected condition code");
226*9880d681SAndroid Build Coastguard Worker }
227*9880d681SAndroid Build Coastguard Worker }
228*9880d681SAndroid Build Coastguard Worker
229*9880d681SAndroid Build Coastguard Worker // Transforms GT -> GE, GE -> GT, LT -> LE, LE -> LT by updating comparison
230*9880d681SAndroid Build Coastguard Worker // operator and condition code.
adjustCmp(MachineInstr * CmpMI,AArch64CC::CondCode Cmp)231*9880d681SAndroid Build Coastguard Worker AArch64ConditionOptimizer::CmpInfo AArch64ConditionOptimizer::adjustCmp(
232*9880d681SAndroid Build Coastguard Worker MachineInstr *CmpMI, AArch64CC::CondCode Cmp) {
233*9880d681SAndroid Build Coastguard Worker unsigned Opc = CmpMI->getOpcode();
234*9880d681SAndroid Build Coastguard Worker
235*9880d681SAndroid Build Coastguard Worker // CMN (compare with negative immediate) is an alias to ADDS (as
236*9880d681SAndroid Build Coastguard Worker // "operand - negative" == "operand + positive")
237*9880d681SAndroid Build Coastguard Worker bool Negative = (Opc == AArch64::ADDSWri || Opc == AArch64::ADDSXri);
238*9880d681SAndroid Build Coastguard Worker
239*9880d681SAndroid Build Coastguard Worker int Correction = (Cmp == AArch64CC::GT) ? 1 : -1;
240*9880d681SAndroid Build Coastguard Worker // Negate Correction value for comparison with negative immediate (CMN).
241*9880d681SAndroid Build Coastguard Worker if (Negative) {
242*9880d681SAndroid Build Coastguard Worker Correction = -Correction;
243*9880d681SAndroid Build Coastguard Worker }
244*9880d681SAndroid Build Coastguard Worker
245*9880d681SAndroid Build Coastguard Worker const int OldImm = (int)CmpMI->getOperand(2).getImm();
246*9880d681SAndroid Build Coastguard Worker const int NewImm = std::abs(OldImm + Correction);
247*9880d681SAndroid Build Coastguard Worker
248*9880d681SAndroid Build Coastguard Worker // Handle +0 -> -1 and -0 -> +1 (CMN with 0 immediate) transitions by
249*9880d681SAndroid Build Coastguard Worker // adjusting compare instruction opcode.
250*9880d681SAndroid Build Coastguard Worker if (OldImm == 0 && ((Negative && Correction == 1) ||
251*9880d681SAndroid Build Coastguard Worker (!Negative && Correction == -1))) {
252*9880d681SAndroid Build Coastguard Worker Opc = getComplementOpc(Opc);
253*9880d681SAndroid Build Coastguard Worker }
254*9880d681SAndroid Build Coastguard Worker
255*9880d681SAndroid Build Coastguard Worker return CmpInfo(NewImm, Opc, getAdjustedCmp(Cmp));
256*9880d681SAndroid Build Coastguard Worker }
257*9880d681SAndroid Build Coastguard Worker
258*9880d681SAndroid Build Coastguard Worker // Applies changes to comparison instruction suggested by adjustCmp().
modifyCmp(MachineInstr * CmpMI,const CmpInfo & Info)259*9880d681SAndroid Build Coastguard Worker void AArch64ConditionOptimizer::modifyCmp(MachineInstr *CmpMI,
260*9880d681SAndroid Build Coastguard Worker const CmpInfo &Info) {
261*9880d681SAndroid Build Coastguard Worker int Imm;
262*9880d681SAndroid Build Coastguard Worker unsigned Opc;
263*9880d681SAndroid Build Coastguard Worker AArch64CC::CondCode Cmp;
264*9880d681SAndroid Build Coastguard Worker std::tie(Imm, Opc, Cmp) = Info;
265*9880d681SAndroid Build Coastguard Worker
266*9880d681SAndroid Build Coastguard Worker MachineBasicBlock *const MBB = CmpMI->getParent();
267*9880d681SAndroid Build Coastguard Worker
268*9880d681SAndroid Build Coastguard Worker // Change immediate in comparison instruction (ADDS or SUBS).
269*9880d681SAndroid Build Coastguard Worker BuildMI(*MBB, CmpMI, CmpMI->getDebugLoc(), TII->get(Opc))
270*9880d681SAndroid Build Coastguard Worker .addOperand(CmpMI->getOperand(0))
271*9880d681SAndroid Build Coastguard Worker .addOperand(CmpMI->getOperand(1))
272*9880d681SAndroid Build Coastguard Worker .addImm(Imm)
273*9880d681SAndroid Build Coastguard Worker .addOperand(CmpMI->getOperand(3));
274*9880d681SAndroid Build Coastguard Worker CmpMI->eraseFromParent();
275*9880d681SAndroid Build Coastguard Worker
276*9880d681SAndroid Build Coastguard Worker // The fact that this comparison was picked ensures that it's related to the
277*9880d681SAndroid Build Coastguard Worker // first terminator instruction.
278*9880d681SAndroid Build Coastguard Worker MachineInstr &BrMI = *MBB->getFirstTerminator();
279*9880d681SAndroid Build Coastguard Worker
280*9880d681SAndroid Build Coastguard Worker // Change condition in branch instruction.
281*9880d681SAndroid Build Coastguard Worker BuildMI(*MBB, BrMI, BrMI.getDebugLoc(), TII->get(AArch64::Bcc))
282*9880d681SAndroid Build Coastguard Worker .addImm(Cmp)
283*9880d681SAndroid Build Coastguard Worker .addOperand(BrMI.getOperand(1));
284*9880d681SAndroid Build Coastguard Worker BrMI.eraseFromParent();
285*9880d681SAndroid Build Coastguard Worker
286*9880d681SAndroid Build Coastguard Worker MBB->updateTerminator();
287*9880d681SAndroid Build Coastguard Worker
288*9880d681SAndroid Build Coastguard Worker ++NumConditionsAdjusted;
289*9880d681SAndroid Build Coastguard Worker }
290*9880d681SAndroid Build Coastguard Worker
291*9880d681SAndroid Build Coastguard Worker // Parse a condition code returned by AnalyzeBranch, and compute the CondCode
292*9880d681SAndroid Build Coastguard Worker // corresponding to TBB.
293*9880d681SAndroid Build Coastguard Worker // Returns true if parsing was successful, otherwise false is returned.
parseCond(ArrayRef<MachineOperand> Cond,AArch64CC::CondCode & CC)294*9880d681SAndroid Build Coastguard Worker static bool parseCond(ArrayRef<MachineOperand> Cond, AArch64CC::CondCode &CC) {
295*9880d681SAndroid Build Coastguard Worker // A normal br.cond simply has the condition code.
296*9880d681SAndroid Build Coastguard Worker if (Cond[0].getImm() != -1) {
297*9880d681SAndroid Build Coastguard Worker assert(Cond.size() == 1 && "Unknown Cond array format");
298*9880d681SAndroid Build Coastguard Worker CC = (AArch64CC::CondCode)(int)Cond[0].getImm();
299*9880d681SAndroid Build Coastguard Worker return true;
300*9880d681SAndroid Build Coastguard Worker }
301*9880d681SAndroid Build Coastguard Worker return false;
302*9880d681SAndroid Build Coastguard Worker }
303*9880d681SAndroid Build Coastguard Worker
304*9880d681SAndroid Build Coastguard Worker // Adjusts one cmp instruction to another one if result of adjustment will allow
305*9880d681SAndroid Build Coastguard Worker // CSE. Returns true if compare instruction was changed, otherwise false is
306*9880d681SAndroid Build Coastguard Worker // returned.
adjustTo(MachineInstr * CmpMI,AArch64CC::CondCode Cmp,MachineInstr * To,int ToImm)307*9880d681SAndroid Build Coastguard Worker bool AArch64ConditionOptimizer::adjustTo(MachineInstr *CmpMI,
308*9880d681SAndroid Build Coastguard Worker AArch64CC::CondCode Cmp, MachineInstr *To, int ToImm)
309*9880d681SAndroid Build Coastguard Worker {
310*9880d681SAndroid Build Coastguard Worker CmpInfo Info = adjustCmp(CmpMI, Cmp);
311*9880d681SAndroid Build Coastguard Worker if (std::get<0>(Info) == ToImm && std::get<1>(Info) == To->getOpcode()) {
312*9880d681SAndroid Build Coastguard Worker modifyCmp(CmpMI, Info);
313*9880d681SAndroid Build Coastguard Worker return true;
314*9880d681SAndroid Build Coastguard Worker }
315*9880d681SAndroid Build Coastguard Worker return false;
316*9880d681SAndroid Build Coastguard Worker }
317*9880d681SAndroid Build Coastguard Worker
runOnMachineFunction(MachineFunction & MF)318*9880d681SAndroid Build Coastguard Worker bool AArch64ConditionOptimizer::runOnMachineFunction(MachineFunction &MF) {
319*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "********** AArch64 Conditional Compares **********\n"
320*9880d681SAndroid Build Coastguard Worker << "********** Function: " << MF.getName() << '\n');
321*9880d681SAndroid Build Coastguard Worker if (skipFunction(*MF.getFunction()))
322*9880d681SAndroid Build Coastguard Worker return false;
323*9880d681SAndroid Build Coastguard Worker
324*9880d681SAndroid Build Coastguard Worker TII = MF.getSubtarget().getInstrInfo();
325*9880d681SAndroid Build Coastguard Worker DomTree = &getAnalysis<MachineDominatorTree>();
326*9880d681SAndroid Build Coastguard Worker MRI = &MF.getRegInfo();
327*9880d681SAndroid Build Coastguard Worker
328*9880d681SAndroid Build Coastguard Worker bool Changed = false;
329*9880d681SAndroid Build Coastguard Worker
330*9880d681SAndroid Build Coastguard Worker // Visit blocks in dominator tree pre-order. The pre-order enables multiple
331*9880d681SAndroid Build Coastguard Worker // cmp-conversions from the same head block.
332*9880d681SAndroid Build Coastguard Worker // Note that updateDomTree() modifies the children of the DomTree node
333*9880d681SAndroid Build Coastguard Worker // currently being visited. The df_iterator supports that; it doesn't look at
334*9880d681SAndroid Build Coastguard Worker // child_begin() / child_end() until after a node has been visited.
335*9880d681SAndroid Build Coastguard Worker for (MachineDomTreeNode *I : depth_first(DomTree)) {
336*9880d681SAndroid Build Coastguard Worker MachineBasicBlock *HBB = I->getBlock();
337*9880d681SAndroid Build Coastguard Worker
338*9880d681SAndroid Build Coastguard Worker SmallVector<MachineOperand, 4> HeadCond;
339*9880d681SAndroid Build Coastguard Worker MachineBasicBlock *TBB = nullptr, *FBB = nullptr;
340*9880d681SAndroid Build Coastguard Worker if (TII->analyzeBranch(*HBB, TBB, FBB, HeadCond)) {
341*9880d681SAndroid Build Coastguard Worker continue;
342*9880d681SAndroid Build Coastguard Worker }
343*9880d681SAndroid Build Coastguard Worker
344*9880d681SAndroid Build Coastguard Worker // Equivalence check is to skip loops.
345*9880d681SAndroid Build Coastguard Worker if (!TBB || TBB == HBB) {
346*9880d681SAndroid Build Coastguard Worker continue;
347*9880d681SAndroid Build Coastguard Worker }
348*9880d681SAndroid Build Coastguard Worker
349*9880d681SAndroid Build Coastguard Worker SmallVector<MachineOperand, 4> TrueCond;
350*9880d681SAndroid Build Coastguard Worker MachineBasicBlock *TBB_TBB = nullptr, *TBB_FBB = nullptr;
351*9880d681SAndroid Build Coastguard Worker if (TII->analyzeBranch(*TBB, TBB_TBB, TBB_FBB, TrueCond)) {
352*9880d681SAndroid Build Coastguard Worker continue;
353*9880d681SAndroid Build Coastguard Worker }
354*9880d681SAndroid Build Coastguard Worker
355*9880d681SAndroid Build Coastguard Worker MachineInstr *HeadCmpMI = findSuitableCompare(HBB);
356*9880d681SAndroid Build Coastguard Worker if (!HeadCmpMI) {
357*9880d681SAndroid Build Coastguard Worker continue;
358*9880d681SAndroid Build Coastguard Worker }
359*9880d681SAndroid Build Coastguard Worker
360*9880d681SAndroid Build Coastguard Worker MachineInstr *TrueCmpMI = findSuitableCompare(TBB);
361*9880d681SAndroid Build Coastguard Worker if (!TrueCmpMI) {
362*9880d681SAndroid Build Coastguard Worker continue;
363*9880d681SAndroid Build Coastguard Worker }
364*9880d681SAndroid Build Coastguard Worker
365*9880d681SAndroid Build Coastguard Worker AArch64CC::CondCode HeadCmp;
366*9880d681SAndroid Build Coastguard Worker if (HeadCond.empty() || !parseCond(HeadCond, HeadCmp)) {
367*9880d681SAndroid Build Coastguard Worker continue;
368*9880d681SAndroid Build Coastguard Worker }
369*9880d681SAndroid Build Coastguard Worker
370*9880d681SAndroid Build Coastguard Worker AArch64CC::CondCode TrueCmp;
371*9880d681SAndroid Build Coastguard Worker if (TrueCond.empty() || !parseCond(TrueCond, TrueCmp)) {
372*9880d681SAndroid Build Coastguard Worker continue;
373*9880d681SAndroid Build Coastguard Worker }
374*9880d681SAndroid Build Coastguard Worker
375*9880d681SAndroid Build Coastguard Worker const int HeadImm = (int)HeadCmpMI->getOperand(2).getImm();
376*9880d681SAndroid Build Coastguard Worker const int TrueImm = (int)TrueCmpMI->getOperand(2).getImm();
377*9880d681SAndroid Build Coastguard Worker
378*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "Head branch:\n");
379*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "\tcondition: "
380*9880d681SAndroid Build Coastguard Worker << AArch64CC::getCondCodeName(HeadCmp) << '\n');
381*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "\timmediate: " << HeadImm << '\n');
382*9880d681SAndroid Build Coastguard Worker
383*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "True branch:\n");
384*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "\tcondition: "
385*9880d681SAndroid Build Coastguard Worker << AArch64CC::getCondCodeName(TrueCmp) << '\n');
386*9880d681SAndroid Build Coastguard Worker DEBUG(dbgs() << "\timmediate: " << TrueImm << '\n');
387*9880d681SAndroid Build Coastguard Worker
388*9880d681SAndroid Build Coastguard Worker if (((HeadCmp == AArch64CC::GT && TrueCmp == AArch64CC::LT) ||
389*9880d681SAndroid Build Coastguard Worker (HeadCmp == AArch64CC::LT && TrueCmp == AArch64CC::GT)) &&
390*9880d681SAndroid Build Coastguard Worker std::abs(TrueImm - HeadImm) == 2) {
391*9880d681SAndroid Build Coastguard Worker // This branch transforms machine instructions that correspond to
392*9880d681SAndroid Build Coastguard Worker //
393*9880d681SAndroid Build Coastguard Worker // 1) (a > {TrueImm} && ...) || (a < {HeadImm} && ...)
394*9880d681SAndroid Build Coastguard Worker // 2) (a < {TrueImm} && ...) || (a > {HeadImm} && ...)
395*9880d681SAndroid Build Coastguard Worker //
396*9880d681SAndroid Build Coastguard Worker // into
397*9880d681SAndroid Build Coastguard Worker //
398*9880d681SAndroid Build Coastguard Worker // 1) (a >= {NewImm} && ...) || (a <= {NewImm} && ...)
399*9880d681SAndroid Build Coastguard Worker // 2) (a <= {NewImm} && ...) || (a >= {NewImm} && ...)
400*9880d681SAndroid Build Coastguard Worker
401*9880d681SAndroid Build Coastguard Worker CmpInfo HeadCmpInfo = adjustCmp(HeadCmpMI, HeadCmp);
402*9880d681SAndroid Build Coastguard Worker CmpInfo TrueCmpInfo = adjustCmp(TrueCmpMI, TrueCmp);
403*9880d681SAndroid Build Coastguard Worker if (std::get<0>(HeadCmpInfo) == std::get<0>(TrueCmpInfo) &&
404*9880d681SAndroid Build Coastguard Worker std::get<1>(HeadCmpInfo) == std::get<1>(TrueCmpInfo)) {
405*9880d681SAndroid Build Coastguard Worker modifyCmp(HeadCmpMI, HeadCmpInfo);
406*9880d681SAndroid Build Coastguard Worker modifyCmp(TrueCmpMI, TrueCmpInfo);
407*9880d681SAndroid Build Coastguard Worker Changed = true;
408*9880d681SAndroid Build Coastguard Worker }
409*9880d681SAndroid Build Coastguard Worker } else if (((HeadCmp == AArch64CC::GT && TrueCmp == AArch64CC::GT) ||
410*9880d681SAndroid Build Coastguard Worker (HeadCmp == AArch64CC::LT && TrueCmp == AArch64CC::LT)) &&
411*9880d681SAndroid Build Coastguard Worker std::abs(TrueImm - HeadImm) == 1) {
412*9880d681SAndroid Build Coastguard Worker // This branch transforms machine instructions that correspond to
413*9880d681SAndroid Build Coastguard Worker //
414*9880d681SAndroid Build Coastguard Worker // 1) (a > {TrueImm} && ...) || (a > {HeadImm} && ...)
415*9880d681SAndroid Build Coastguard Worker // 2) (a < {TrueImm} && ...) || (a < {HeadImm} && ...)
416*9880d681SAndroid Build Coastguard Worker //
417*9880d681SAndroid Build Coastguard Worker // into
418*9880d681SAndroid Build Coastguard Worker //
419*9880d681SAndroid Build Coastguard Worker // 1) (a <= {NewImm} && ...) || (a > {NewImm} && ...)
420*9880d681SAndroid Build Coastguard Worker // 2) (a < {NewImm} && ...) || (a >= {NewImm} && ...)
421*9880d681SAndroid Build Coastguard Worker
422*9880d681SAndroid Build Coastguard Worker // GT -> GE transformation increases immediate value, so picking the
423*9880d681SAndroid Build Coastguard Worker // smaller one; LT -> LE decreases immediate value so invert the choice.
424*9880d681SAndroid Build Coastguard Worker bool adjustHeadCond = (HeadImm < TrueImm);
425*9880d681SAndroid Build Coastguard Worker if (HeadCmp == AArch64CC::LT) {
426*9880d681SAndroid Build Coastguard Worker adjustHeadCond = !adjustHeadCond;
427*9880d681SAndroid Build Coastguard Worker }
428*9880d681SAndroid Build Coastguard Worker
429*9880d681SAndroid Build Coastguard Worker if (adjustHeadCond) {
430*9880d681SAndroid Build Coastguard Worker Changed |= adjustTo(HeadCmpMI, HeadCmp, TrueCmpMI, TrueImm);
431*9880d681SAndroid Build Coastguard Worker } else {
432*9880d681SAndroid Build Coastguard Worker Changed |= adjustTo(TrueCmpMI, TrueCmp, HeadCmpMI, HeadImm);
433*9880d681SAndroid Build Coastguard Worker }
434*9880d681SAndroid Build Coastguard Worker }
435*9880d681SAndroid Build Coastguard Worker // Other transformation cases almost never occur due to generation of < or >
436*9880d681SAndroid Build Coastguard Worker // comparisons instead of <= and >=.
437*9880d681SAndroid Build Coastguard Worker }
438*9880d681SAndroid Build Coastguard Worker
439*9880d681SAndroid Build Coastguard Worker return Changed;
440*9880d681SAndroid Build Coastguard Worker }
441