Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 1 | //===-- IfConversion.cpp - Machine code if conversion pass. ---------------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file was developed by the Evan Cheng and is distributed under |
| 6 | // the University of Illinois Open Source License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This file implements the machine instruction level if-conversion pass. |
| 11 | // |
| 12 | //===----------------------------------------------------------------------===// |
| 13 | |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 14 | #define DEBUG_TYPE "ifcvt" |
| 15 | #include "llvm/Function.h" |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 16 | #include "llvm/CodeGen/Passes.h" |
| 17 | #include "llvm/CodeGen/MachineModuleInfo.h" |
| 18 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 19 | #include "llvm/Target/TargetInstrInfo.h" |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 20 | #include "llvm/Target/TargetLowering.h" |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 21 | #include "llvm/Target/TargetMachine.h" |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 22 | #include "llvm/Support/CommandLine.h" |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 23 | #include "llvm/Support/Debug.h" |
Evan Cheng | 36489bb | 2007-05-18 19:26:33 +0000 | [diff] [blame] | 24 | #include "llvm/ADT/DepthFirstIterator.h" |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 25 | #include "llvm/ADT/Statistic.h" |
| 26 | using namespace llvm; |
| 27 | |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 28 | namespace { |
| 29 | // Hidden options for help debugging. |
| 30 | cl::opt<int> IfCvtFnStart("ifcvt-fn-start", cl::init(-1), cl::Hidden); |
| 31 | cl::opt<int> IfCvtFnStop("ifcvt-fn-stop", cl::init(-1), cl::Hidden); |
| 32 | cl::opt<int> IfCvtLimit("ifcvt-limit", cl::init(-1), cl::Hidden); |
| 33 | cl::opt<bool> DisableSimple("disable-ifcvt-simple", |
| 34 | cl::init(false), cl::Hidden); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 35 | cl::opt<bool> DisableSimpleF("disable-ifcvt-simple-false", |
| 36 | cl::init(false), cl::Hidden); |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 37 | cl::opt<bool> DisableTriangle("disable-ifcvt-triangle", |
| 38 | cl::init(false), cl::Hidden); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 39 | cl::opt<bool> DisableTriangleF("disable-ifcvt-triangle-false", |
| 40 | cl::init(false), cl::Hidden); |
| 41 | cl::opt<bool> DisableTriangleFR("disable-ifcvt-triangle-false-rev", |
| 42 | cl::init(false), cl::Hidden); |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 43 | cl::opt<bool> DisableDiamond("disable-ifcvt-diamond", |
| 44 | cl::init(false), cl::Hidden); |
| 45 | } |
| 46 | |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 47 | STATISTIC(NumSimple, "Number of simple if-conversions performed"); |
| 48 | STATISTIC(NumSimpleFalse, "Number of simple (F) if-conversions performed"); |
| 49 | STATISTIC(NumTriangle, "Number of triangle if-conversions performed"); |
| 50 | STATISTIC(NumTriangleFalse,"Number of triangle (F) if-conversions performed"); |
| 51 | STATISTIC(NumTriangleFRev, "Number of triangle (F/R) if-conversions performed"); |
| 52 | STATISTIC(NumDiamonds, "Number of diamond if-conversions performed"); |
| 53 | STATISTIC(NumIfConvBBs, "Number of if-converted blocks"); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 54 | |
| 55 | namespace { |
| 56 | class IfConverter : public MachineFunctionPass { |
| 57 | enum BBICKind { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 58 | ICNotAnalyzed, // BB has not been analyzed. |
| 59 | ICReAnalyze, // BB must be re-analyzed. |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 60 | ICNotClassfied, // BB data valid, but not classified. |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 61 | ICSimple, // BB is entry of an one split, no rejoin sub-CFG. |
| 62 | ICSimpleFalse, // Same as ICSimple, but on the false path. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 63 | ICTriangle, // BB is entry of a triangle sub-CFG. |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 64 | ICTriangleFalse, // Same as ICTriangle, but on the false path. |
| 65 | ICTriangleFRev, // Same as ICTriangleFalse, but false path rev condition. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 66 | ICDiamond, // BB is entry of a diamond sub-CFG. |
| 67 | ICChild, // BB is part of the sub-CFG that'll be predicated. |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 68 | ICDead // BB cannot be if-converted again. |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 69 | }; |
| 70 | |
| 71 | /// BBInfo - One per MachineBasicBlock, this is used to cache the result |
| 72 | /// if-conversion feasibility analysis. This includes results from |
| 73 | /// TargetInstrInfo::AnalyzeBranch() (i.e. TBB, FBB, and Cond), and its |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 74 | /// classification, and common tail block of its successors (if it's a |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 75 | /// diamond shape), its size, whether it's predicable, and whether any |
| 76 | /// instruction can clobber the 'would-be' predicate. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 77 | /// |
| 78 | /// Kind - Type of block. See BBICKind. |
| 79 | /// NonPredSize - Number of non-predicated instructions. |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 80 | /// IsAnalyzable - True if AnalyzeBranch() returns false. |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 81 | /// ModifyPredicate - True if BB would modify the predicate (e.g. has |
| 82 | /// cmp, call, etc.) |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 83 | /// BB - Corresponding MachineBasicBlock. |
| 84 | /// TrueBB / FalseBB- See AnalyzeBranch(). |
| 85 | /// BrCond - Conditions for end of block conditional branches. |
| 86 | /// Predicate - Predicate used in the BB. |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 87 | struct BBInfo { |
| 88 | BBICKind Kind; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 89 | unsigned NonPredSize; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 90 | bool IsAnalyzable; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 91 | bool HasFallThrough; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 92 | bool ModifyPredicate; |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 93 | MachineBasicBlock *BB; |
| 94 | MachineBasicBlock *TrueBB; |
| 95 | MachineBasicBlock *FalseBB; |
| 96 | MachineBasicBlock *TailBB; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 97 | std::vector<MachineOperand> BrCond; |
| 98 | std::vector<MachineOperand> Predicate; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 99 | BBInfo() : Kind(ICNotAnalyzed), NonPredSize(0), |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 100 | IsAnalyzable(false), HasFallThrough(false), |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 101 | ModifyPredicate(false), |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 102 | BB(0), TrueBB(0), FalseBB(0), TailBB(0) {} |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 103 | }; |
| 104 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 105 | /// Roots - Basic blocks that do not have successors. These are the starting |
| 106 | /// points of Graph traversal. |
| 107 | std::vector<MachineBasicBlock*> Roots; |
| 108 | |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 109 | /// BBAnalysis - Results of if-conversion feasibility analysis indexed by |
| 110 | /// basic block number. |
| 111 | std::vector<BBInfo> BBAnalysis; |
| 112 | |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 113 | const TargetLowering *TLI; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 114 | const TargetInstrInfo *TII; |
| 115 | bool MadeChange; |
| 116 | public: |
| 117 | static char ID; |
| 118 | IfConverter() : MachineFunctionPass((intptr_t)&ID) {} |
| 119 | |
| 120 | virtual bool runOnMachineFunction(MachineFunction &MF); |
| 121 | virtual const char *getPassName() const { return "If converter"; } |
| 122 | |
| 123 | private: |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 124 | bool ReverseBranchCondition(BBInfo &BBI); |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 125 | bool ValidSimple(BBInfo &TrueBBI) const; |
| 126 | bool ValidTriangle(BBInfo &TrueBBI, BBInfo &FalseBBI, |
| 127 | bool FalseBranch = false) const; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 128 | bool ValidDiamond(BBInfo &TrueBBI, BBInfo &FalseBBI) const; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 129 | void ScanInstructions(BBInfo &BBI); |
| 130 | void AnalyzeBlock(MachineBasicBlock *BB); |
| 131 | bool FeasibilityAnalysis(BBInfo &BBI, std::vector<MachineOperand> &Cond, |
| 132 | bool isTriangle = false, bool RevBranch = false); |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 133 | bool AttemptRestructuring(BBInfo &BBI); |
| 134 | bool AnalyzeBlocks(MachineFunction &MF, |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 135 | std::vector<BBInfo*> &Candidates); |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 136 | void ReTryPreds(MachineBasicBlock *BB); |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 137 | void RemoveExtraEdges(BBInfo &BBI); |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 138 | bool IfConvertSimple(BBInfo &BBI); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 139 | bool IfConvertTriangle(BBInfo &BBI); |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 140 | bool IfConvertDiamond(BBInfo &BBI); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 141 | void PredicateBlock(BBInfo &BBI, |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 142 | std::vector<MachineOperand> &Cond, |
| 143 | bool IgnoreTerm = false); |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 144 | void MergeBlocks(BBInfo &TrueBBI, BBInfo &FalseBBI); |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 145 | |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 146 | // blockAlwaysFallThrough - Block ends without a terminator. |
| 147 | bool blockAlwaysFallThrough(BBInfo &BBI) const { |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 148 | return BBI.IsAnalyzable && BBI.TrueBB == NULL; |
| 149 | } |
| 150 | |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 151 | // IfcvtCandidateCmp - Used to sort if-conversion candidates. |
| 152 | static bool IfcvtCandidateCmp(BBInfo* C1, BBInfo* C2){ |
| 153 | // Favor diamond over triangle, etc. |
| 154 | return (unsigned)C1->Kind < (unsigned)C2->Kind; |
| 155 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 156 | }; |
| 157 | char IfConverter::ID = 0; |
| 158 | } |
| 159 | |
| 160 | FunctionPass *llvm::createIfConverterPass() { return new IfConverter(); } |
| 161 | |
| 162 | bool IfConverter::runOnMachineFunction(MachineFunction &MF) { |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 163 | TLI = MF.getTarget().getTargetLowering(); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 164 | TII = MF.getTarget().getInstrInfo(); |
| 165 | if (!TII) return false; |
| 166 | |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 167 | static int FnNum = -1; |
| 168 | DOUT << "\nIfcvt: function (" << ++FnNum << ") \'" |
| 169 | << MF.getFunction()->getName() << "\'"; |
| 170 | |
| 171 | if (FnNum < IfCvtFnStart || (IfCvtFnStop != -1 && FnNum > IfCvtFnStop)) { |
| 172 | DOUT << " skipped\n"; |
| 173 | return false; |
| 174 | } |
| 175 | DOUT << "\n"; |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 176 | |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 177 | MF.RenumberBlocks(); |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 178 | BBAnalysis.resize(MF.getNumBlockIDs()); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 179 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 180 | // Look for root nodes, i.e. blocks without successors. |
| 181 | for (MachineFunction::iterator I = MF.begin(), E = MF.end(); I != E; ++I) |
| 182 | if (I->succ_size() == 0) |
| 183 | Roots.push_back(I); |
| 184 | |
Evan Cheng | 7f8ff8a | 2007-05-18 19:32:08 +0000 | [diff] [blame] | 185 | std::vector<BBInfo*> Candidates; |
Evan Cheng | 47d2502 | 2007-05-18 01:55:58 +0000 | [diff] [blame] | 186 | MadeChange = false; |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 187 | while (IfCvtLimit == -1 || (int)NumIfConvBBs < IfCvtLimit) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 188 | // Do an intial analysis for each basic block and finding all the potential |
| 189 | // candidates to perform if-convesion. |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 190 | bool Change = AnalyzeBlocks(MF, Candidates); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 191 | while (!Candidates.empty()) { |
| 192 | BBInfo &BBI = *Candidates.back(); |
| 193 | Candidates.pop_back(); |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 194 | |
| 195 | bool RetVal = false; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 196 | switch (BBI.Kind) { |
| 197 | default: assert(false && "Unexpected!"); |
| 198 | break; |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 199 | case ICReAnalyze: |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 200 | // One or more of 'children' have been modified, abort! |
| 201 | case ICDead: |
| 202 | // Block has been already been if-converted, abort! |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 203 | break; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 204 | case ICSimple: |
Evan Cheng | cb78d67 | 2007-06-06 01:12:44 +0000 | [diff] [blame] | 205 | case ICSimpleFalse: { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 206 | bool isFalse = BBI.Kind == ICSimpleFalse; |
| 207 | if ((isFalse && DisableSimpleF) || (!isFalse && DisableSimple)) break; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 208 | DOUT << "Ifcvt (Simple" << (BBI.Kind == ICSimpleFalse ? " false" : "") |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 209 | << "): BB#" << BBI.BB->getNumber() << " (" |
| 210 | << ((BBI.Kind == ICSimpleFalse) |
| 211 | ? BBI.FalseBB->getNumber() : BBI.TrueBB->getNumber()) << ") "; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 212 | RetVal = IfConvertSimple(BBI); |
| 213 | DOUT << (RetVal ? "succeeded!" : "failed!") << "\n"; |
| 214 | if (RetVal) |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 215 | if (isFalse) NumSimpleFalse++; |
| 216 | else NumSimple++; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 217 | break; |
Evan Cheng | cb78d67 | 2007-06-06 01:12:44 +0000 | [diff] [blame] | 218 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 219 | case ICTriangle: |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 220 | case ICTriangleFalse: |
| 221 | case ICTriangleFRev: |
| 222 | bool isFalse = BBI.Kind == ICTriangleFalse; |
| 223 | bool isFalseRev = BBI.Kind == ICTriangleFRev; |
| 224 | if (DisableTriangle && !isFalse && !isFalseRev) break; |
| 225 | if (DisableTriangleF && isFalse) break; |
| 226 | if (DisableTriangleFR && isFalseRev) break; |
| 227 | DOUT << "Ifcvt (Triangle"; |
| 228 | if (isFalse) |
| 229 | DOUT << " false"; |
| 230 | if (isFalseRev) |
| 231 | DOUT << " false/rev"; |
| 232 | DOUT << "): BB#" << BBI.BB->getNumber() << " (T:" |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 233 | << BBI.TrueBB->getNumber() << ",F:" << BBI.FalseBB->getNumber() |
| 234 | << ") "; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 235 | RetVal = IfConvertTriangle(BBI); |
| 236 | DOUT << (RetVal ? "succeeded!" : "failed!") << "\n"; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 237 | if (RetVal) { |
| 238 | if (isFalseRev) NumTriangleFRev++; |
| 239 | else if (isFalse) NumTriangleFalse++; |
| 240 | else NumTriangle++; |
| 241 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 242 | break; |
| 243 | case ICDiamond: |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 244 | if (DisableDiamond) break; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 245 | DOUT << "Ifcvt (Diamond): BB#" << BBI.BB->getNumber() << " (T:" |
| 246 | << BBI.TrueBB->getNumber() << ",F:" << BBI.FalseBB->getNumber(); |
| 247 | if (BBI.TailBB) |
| 248 | DOUT << "," << BBI.TailBB->getNumber() ; |
| 249 | DOUT << ") "; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 250 | RetVal = IfConvertDiamond(BBI); |
| 251 | DOUT << (RetVal ? "succeeded!" : "failed!") << "\n"; |
| 252 | if (RetVal) NumDiamonds++; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 253 | break; |
| 254 | } |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 255 | Change |= RetVal; |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 256 | |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 257 | if (IfCvtLimit != -1 && (int)NumIfConvBBs >= IfCvtLimit) |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 258 | break; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 259 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 260 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 261 | if (!Change) |
| 262 | break; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 263 | MadeChange |= Change; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 264 | } |
Evan Cheng | 47d2502 | 2007-05-18 01:55:58 +0000 | [diff] [blame] | 265 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 266 | Roots.clear(); |
Evan Cheng | 47d2502 | 2007-05-18 01:55:58 +0000 | [diff] [blame] | 267 | BBAnalysis.clear(); |
| 268 | |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 269 | return MadeChange; |
| 270 | } |
| 271 | |
| 272 | static MachineBasicBlock *findFalseBlock(MachineBasicBlock *BB, |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 273 | MachineBasicBlock *TrueBB) { |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 274 | for (MachineBasicBlock::succ_iterator SI = BB->succ_begin(), |
| 275 | E = BB->succ_end(); SI != E; ++SI) { |
| 276 | MachineBasicBlock *SuccBB = *SI; |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 277 | if (SuccBB != TrueBB) |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 278 | return SuccBB; |
| 279 | } |
| 280 | return NULL; |
| 281 | } |
| 282 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 283 | bool IfConverter::ReverseBranchCondition(BBInfo &BBI) { |
| 284 | if (!TII->ReverseBranchCondition(BBI.BrCond)) { |
| 285 | TII->RemoveBranch(*BBI.BB); |
| 286 | TII->InsertBranch(*BBI.BB, BBI.FalseBB, BBI.TrueBB, BBI.BrCond); |
| 287 | std::swap(BBI.TrueBB, BBI.FalseBB); |
| 288 | return true; |
| 289 | } |
| 290 | return false; |
| 291 | } |
| 292 | |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 293 | /// getNextBlock - Returns the next block in the function blocks ordering. If |
| 294 | /// it is the end, returns NULL. |
| 295 | static inline MachineBasicBlock *getNextBlock(MachineBasicBlock *BB) { |
| 296 | MachineFunction::iterator I = BB; |
| 297 | MachineFunction::iterator E = BB->getParent()->end(); |
| 298 | if (++I == E) |
| 299 | return NULL; |
| 300 | return I; |
| 301 | } |
| 302 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 303 | /// ValidSimple - Returns true if the 'true' block (along with its |
| 304 | /// predecessor) forms a valid simple shape for ifcvt. |
| 305 | bool IfConverter::ValidSimple(BBInfo &TrueBBI) const { |
| 306 | return !blockAlwaysFallThrough(TrueBBI) && |
| 307 | TrueBBI.BrCond.size() == 0 && TrueBBI.BB->pred_size() == 1; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 308 | } |
| 309 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 310 | /// ValidTriangle - Returns true if the 'true' and 'false' blocks (along |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 311 | /// with their common predecessor) forms a valid triangle shape for ifcvt. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 312 | bool IfConverter::ValidTriangle(BBInfo &TrueBBI, BBInfo &FalseBBI, |
| 313 | bool FalseBranch) const { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 314 | if (TrueBBI.BB->pred_size() != 1) |
| 315 | return false; |
| 316 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 317 | MachineBasicBlock *TExit = FalseBranch ? TrueBBI.FalseBB : TrueBBI.TrueBB; |
| 318 | if (!TExit && blockAlwaysFallThrough(TrueBBI)) { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 319 | MachineFunction::iterator I = TrueBBI.BB; |
| 320 | if (++I == TrueBBI.BB->getParent()->end()) |
| 321 | return false; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 322 | TExit = I; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 323 | } |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 324 | return TExit && TExit == FalseBBI.BB; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 325 | } |
| 326 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 327 | /// ValidDiamond - Returns true if the 'true' and 'false' blocks (along |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 328 | /// with their common predecessor) forms a valid diamond shape for ifcvt. |
| 329 | bool IfConverter::ValidDiamond(BBInfo &TrueBBI, BBInfo &FalseBBI) const { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 330 | MachineBasicBlock *TT = TrueBBI.TrueBB; |
| 331 | MachineBasicBlock *FT = FalseBBI.TrueBB; |
| 332 | |
| 333 | if (!TT && blockAlwaysFallThrough(TrueBBI)) |
| 334 | TT = getNextBlock(TrueBBI.BB); |
| 335 | if (!FT && blockAlwaysFallThrough(FalseBBI)) |
| 336 | FT = getNextBlock(FalseBBI.BB); |
| 337 | if (TT != FT) |
| 338 | return false; |
| 339 | if (TT == NULL && (TrueBBI.IsAnalyzable || FalseBBI.IsAnalyzable)) |
| 340 | return false; |
| 341 | // FIXME: Allow false block to have an early exit? |
| 342 | return (TrueBBI.BB->pred_size() == 1 && |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 343 | FalseBBI.BB->pred_size() == 1 && |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 344 | !TrueBBI.FalseBB && !FalseBBI.FalseBB); |
| 345 | } |
| 346 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 347 | /// AnalyzeBlock - Analyze the structure of the sub-CFG starting from |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 348 | /// the specified block. Record its successors and whether it looks like an |
| 349 | /// if-conversion candidate. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 350 | void IfConverter::AnalyzeBlock(MachineBasicBlock *BB) { |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 351 | BBInfo &BBI = BBAnalysis[BB->getNumber()]; |
| 352 | |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 353 | if (BBI.Kind == ICReAnalyze) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 354 | BBI.BrCond.clear(); |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 355 | BBI.TrueBB = BBI.FalseBB = NULL; |
| 356 | } else { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 357 | if (BBI.Kind != ICNotAnalyzed) |
| 358 | return; // Already analyzed. |
| 359 | BBI.BB = BB; |
| 360 | BBI.NonPredSize = std::distance(BB->begin(), BB->end()); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 361 | } |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 362 | |
Evan Cheng | 4bec8ae | 2007-05-25 00:59:01 +0000 | [diff] [blame] | 363 | // Look for 'root' of a simple (non-nested) triangle or diamond. |
| 364 | BBI.Kind = ICNotClassfied; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 365 | BBI.IsAnalyzable = |
| 366 | !TII->AnalyzeBranch(*BB, BBI.TrueBB, BBI.FalseBB, BBI.BrCond); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 367 | BBI.HasFallThrough = BBI.IsAnalyzable && BBI.FalseBB == NULL; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 368 | // Unanalyable or ends with fallthrough or unconditional branch. |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 369 | if (!BBI.IsAnalyzable || BBI.BrCond.size() == 0) |
Evan Cheng | 4bec8ae | 2007-05-25 00:59:01 +0000 | [diff] [blame] | 370 | return; |
Evan Cheng | e004317 | 2007-06-05 20:38:42 +0000 | [diff] [blame] | 371 | // Do not ifcvt if either path is a back edge to the entry block. |
| 372 | if (BBI.TrueBB == BB || BBI.FalseBB == BB) |
| 373 | return; |
Evan Cheng | 4bec8ae | 2007-05-25 00:59:01 +0000 | [diff] [blame] | 374 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 375 | AnalyzeBlock(BBI.TrueBB); |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 376 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 377 | |
| 378 | // No false branch. This BB must end with a conditional branch and a |
| 379 | // fallthrough. |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 380 | if (!BBI.FalseBB) |
| 381 | BBI.FalseBB = findFalseBlock(BB, BBI.TrueBB); |
| 382 | assert(BBI.FalseBB && "Expected to find the fallthrough block!"); |
Evan Cheng | c5d05ef | 2007-05-16 05:11:10 +0000 | [diff] [blame] | 383 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 384 | AnalyzeBlock(BBI.FalseBB); |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 385 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 386 | |
| 387 | // If both paths are dead, then forget about it. |
| 388 | if (TrueBBI.Kind == ICDead && FalseBBI.Kind == ICDead) { |
| 389 | BBI.Kind = ICDead; |
| 390 | return; |
| 391 | } |
| 392 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 393 | // Look for more opportunities to if-convert a triangle. Try to restructure |
| 394 | // the CFG to form a triangle with the 'false' path. |
| 395 | std::vector<MachineOperand> RevCond(BBI.BrCond); |
| 396 | bool CanRevCond = !TII->ReverseBranchCondition(RevCond); |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 397 | |
| 398 | if (CanRevCond && ValidDiamond(TrueBBI, FalseBBI) && |
| 399 | !(TrueBBI.ModifyPredicate && FalseBBI.ModifyPredicate) && |
| 400 | FeasibilityAnalysis(TrueBBI, BBI.BrCond) && |
| 401 | FeasibilityAnalysis(FalseBBI, RevCond)) { |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 402 | // Diamond: |
| 403 | // EBB |
| 404 | // / \_ |
| 405 | // | | |
| 406 | // TBB FBB |
| 407 | // \ / |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 408 | // TailBB |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 409 | // Note TailBB can be empty. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 410 | BBI.Kind = ICDiamond; |
| 411 | TrueBBI.Kind = FalseBBI.Kind = ICChild; |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 412 | BBI.TailBB = TrueBBI.TrueBB; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 413 | } else { |
| 414 | // FIXME: Consider duplicating if BB is small. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 415 | if (ValidTriangle(TrueBBI, FalseBBI) && |
| 416 | FeasibilityAnalysis(TrueBBI, BBI.BrCond, true)) { |
| 417 | // Triangle: |
| 418 | // EBB |
| 419 | // | \_ |
| 420 | // | | |
| 421 | // | TBB |
| 422 | // | / |
| 423 | // FBB |
| 424 | BBI.Kind = ICTriangle; |
| 425 | TrueBBI.Kind = ICChild; |
| 426 | } else if (ValidSimple(TrueBBI) && |
| 427 | FeasibilityAnalysis(TrueBBI, BBI.BrCond)) { |
| 428 | // Simple (split, no rejoin): |
| 429 | // EBB |
| 430 | // | \_ |
| 431 | // | | |
| 432 | // | TBB---> exit |
| 433 | // | |
| 434 | // FBB |
| 435 | BBI.Kind = ICSimple; |
| 436 | TrueBBI.Kind = ICChild; |
| 437 | } else if (CanRevCond) { |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 438 | // Try the other path... |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 439 | if (ValidTriangle(FalseBBI, TrueBBI) && |
| 440 | FeasibilityAnalysis(FalseBBI, RevCond, true)) { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 441 | BBI.Kind = ICTriangleFalse; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 442 | FalseBBI.Kind = ICChild; |
| 443 | } else if (ValidTriangle(FalseBBI, TrueBBI, true) && |
| 444 | FeasibilityAnalysis(FalseBBI, RevCond, true, true)) { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 445 | BBI.Kind = ICTriangleFRev; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 446 | FalseBBI.Kind = ICChild; |
| 447 | } else if (ValidSimple(FalseBBI) && |
| 448 | FeasibilityAnalysis(FalseBBI, RevCond)) { |
| 449 | BBI.Kind = ICSimpleFalse; |
| 450 | FalseBBI.Kind = ICChild; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 451 | } |
| 452 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 453 | } |
| 454 | return; |
| 455 | } |
| 456 | |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 457 | /// FeasibilityAnalysis - Determine if the block is predicable. In most |
| 458 | /// cases, that means all the instructions in the block has M_PREDICABLE flag. |
| 459 | /// Also checks if the block contains any instruction which can clobber a |
| 460 | /// predicate (e.g. condition code register). If so, the block is not |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 461 | /// predicable unless it's the last instruction. |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 462 | bool IfConverter::FeasibilityAnalysis(BBInfo &BBI, |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 463 | std::vector<MachineOperand> &Pred, |
| 464 | bool isTriangle, bool RevBranch) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 465 | // If the block is dead, or it is going to be the entry block of a sub-CFG |
| 466 | // that will be if-converted, then it cannot be predicated. |
| 467 | if (BBI.Kind != ICNotAnalyzed && |
| 468 | BBI.Kind != ICNotClassfied && |
| 469 | BBI.Kind != ICChild) |
| 470 | return false; |
| 471 | |
| 472 | // Check predication threshold. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 473 | if (BBI.NonPredSize == 0 || BBI.NonPredSize > TLI->getIfCvtBlockSizeLimit()) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 474 | return false; |
| 475 | |
| 476 | // If it is already predicated, check if its predicate subsumes the new |
| 477 | // predicate. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 478 | if (BBI.Predicate.size() && !TII->SubsumesPredicate(BBI.Predicate, Pred)) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 479 | return false; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 480 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 481 | bool SeenPredMod = false; |
| 482 | bool SeenCondBr = false; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 483 | for (MachineBasicBlock::iterator I = BBI.BB->begin(), E = BBI.BB->end(); |
| 484 | I != E; ++I) { |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 485 | const TargetInstrDescriptor *TID = I->getInstrDescriptor(); |
| 486 | if (SeenPredMod) { |
| 487 | // Predicate modification instruction should end the block (except for |
| 488 | // already predicated instructions and end of block branches). |
| 489 | if (!TII->isPredicated(I)) { |
| 490 | // This is the 'true' block of a triangle, i.e. its 'true' block is |
| 491 | // the same as the 'false' block of the entry. So false positive |
| 492 | // is ok. |
| 493 | if (isTriangle && !SeenCondBr && BBI.IsAnalyzable && |
| 494 | (TID->Flags & M_BRANCH_FLAG) != 0 && |
| 495 | (TID->Flags & M_BARRIER_FLAG) == 0) { |
| 496 | // This is the first conditional branch, test predicate subsumsion. |
| 497 | std::vector<MachineOperand> RevPred(Pred); |
| 498 | std::vector<MachineOperand> Cond(BBI.BrCond); |
| 499 | if (RevBranch) { |
| 500 | if (TII->ReverseBranchCondition(Cond)) |
| 501 | return false; |
| 502 | } |
| 503 | if (TII->ReverseBranchCondition(RevPred) || |
| 504 | !TII->SubsumesPredicate(Cond, RevPred)) |
| 505 | return false; |
| 506 | SeenCondBr = true; |
| 507 | continue; // Conditional branches is not predicable. |
| 508 | } |
| 509 | return false; |
| 510 | } |
| 511 | } |
| 512 | |
| 513 | if (TID->Flags & M_CLOBBERS_PRED) { |
| 514 | BBI.ModifyPredicate = true; |
| 515 | SeenPredMod = true; |
| 516 | } |
| 517 | |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 518 | if (!I->isPredicable()) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 519 | return false; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 520 | } |
| 521 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 522 | return true; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 523 | } |
| 524 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 525 | /// AttemptRestructuring - Restructure the sub-CFG rooted in the given block to |
| 526 | /// expose more if-conversion opportunities. e.g. |
| 527 | /// |
| 528 | /// cmp |
| 529 | /// b le BB1 |
| 530 | /// / \____ |
| 531 | /// / | |
| 532 | /// cmp | |
| 533 | /// b eq BB1 | |
| 534 | /// / \____ | |
| 535 | /// / \ | |
| 536 | /// BB1 |
| 537 | /// ==> |
| 538 | /// |
| 539 | /// cmp |
| 540 | /// b eq BB1 |
| 541 | /// / \____ |
| 542 | /// / | |
| 543 | /// cmp | |
| 544 | /// b le BB1 | |
| 545 | /// / \____ | |
| 546 | /// / \ | |
| 547 | /// BB1 |
| 548 | bool IfConverter::AttemptRestructuring(BBInfo &BBI) { |
| 549 | return false; |
| 550 | } |
| 551 | |
| 552 | /// AnalyzeBlocks - Analyze all blocks and find entries for all if-conversion |
| 553 | /// candidates. It returns true if any CFG restructuring is done to expose more |
| 554 | /// if-conversion opportunities. |
| 555 | bool IfConverter::AnalyzeBlocks(MachineFunction &MF, |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 556 | std::vector<BBInfo*> &Candidates) { |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 557 | bool Change = false; |
Evan Cheng | 36489bb | 2007-05-18 19:26:33 +0000 | [diff] [blame] | 558 | std::set<MachineBasicBlock*> Visited; |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 559 | for (unsigned i = 0, e = Roots.size(); i != e; ++i) { |
| 560 | for (idf_ext_iterator<MachineBasicBlock*> I=idf_ext_begin(Roots[i],Visited), |
| 561 | E = idf_ext_end(Roots[i], Visited); I != E; ++I) { |
| 562 | MachineBasicBlock *BB = *I; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 563 | AnalyzeBlock(BB); |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 564 | BBInfo &BBI = BBAnalysis[BB->getNumber()]; |
| 565 | switch (BBI.Kind) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 566 | case ICSimple: |
| 567 | case ICSimpleFalse: |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 568 | case ICTriangle: |
| 569 | case ICDiamond: |
| 570 | Candidates.push_back(&BBI); |
| 571 | break; |
| 572 | default: |
| 573 | Change |= AttemptRestructuring(BBI); |
| 574 | break; |
| 575 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 576 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 577 | } |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 578 | |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 579 | // Sort to favor more complex ifcvt scheme. |
| 580 | std::stable_sort(Candidates.begin(), Candidates.end(), IfcvtCandidateCmp); |
| 581 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 582 | return Change; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 583 | } |
| 584 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 585 | /// canFallThroughTo - Returns true either if ToBB is the next block after BB or |
| 586 | /// that all the intervening blocks are empty (given BB can fall through to its |
| 587 | /// next block). |
| 588 | static bool canFallThroughTo(MachineBasicBlock *BB, MachineBasicBlock *ToBB) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 589 | MachineFunction::iterator I = BB; |
Evan Cheng | c53ef58 | 2007-06-05 07:05:25 +0000 | [diff] [blame] | 590 | MachineFunction::iterator TI = ToBB; |
| 591 | MachineFunction::iterator E = BB->getParent()->end(); |
| 592 | while (++I != TI) |
| 593 | if (I == E || !I->empty()) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 594 | return false; |
| 595 | return true; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 596 | } |
| 597 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 598 | /// ReTryPreds - Invalidate predecessor BB info so it would be re-analyzed |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 599 | /// to determine if it can be if-converted. |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 600 | void IfConverter::ReTryPreds(MachineBasicBlock *BB) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 601 | for (MachineBasicBlock::pred_iterator PI = BB->pred_begin(), |
| 602 | E = BB->pred_end(); PI != E; ++PI) { |
| 603 | BBInfo &PBBI = BBAnalysis[(*PI)->getNumber()]; |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 604 | if (PBBI.Kind == ICNotClassfied) |
| 605 | PBBI.Kind = ICReAnalyze; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 606 | } |
| 607 | } |
| 608 | |
Evan Cheng | c8ed9ba | 2007-05-29 22:31:16 +0000 | [diff] [blame] | 609 | /// InsertUncondBranch - Inserts an unconditional branch from BB to ToBB. |
| 610 | /// |
| 611 | static void InsertUncondBranch(MachineBasicBlock *BB, MachineBasicBlock *ToBB, |
| 612 | const TargetInstrInfo *TII) { |
| 613 | std::vector<MachineOperand> NoCond; |
| 614 | TII->InsertBranch(*BB, ToBB, NULL, NoCond); |
| 615 | } |
| 616 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 617 | /// RemoveExtraEdges - Remove true / false edges if either / both are no longer |
| 618 | /// successors. |
| 619 | void IfConverter::RemoveExtraEdges(BBInfo &BBI) { |
| 620 | MachineBasicBlock *TBB = NULL, *FBB = NULL; |
| 621 | std::vector<MachineOperand> Cond; |
| 622 | bool isAnalyzable = !TII->AnalyzeBranch(*BBI.BB, TBB, FBB, Cond); |
| 623 | bool CanFallthrough = isAnalyzable && (TBB == NULL || FBB == NULL); |
| 624 | if (BBI.TrueBB && BBI.BB->isSuccessor(BBI.TrueBB)) |
| 625 | if (!(BBI.TrueBB == TBB || BBI.TrueBB == FBB || |
| 626 | (CanFallthrough && getNextBlock(BBI.BB) == BBI.TrueBB))) |
| 627 | BBI.BB->removeSuccessor(BBI.TrueBB); |
| 628 | if (BBI.FalseBB && BBI.BB->isSuccessor(BBI.FalseBB)) |
| 629 | if (!(BBI.FalseBB == TBB || BBI.FalseBB == FBB || |
| 630 | (CanFallthrough && getNextBlock(BBI.BB) == BBI.FalseBB))) |
| 631 | BBI.BB->removeSuccessor(BBI.FalseBB); |
| 632 | } |
| 633 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 634 | /// IfConvertSimple - If convert a simple (split, no rejoin) sub-CFG. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 635 | /// |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 636 | bool IfConverter::IfConvertSimple(BBInfo &BBI) { |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 637 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
| 638 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
| 639 | BBInfo *CvtBBI = &TrueBBI; |
| 640 | BBInfo *NextBBI = &FalseBBI; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 641 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 642 | std::vector<MachineOperand> Cond(BBI.BrCond); |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 643 | if (BBI.Kind == ICSimpleFalse) { |
Evan Cheng | b5a0690 | 2007-06-01 20:29:21 +0000 | [diff] [blame] | 644 | std::swap(CvtBBI, NextBBI); |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 645 | TII->ReverseBranchCondition(Cond); |
Evan Cheng | b5a0690 | 2007-06-01 20:29:21 +0000 | [diff] [blame] | 646 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 647 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 648 | PredicateBlock(*CvtBBI, Cond); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 649 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 650 | // Merge converted block into entry block. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 651 | BBI.NonPredSize -= TII->RemoveBranch(*BBI.BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 652 | MergeBlocks(BBI, *CvtBBI); |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 653 | |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 654 | bool IterIfcvt = true; |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 655 | if (!canFallThroughTo(BBI.BB, NextBBI->BB)) { |
Evan Cheng | c8ed9ba | 2007-05-29 22:31:16 +0000 | [diff] [blame] | 656 | InsertUncondBranch(BBI.BB, NextBBI->BB, TII); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 657 | BBI.HasFallThrough = false; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 658 | // Now ifcvt'd block will look like this: |
| 659 | // BB: |
| 660 | // ... |
| 661 | // t, f = cmp |
| 662 | // if t op |
| 663 | // b BBf |
| 664 | // |
| 665 | // We cannot further ifcvt this block because the unconditional branch |
| 666 | // will have to be predicated on the new condition, that will not be |
| 667 | // available if cmp executes. |
| 668 | IterIfcvt = false; |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 669 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 670 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 671 | RemoveExtraEdges(BBI); |
| 672 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 673 | // Update block info. BB can be iteratively if-converted. |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 674 | if (IterIfcvt) |
| 675 | BBI.Kind = ICReAnalyze; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 676 | else |
| 677 | BBI.Kind = ICDead; |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 678 | ReTryPreds(BBI.BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 679 | CvtBBI->Kind = ICDead; |
| 680 | |
| 681 | // FIXME: Must maintain LiveIns. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 682 | return true; |
| 683 | } |
| 684 | |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 685 | /// IfConvertTriangle - If convert a triangle sub-CFG. |
| 686 | /// |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 687 | bool IfConverter::IfConvertTriangle(BBInfo &BBI) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 688 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 689 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
| 690 | BBInfo *CvtBBI = &TrueBBI; |
| 691 | BBInfo *NextBBI = &FalseBBI; |
| 692 | |
| 693 | std::vector<MachineOperand> Cond(BBI.BrCond); |
| 694 | if (BBI.Kind == ICTriangleFalse || BBI.Kind == ICTriangleFRev) { |
| 695 | if (BBI.Kind == ICTriangleFRev) { |
| 696 | ReverseBranchCondition(*NextBBI); |
| 697 | // False BB has been changed, modify its predecessors (except for this |
| 698 | // one) so they don't get ifcvt'ed based on bad intel. |
| 699 | for (MachineBasicBlock::pred_iterator PI = NextBBI->BB->pred_begin(), |
| 700 | E = NextBBI->BB->pred_end(); PI != E; ++PI) { |
| 701 | MachineBasicBlock *PBB = *PI; |
| 702 | if (PBB == BBI.BB) |
| 703 | continue; |
| 704 | BBInfo &PBBI = BBAnalysis[PBB->getNumber()]; |
| 705 | PBBI.Kind = ICReAnalyze; |
| 706 | } |
| 707 | } |
| 708 | std::swap(CvtBBI, NextBBI); |
| 709 | TII->ReverseBranchCondition(Cond); |
| 710 | } |
Evan Cheng | 8c52938 | 2007-06-01 07:41:07 +0000 | [diff] [blame] | 711 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 712 | // Predicate the 'true' block after removing its branch. |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 713 | TrueBBI.NonPredSize -= TII->RemoveBranch(*CvtBBI->BB); |
| 714 | PredicateBlock(*CvtBBI, Cond); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 715 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 716 | // If 'true' block has a 'false' successor, add an exit branch to it. |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 717 | bool HasEarlyExit = CvtBBI->FalseBB != NULL; |
Evan Cheng | 8ed680c | 2007-06-05 01:31:40 +0000 | [diff] [blame] | 718 | if (HasEarlyExit) { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 719 | std::vector<MachineOperand> RevCond(CvtBBI->BrCond); |
Evan Cheng | 8c52938 | 2007-06-01 07:41:07 +0000 | [diff] [blame] | 720 | if (TII->ReverseBranchCondition(RevCond)) |
| 721 | assert(false && "Unable to reverse branch condition!"); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 722 | TII->InsertBranch(*CvtBBI->BB, CvtBBI->FalseBB, NULL, RevCond); |
Evan Cheng | 8c52938 | 2007-06-01 07:41:07 +0000 | [diff] [blame] | 723 | } |
| 724 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 725 | // Now merge the entry of the triangle with the true block. |
| 726 | BBI.NonPredSize -= TII->RemoveBranch(*BBI.BB); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 727 | MergeBlocks(BBI, *CvtBBI); |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 728 | |
| 729 | // Merge in the 'false' block if the 'false' block has no other |
| 730 | // predecessors. Otherwise, add a unconditional branch from to 'false'. |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 731 | bool FalseBBDead = false; |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 732 | bool IterIfcvt = true; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 733 | bool isFallThrough = canFallThroughTo(BBI.BB, NextBBI->BB); |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 734 | if (!isFallThrough) { |
| 735 | // Only merge them if the true block does not fallthrough to the false |
| 736 | // block. By not merging them, we make it possible to iteratively |
| 737 | // ifcvt the blocks. |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 738 | if (!HasEarlyExit && NextBBI->BB->pred_size() == 1) { |
| 739 | MergeBlocks(BBI, *NextBBI); |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 740 | FalseBBDead = true; |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 741 | } else { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 742 | InsertUncondBranch(BBI.BB, NextBBI->BB, TII); |
| 743 | BBI.HasFallThrough = false; |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 744 | } |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 745 | // Mixed predicated and unpredicated code. This cannot be iteratively |
| 746 | // predicated. |
| 747 | IterIfcvt = false; |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 748 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 749 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 750 | RemoveExtraEdges(BBI); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 751 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 752 | // Update block info. BB can be iteratively if-converted. |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 753 | if (IterIfcvt) |
| 754 | BBI.Kind = ICReAnalyze; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 755 | else |
| 756 | BBI.Kind = ICDead; |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 757 | ReTryPreds(BBI.BB); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 758 | CvtBBI->Kind = ICDead; |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 759 | if (FalseBBDead) |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 760 | NextBBI->Kind = ICDead; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 761 | |
| 762 | // FIXME: Must maintain LiveIns. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 763 | return true; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 764 | } |
| 765 | |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 766 | /// IfConvertDiamond - If convert a diamond sub-CFG. |
| 767 | /// |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 768 | bool IfConverter::IfConvertDiamond(BBInfo &BBI) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 769 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 770 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 771 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 772 | SmallVector<MachineInstr*, 2> Dups; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 773 | if (!BBI.TailBB) { |
| 774 | // No common merge block. Check if the terminators (e.g. return) are |
| 775 | // the same or predicable. |
| 776 | MachineBasicBlock::iterator TT = BBI.TrueBB->getFirstTerminator(); |
| 777 | MachineBasicBlock::iterator FT = BBI.FalseBB->getFirstTerminator(); |
| 778 | while (TT != BBI.TrueBB->end() && FT != BBI.FalseBB->end()) { |
| 779 | if (TT->isIdenticalTo(FT)) |
| 780 | Dups.push_back(TT); // Will erase these later. |
| 781 | else if (!TT->isPredicable() && !FT->isPredicable()) |
| 782 | return false; // Can't if-convert. Abort! |
| 783 | ++TT; |
| 784 | ++FT; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 785 | } |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 786 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 787 | // One of the two pathes have more terminators, make sure they are |
| 788 | // all predicable. |
| 789 | while (TT != BBI.TrueBB->end()) { |
| 790 | if (!TT->isPredicable()) { |
| 791 | return false; // Can't if-convert. Abort! |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 792 | } |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 793 | ++TT; |
| 794 | } |
| 795 | while (FT != BBI.FalseBB->end()) { |
| 796 | if (!FT->isPredicable()) { |
| 797 | return false; // Can't if-convert. Abort! |
| 798 | } |
| 799 | ++FT; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 800 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 801 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 802 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 803 | // Remove the duplicated instructions from the 'true' block. |
| 804 | for (unsigned i = 0, e = Dups.size(); i != e; ++i) { |
| 805 | Dups[i]->eraseFromParent(); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 806 | --TrueBBI.NonPredSize; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 807 | } |
| 808 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 809 | // Merge the 'true' and 'false' blocks by copying the instructions |
| 810 | // from the 'false' block to the 'true' block. That is, unless the true |
| 811 | // block would clobber the predicate, in that case, do the opposite. |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 812 | BBInfo *BBI1 = &TrueBBI; |
| 813 | BBInfo *BBI2 = &FalseBBI; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 814 | std::vector<MachineOperand> RevCond(BBI.BrCond); |
| 815 | TII->ReverseBranchCondition(RevCond); |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 816 | std::vector<MachineOperand> *Cond1 = &BBI.BrCond; |
| 817 | std::vector<MachineOperand> *Cond2 = &RevCond; |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 818 | // Check the 'true' and 'false' blocks if either isn't ended with a branch. |
| 819 | // Either the block fallthrough to another block or it ends with a |
| 820 | // return. If it's the former, add a branch to its successor. |
| 821 | bool NeedBr1 = !BBI1->TrueBB && BBI1->BB->succ_size(); |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 822 | bool NeedBr2 = !BBI2->TrueBB && BBI2->BB->succ_size(); |
| 823 | |
| 824 | if ((TrueBBI.ModifyPredicate && !FalseBBI.ModifyPredicate) || |
| 825 | (!TrueBBI.ModifyPredicate && !FalseBBI.ModifyPredicate && |
| 826 | NeedBr1 && !NeedBr2)) { |
| 827 | std::swap(BBI1, BBI2); |
| 828 | std::swap(Cond1, Cond2); |
| 829 | std::swap(NeedBr1, NeedBr2); |
| 830 | } |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 831 | |
| 832 | // Predicate the 'true' block after removing its branch. |
| 833 | BBI1->NonPredSize -= TII->RemoveBranch(*BBI1->BB); |
| 834 | PredicateBlock(*BBI1, *Cond1); |
| 835 | |
| 836 | // Add an early exit branch if needed. |
| 837 | if (NeedBr1) |
| 838 | TII->InsertBranch(*BBI1->BB, *BBI1->BB->succ_begin(), NULL, *Cond1); |
| 839 | |
| 840 | // Predicate the 'false' block. |
| 841 | PredicateBlock(*BBI2, *Cond2, true); |
| 842 | |
| 843 | // Add an unconditional branch from 'false' to to 'false' successor if it |
| 844 | // will not be the fallthrough block. |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 845 | if (NeedBr2 && !NeedBr1) { |
| 846 | // If BBI2 isn't going to be merged in, then the existing fallthrough |
| 847 | // or branch is fine. |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 848 | if (!canFallThroughTo(BBI.BB, *BBI2->BB->succ_begin())) { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 849 | InsertUncondBranch(BBI2->BB, *BBI2->BB->succ_begin(), TII); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 850 | BBI2->HasFallThrough = false; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 851 | } |
| 852 | } |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 853 | |
| 854 | // Keep them as two separate blocks if there is an early exit. |
| 855 | if (!NeedBr1) |
| 856 | MergeBlocks(*BBI1, *BBI2); |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 857 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 858 | // Remove the conditional branch from entry to the blocks. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 859 | BBI.NonPredSize -= TII->RemoveBranch(*BBI.BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 860 | |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 861 | // Merge the combined block into the entry of the diamond. |
| 862 | MergeBlocks(BBI, *BBI1); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 863 | |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 864 | // 'True' and 'false' aren't combined, see if we need to add a unconditional |
| 865 | // branch to the 'false' block. |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 866 | if (NeedBr1 && !canFallThroughTo(BBI.BB, BBI2->BB)) { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 867 | InsertUncondBranch(BBI.BB, BBI2->BB, TII); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 868 | BBI1->HasFallThrough = false; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 869 | } |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 870 | |
Evan Cheng | 58fbb9f | 2007-05-29 23:37:20 +0000 | [diff] [blame] | 871 | // If the if-converted block fallthrough or unconditionally branch into the |
| 872 | // tail block, and the tail block does not have other predecessors, then |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 873 | // fold the tail block in as well. |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 874 | BBInfo *CvtBBI = NeedBr1 ? BBI2 : &BBI; |
Evan Cheng | 58fbb9f | 2007-05-29 23:37:20 +0000 | [diff] [blame] | 875 | if (BBI.TailBB && |
Evan Cheng | f15d44c | 2007-05-31 20:53:33 +0000 | [diff] [blame] | 876 | BBI.TailBB->pred_size() == 1 && CvtBBI->BB->succ_size() == 1) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 877 | CvtBBI->NonPredSize -= TII->RemoveBranch(*CvtBBI->BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 878 | BBInfo TailBBI = BBAnalysis[BBI.TailBB->getNumber()]; |
| 879 | MergeBlocks(*CvtBBI, TailBBI); |
| 880 | TailBBI.Kind = ICDead; |
| 881 | } |
| 882 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 883 | RemoveExtraEdges(BBI); |
| 884 | |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 885 | // Update block info. |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 886 | BBI.Kind = ICDead; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 887 | TrueBBI.Kind = ICDead; |
| 888 | FalseBBI.Kind = ICDead; |
| 889 | |
| 890 | // FIXME: Must maintain LiveIns. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 891 | return true; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 892 | } |
| 893 | |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 894 | /// PredicateBlock - Predicate every instruction in the block with the specified |
| 895 | /// condition. If IgnoreTerm is true, skip over all terminator instructions. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 896 | void IfConverter::PredicateBlock(BBInfo &BBI, |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 897 | std::vector<MachineOperand> &Cond, |
| 898 | bool IgnoreTerm) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 899 | for (MachineBasicBlock::iterator I = BBI.BB->begin(), E = BBI.BB->end(); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 900 | I != E; ++I) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 901 | if (IgnoreTerm && TII->isTerminatorInstr(I->getOpcode())) |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 902 | continue; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 903 | if (TII->isPredicated(I)) |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 904 | continue; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 905 | if (!TII->PredicateInstruction(I, Cond)) { |
Evan Cheng | fe57a7e | 2007-06-01 00:55:26 +0000 | [diff] [blame] | 906 | cerr << "Unable to predicate " << *I << "!\n"; |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 907 | abort(); |
| 908 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 909 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 910 | |
| 911 | BBI.NonPredSize = 0; |
Evan Cheng | 2acdbcc | 2007-06-08 19:17:12 +0000 | [diff] [blame] | 912 | std::copy(Cond.begin(), Cond.end(), std::back_inserter(BBI.Predicate)); |
| 913 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 914 | NumIfConvBBs++; |
| 915 | } |
| 916 | |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 917 | /// MergeBlocks - Move all instructions from FromBB to the end of ToBB. |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 918 | /// |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 919 | void IfConverter::MergeBlocks(BBInfo &ToBBI, BBInfo &FromBBI) { |
| 920 | ToBBI.BB->splice(ToBBI.BB->end(), |
| 921 | FromBBI.BB, FromBBI.BB->begin(), FromBBI.BB->end()); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 922 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 923 | // Redirect all branches to FromBB to ToBB. |
Evan Cheng | a1a9f40 | 2007-06-05 22:03:53 +0000 | [diff] [blame] | 924 | std::vector<MachineBasicBlock *> Preds(FromBBI.BB->pred_begin(), |
| 925 | FromBBI.BB->pred_end()); |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 926 | for (unsigned i = 0, e = Preds.size(); i != e; ++i) { |
| 927 | MachineBasicBlock *Pred = Preds[i]; |
| 928 | if (Pred == ToBBI.BB) |
| 929 | continue; |
| 930 | Pred->ReplaceUsesOfBlockWith(FromBBI.BB, ToBBI.BB); |
| 931 | } |
| 932 | |
| 933 | std::vector<MachineBasicBlock *> Succs(FromBBI.BB->succ_begin(), |
| 934 | FromBBI.BB->succ_end()); |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 935 | MachineBasicBlock *NBB = getNextBlock(FromBBI.BB); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 936 | MachineBasicBlock *FallThrough = FromBBI.HasFallThrough ? NBB : NULL; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 937 | |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 938 | for (unsigned i = 0, e = Succs.size(); i != e; ++i) { |
| 939 | MachineBasicBlock *Succ = Succs[i]; |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 940 | // Fallthrough edge can't be transferred. |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 941 | if (Succ == FallThrough) |
| 942 | continue; |
| 943 | FromBBI.BB->removeSuccessor(Succ); |
| 944 | if (!ToBBI.BB->isSuccessor(Succ)) |
| 945 | ToBBI.BB->addSuccessor(Succ); |
| 946 | } |
| 947 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 948 | // Now FromBBI always fall through to the next block! |
| 949 | if (NBB) |
| 950 | FromBBI.BB->addSuccessor(NBB); |
| 951 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 952 | ToBBI.NonPredSize += FromBBI.NonPredSize; |
| 953 | FromBBI.NonPredSize = 0; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 954 | |
| 955 | ToBBI.ModifyPredicate |= FromBBI.ModifyPredicate; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame^] | 956 | ToBBI.HasFallThrough = FromBBI.HasFallThrough; |
Evan Cheng | 2acdbcc | 2007-06-08 19:17:12 +0000 | [diff] [blame] | 957 | |
| 958 | std::copy(FromBBI.Predicate.begin(), FromBBI.Predicate.end(), |
| 959 | std::back_inserter(ToBBI.Predicate)); |
| 960 | FromBBI.Predicate.clear(); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 961 | } |