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: |
Reid Spencer | a9bf49c | 2007-06-10 00:19:17 +0000 | [diff] [blame^] | 221 | case ICTriangleFRev: { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 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; |
Reid Spencer | a9bf49c | 2007-06-10 00:19:17 +0000 | [diff] [blame^] | 243 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 244 | case ICDiamond: |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 245 | if (DisableDiamond) break; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 246 | DOUT << "Ifcvt (Diamond): BB#" << BBI.BB->getNumber() << " (T:" |
| 247 | << BBI.TrueBB->getNumber() << ",F:" << BBI.FalseBB->getNumber(); |
| 248 | if (BBI.TailBB) |
| 249 | DOUT << "," << BBI.TailBB->getNumber() ; |
| 250 | DOUT << ") "; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 251 | RetVal = IfConvertDiamond(BBI); |
| 252 | DOUT << (RetVal ? "succeeded!" : "failed!") << "\n"; |
| 253 | if (RetVal) NumDiamonds++; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 254 | break; |
| 255 | } |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 256 | Change |= RetVal; |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 257 | |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 258 | if (IfCvtLimit != -1 && (int)NumIfConvBBs >= IfCvtLimit) |
Evan Cheng | edf4896 | 2007-06-08 19:10:51 +0000 | [diff] [blame] | 259 | break; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 260 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 261 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 262 | if (!Change) |
| 263 | break; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 264 | MadeChange |= Change; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 265 | } |
Evan Cheng | 47d2502 | 2007-05-18 01:55:58 +0000 | [diff] [blame] | 266 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 267 | Roots.clear(); |
Evan Cheng | 47d2502 | 2007-05-18 01:55:58 +0000 | [diff] [blame] | 268 | BBAnalysis.clear(); |
| 269 | |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 270 | return MadeChange; |
| 271 | } |
| 272 | |
| 273 | static MachineBasicBlock *findFalseBlock(MachineBasicBlock *BB, |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 274 | MachineBasicBlock *TrueBB) { |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 275 | for (MachineBasicBlock::succ_iterator SI = BB->succ_begin(), |
| 276 | E = BB->succ_end(); SI != E; ++SI) { |
| 277 | MachineBasicBlock *SuccBB = *SI; |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 278 | if (SuccBB != TrueBB) |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 279 | return SuccBB; |
| 280 | } |
| 281 | return NULL; |
| 282 | } |
| 283 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 284 | bool IfConverter::ReverseBranchCondition(BBInfo &BBI) { |
| 285 | if (!TII->ReverseBranchCondition(BBI.BrCond)) { |
| 286 | TII->RemoveBranch(*BBI.BB); |
| 287 | TII->InsertBranch(*BBI.BB, BBI.FalseBB, BBI.TrueBB, BBI.BrCond); |
| 288 | std::swap(BBI.TrueBB, BBI.FalseBB); |
| 289 | return true; |
| 290 | } |
| 291 | return false; |
| 292 | } |
| 293 | |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 294 | /// getNextBlock - Returns the next block in the function blocks ordering. If |
| 295 | /// it is the end, returns NULL. |
| 296 | static inline MachineBasicBlock *getNextBlock(MachineBasicBlock *BB) { |
| 297 | MachineFunction::iterator I = BB; |
| 298 | MachineFunction::iterator E = BB->getParent()->end(); |
| 299 | if (++I == E) |
| 300 | return NULL; |
| 301 | return I; |
| 302 | } |
| 303 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 304 | /// ValidSimple - Returns true if the 'true' block (along with its |
| 305 | /// predecessor) forms a valid simple shape for ifcvt. |
| 306 | bool IfConverter::ValidSimple(BBInfo &TrueBBI) const { |
| 307 | return !blockAlwaysFallThrough(TrueBBI) && |
| 308 | TrueBBI.BrCond.size() == 0 && TrueBBI.BB->pred_size() == 1; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 309 | } |
| 310 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 311 | /// ValidTriangle - Returns true if the 'true' and 'false' blocks (along |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 312 | /// with their common predecessor) forms a valid triangle shape for ifcvt. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 313 | bool IfConverter::ValidTriangle(BBInfo &TrueBBI, BBInfo &FalseBBI, |
| 314 | bool FalseBranch) const { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 315 | if (TrueBBI.BB->pred_size() != 1) |
| 316 | return false; |
| 317 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 318 | MachineBasicBlock *TExit = FalseBranch ? TrueBBI.FalseBB : TrueBBI.TrueBB; |
| 319 | if (!TExit && blockAlwaysFallThrough(TrueBBI)) { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 320 | MachineFunction::iterator I = TrueBBI.BB; |
| 321 | if (++I == TrueBBI.BB->getParent()->end()) |
| 322 | return false; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 323 | TExit = I; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 324 | } |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 325 | return TExit && TExit == FalseBBI.BB; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 326 | } |
| 327 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 328 | /// ValidDiamond - Returns true if the 'true' and 'false' blocks (along |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 329 | /// with their common predecessor) forms a valid diamond shape for ifcvt. |
| 330 | bool IfConverter::ValidDiamond(BBInfo &TrueBBI, BBInfo &FalseBBI) const { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 331 | MachineBasicBlock *TT = TrueBBI.TrueBB; |
| 332 | MachineBasicBlock *FT = FalseBBI.TrueBB; |
| 333 | |
| 334 | if (!TT && blockAlwaysFallThrough(TrueBBI)) |
| 335 | TT = getNextBlock(TrueBBI.BB); |
| 336 | if (!FT && blockAlwaysFallThrough(FalseBBI)) |
| 337 | FT = getNextBlock(FalseBBI.BB); |
| 338 | if (TT != FT) |
| 339 | return false; |
| 340 | if (TT == NULL && (TrueBBI.IsAnalyzable || FalseBBI.IsAnalyzable)) |
| 341 | return false; |
| 342 | // FIXME: Allow false block to have an early exit? |
| 343 | return (TrueBBI.BB->pred_size() == 1 && |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 344 | FalseBBI.BB->pred_size() == 1 && |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 345 | !TrueBBI.FalseBB && !FalseBBI.FalseBB); |
| 346 | } |
| 347 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 348 | /// AnalyzeBlock - Analyze the structure of the sub-CFG starting from |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 349 | /// the specified block. Record its successors and whether it looks like an |
| 350 | /// if-conversion candidate. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 351 | void IfConverter::AnalyzeBlock(MachineBasicBlock *BB) { |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 352 | BBInfo &BBI = BBAnalysis[BB->getNumber()]; |
| 353 | |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 354 | if (BBI.Kind == ICReAnalyze) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 355 | BBI.BrCond.clear(); |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 356 | BBI.TrueBB = BBI.FalseBB = NULL; |
| 357 | } else { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 358 | if (BBI.Kind != ICNotAnalyzed) |
| 359 | return; // Already analyzed. |
| 360 | BBI.BB = BB; |
| 361 | BBI.NonPredSize = std::distance(BB->begin(), BB->end()); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 362 | } |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 363 | |
Evan Cheng | 4bec8ae | 2007-05-25 00:59:01 +0000 | [diff] [blame] | 364 | // Look for 'root' of a simple (non-nested) triangle or diamond. |
| 365 | BBI.Kind = ICNotClassfied; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 366 | BBI.IsAnalyzable = |
| 367 | !TII->AnalyzeBranch(*BB, BBI.TrueBB, BBI.FalseBB, BBI.BrCond); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 368 | BBI.HasFallThrough = BBI.IsAnalyzable && BBI.FalseBB == NULL; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 369 | // Unanalyable or ends with fallthrough or unconditional branch. |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 370 | if (!BBI.IsAnalyzable || BBI.BrCond.size() == 0) |
Evan Cheng | 4bec8ae | 2007-05-25 00:59:01 +0000 | [diff] [blame] | 371 | return; |
Evan Cheng | e004317 | 2007-06-05 20:38:42 +0000 | [diff] [blame] | 372 | // Do not ifcvt if either path is a back edge to the entry block. |
| 373 | if (BBI.TrueBB == BB || BBI.FalseBB == BB) |
| 374 | return; |
Evan Cheng | 4bec8ae | 2007-05-25 00:59:01 +0000 | [diff] [blame] | 375 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 376 | AnalyzeBlock(BBI.TrueBB); |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 377 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 378 | |
| 379 | // No false branch. This BB must end with a conditional branch and a |
| 380 | // fallthrough. |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 381 | if (!BBI.FalseBB) |
| 382 | BBI.FalseBB = findFalseBlock(BB, BBI.TrueBB); |
| 383 | assert(BBI.FalseBB && "Expected to find the fallthrough block!"); |
Evan Cheng | c5d05ef | 2007-05-16 05:11:10 +0000 | [diff] [blame] | 384 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 385 | AnalyzeBlock(BBI.FalseBB); |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 386 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 387 | |
| 388 | // If both paths are dead, then forget about it. |
| 389 | if (TrueBBI.Kind == ICDead && FalseBBI.Kind == ICDead) { |
| 390 | BBI.Kind = ICDead; |
| 391 | return; |
| 392 | } |
| 393 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 394 | // Look for more opportunities to if-convert a triangle. Try to restructure |
| 395 | // the CFG to form a triangle with the 'false' path. |
| 396 | std::vector<MachineOperand> RevCond(BBI.BrCond); |
| 397 | bool CanRevCond = !TII->ReverseBranchCondition(RevCond); |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 398 | |
| 399 | if (CanRevCond && ValidDiamond(TrueBBI, FalseBBI) && |
| 400 | !(TrueBBI.ModifyPredicate && FalseBBI.ModifyPredicate) && |
| 401 | FeasibilityAnalysis(TrueBBI, BBI.BrCond) && |
| 402 | FeasibilityAnalysis(FalseBBI, RevCond)) { |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 403 | // Diamond: |
| 404 | // EBB |
| 405 | // / \_ |
| 406 | // | | |
| 407 | // TBB FBB |
| 408 | // \ / |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 409 | // TailBB |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 410 | // Note TailBB can be empty. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 411 | BBI.Kind = ICDiamond; |
| 412 | TrueBBI.Kind = FalseBBI.Kind = ICChild; |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 413 | BBI.TailBB = TrueBBI.TrueBB; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 414 | } else { |
| 415 | // FIXME: Consider duplicating if BB is small. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 416 | if (ValidTriangle(TrueBBI, FalseBBI) && |
| 417 | FeasibilityAnalysis(TrueBBI, BBI.BrCond, true)) { |
| 418 | // Triangle: |
| 419 | // EBB |
| 420 | // | \_ |
| 421 | // | | |
| 422 | // | TBB |
| 423 | // | / |
| 424 | // FBB |
| 425 | BBI.Kind = ICTriangle; |
| 426 | TrueBBI.Kind = ICChild; |
| 427 | } else if (ValidSimple(TrueBBI) && |
| 428 | FeasibilityAnalysis(TrueBBI, BBI.BrCond)) { |
| 429 | // Simple (split, no rejoin): |
| 430 | // EBB |
| 431 | // | \_ |
| 432 | // | | |
| 433 | // | TBB---> exit |
| 434 | // | |
| 435 | // FBB |
| 436 | BBI.Kind = ICSimple; |
| 437 | TrueBBI.Kind = ICChild; |
| 438 | } else if (CanRevCond) { |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 439 | // Try the other path... |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 440 | if (ValidTriangle(FalseBBI, TrueBBI) && |
| 441 | FeasibilityAnalysis(FalseBBI, RevCond, true)) { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 442 | BBI.Kind = ICTriangleFalse; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 443 | FalseBBI.Kind = ICChild; |
| 444 | } else if (ValidTriangle(FalseBBI, TrueBBI, true) && |
| 445 | FeasibilityAnalysis(FalseBBI, RevCond, true, true)) { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 446 | BBI.Kind = ICTriangleFRev; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 447 | FalseBBI.Kind = ICChild; |
| 448 | } else if (ValidSimple(FalseBBI) && |
| 449 | FeasibilityAnalysis(FalseBBI, RevCond)) { |
| 450 | BBI.Kind = ICSimpleFalse; |
| 451 | FalseBBI.Kind = ICChild; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 452 | } |
| 453 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 454 | } |
| 455 | return; |
| 456 | } |
| 457 | |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 458 | /// FeasibilityAnalysis - Determine if the block is predicable. In most |
| 459 | /// cases, that means all the instructions in the block has M_PREDICABLE flag. |
| 460 | /// Also checks if the block contains any instruction which can clobber a |
| 461 | /// predicate (e.g. condition code register). If so, the block is not |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 462 | /// predicable unless it's the last instruction. |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 463 | bool IfConverter::FeasibilityAnalysis(BBInfo &BBI, |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 464 | std::vector<MachineOperand> &Pred, |
| 465 | bool isTriangle, bool RevBranch) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 466 | // If the block is dead, or it is going to be the entry block of a sub-CFG |
| 467 | // that will be if-converted, then it cannot be predicated. |
| 468 | if (BBI.Kind != ICNotAnalyzed && |
| 469 | BBI.Kind != ICNotClassfied && |
| 470 | BBI.Kind != ICChild) |
| 471 | return false; |
| 472 | |
| 473 | // Check predication threshold. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 474 | if (BBI.NonPredSize == 0 || BBI.NonPredSize > TLI->getIfCvtBlockSizeLimit()) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 475 | return false; |
| 476 | |
| 477 | // If it is already predicated, check if its predicate subsumes the new |
| 478 | // predicate. |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 479 | if (BBI.Predicate.size() && !TII->SubsumesPredicate(BBI.Predicate, Pred)) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 480 | return false; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 481 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 482 | bool SeenPredMod = false; |
| 483 | bool SeenCondBr = false; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 484 | for (MachineBasicBlock::iterator I = BBI.BB->begin(), E = BBI.BB->end(); |
| 485 | I != E; ++I) { |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 486 | const TargetInstrDescriptor *TID = I->getInstrDescriptor(); |
| 487 | if (SeenPredMod) { |
| 488 | // Predicate modification instruction should end the block (except for |
| 489 | // already predicated instructions and end of block branches). |
| 490 | if (!TII->isPredicated(I)) { |
| 491 | // This is the 'true' block of a triangle, i.e. its 'true' block is |
| 492 | // the same as the 'false' block of the entry. So false positive |
| 493 | // is ok. |
| 494 | if (isTriangle && !SeenCondBr && BBI.IsAnalyzable && |
| 495 | (TID->Flags & M_BRANCH_FLAG) != 0 && |
| 496 | (TID->Flags & M_BARRIER_FLAG) == 0) { |
| 497 | // This is the first conditional branch, test predicate subsumsion. |
| 498 | std::vector<MachineOperand> RevPred(Pred); |
| 499 | std::vector<MachineOperand> Cond(BBI.BrCond); |
| 500 | if (RevBranch) { |
| 501 | if (TII->ReverseBranchCondition(Cond)) |
| 502 | return false; |
| 503 | } |
| 504 | if (TII->ReverseBranchCondition(RevPred) || |
| 505 | !TII->SubsumesPredicate(Cond, RevPred)) |
| 506 | return false; |
| 507 | SeenCondBr = true; |
| 508 | continue; // Conditional branches is not predicable. |
| 509 | } |
| 510 | return false; |
| 511 | } |
| 512 | } |
| 513 | |
| 514 | if (TID->Flags & M_CLOBBERS_PRED) { |
| 515 | BBI.ModifyPredicate = true; |
| 516 | SeenPredMod = true; |
| 517 | } |
| 518 | |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 519 | if (!I->isPredicable()) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 520 | return false; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 521 | } |
| 522 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 523 | return true; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 524 | } |
| 525 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 526 | /// AttemptRestructuring - Restructure the sub-CFG rooted in the given block to |
| 527 | /// expose more if-conversion opportunities. e.g. |
| 528 | /// |
| 529 | /// cmp |
| 530 | /// b le BB1 |
| 531 | /// / \____ |
| 532 | /// / | |
| 533 | /// cmp | |
| 534 | /// b eq BB1 | |
| 535 | /// / \____ | |
| 536 | /// / \ | |
| 537 | /// BB1 |
| 538 | /// ==> |
| 539 | /// |
| 540 | /// cmp |
| 541 | /// b eq BB1 |
| 542 | /// / \____ |
| 543 | /// / | |
| 544 | /// cmp | |
| 545 | /// b le BB1 | |
| 546 | /// / \____ | |
| 547 | /// / \ | |
| 548 | /// BB1 |
| 549 | bool IfConverter::AttemptRestructuring(BBInfo &BBI) { |
| 550 | return false; |
| 551 | } |
| 552 | |
| 553 | /// AnalyzeBlocks - Analyze all blocks and find entries for all if-conversion |
| 554 | /// candidates. It returns true if any CFG restructuring is done to expose more |
| 555 | /// if-conversion opportunities. |
| 556 | bool IfConverter::AnalyzeBlocks(MachineFunction &MF, |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 557 | std::vector<BBInfo*> &Candidates) { |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 558 | bool Change = false; |
Evan Cheng | 36489bb | 2007-05-18 19:26:33 +0000 | [diff] [blame] | 559 | std::set<MachineBasicBlock*> Visited; |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 560 | for (unsigned i = 0, e = Roots.size(); i != e; ++i) { |
| 561 | for (idf_ext_iterator<MachineBasicBlock*> I=idf_ext_begin(Roots[i],Visited), |
| 562 | E = idf_ext_end(Roots[i], Visited); I != E; ++I) { |
| 563 | MachineBasicBlock *BB = *I; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 564 | AnalyzeBlock(BB); |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 565 | BBInfo &BBI = BBAnalysis[BB->getNumber()]; |
| 566 | switch (BBI.Kind) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 567 | case ICSimple: |
| 568 | case ICSimpleFalse: |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 569 | case ICTriangle: |
| 570 | case ICDiamond: |
| 571 | Candidates.push_back(&BBI); |
| 572 | break; |
| 573 | default: |
| 574 | Change |= AttemptRestructuring(BBI); |
| 575 | break; |
| 576 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 577 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 578 | } |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 579 | |
Evan Cheng | 5f70218 | 2007-06-01 00:12:12 +0000 | [diff] [blame] | 580 | // Sort to favor more complex ifcvt scheme. |
| 581 | std::stable_sort(Candidates.begin(), Candidates.end(), IfcvtCandidateCmp); |
| 582 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 583 | return Change; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 584 | } |
| 585 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 586 | /// canFallThroughTo - Returns true either if ToBB is the next block after BB or |
| 587 | /// that all the intervening blocks are empty (given BB can fall through to its |
| 588 | /// next block). |
| 589 | static bool canFallThroughTo(MachineBasicBlock *BB, MachineBasicBlock *ToBB) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 590 | MachineFunction::iterator I = BB; |
Evan Cheng | c53ef58 | 2007-06-05 07:05:25 +0000 | [diff] [blame] | 591 | MachineFunction::iterator TI = ToBB; |
| 592 | MachineFunction::iterator E = BB->getParent()->end(); |
| 593 | while (++I != TI) |
| 594 | if (I == E || !I->empty()) |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 595 | return false; |
| 596 | return true; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 597 | } |
| 598 | |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 599 | /// ReTryPreds - Invalidate predecessor BB info so it would be re-analyzed |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 600 | /// to determine if it can be if-converted. |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 601 | void IfConverter::ReTryPreds(MachineBasicBlock *BB) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 602 | for (MachineBasicBlock::pred_iterator PI = BB->pred_begin(), |
| 603 | E = BB->pred_end(); PI != E; ++PI) { |
| 604 | BBInfo &PBBI = BBAnalysis[(*PI)->getNumber()]; |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 605 | if (PBBI.Kind == ICNotClassfied) |
| 606 | PBBI.Kind = ICReAnalyze; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 607 | } |
| 608 | } |
| 609 | |
Evan Cheng | c8ed9ba | 2007-05-29 22:31:16 +0000 | [diff] [blame] | 610 | /// InsertUncondBranch - Inserts an unconditional branch from BB to ToBB. |
| 611 | /// |
| 612 | static void InsertUncondBranch(MachineBasicBlock *BB, MachineBasicBlock *ToBB, |
| 613 | const TargetInstrInfo *TII) { |
| 614 | std::vector<MachineOperand> NoCond; |
| 615 | TII->InsertBranch(*BB, ToBB, NULL, NoCond); |
| 616 | } |
| 617 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 618 | /// RemoveExtraEdges - Remove true / false edges if either / both are no longer |
| 619 | /// successors. |
| 620 | void IfConverter::RemoveExtraEdges(BBInfo &BBI) { |
| 621 | MachineBasicBlock *TBB = NULL, *FBB = NULL; |
| 622 | std::vector<MachineOperand> Cond; |
| 623 | bool isAnalyzable = !TII->AnalyzeBranch(*BBI.BB, TBB, FBB, Cond); |
| 624 | bool CanFallthrough = isAnalyzable && (TBB == NULL || FBB == NULL); |
| 625 | if (BBI.TrueBB && BBI.BB->isSuccessor(BBI.TrueBB)) |
| 626 | if (!(BBI.TrueBB == TBB || BBI.TrueBB == FBB || |
| 627 | (CanFallthrough && getNextBlock(BBI.BB) == BBI.TrueBB))) |
| 628 | BBI.BB->removeSuccessor(BBI.TrueBB); |
| 629 | if (BBI.FalseBB && BBI.BB->isSuccessor(BBI.FalseBB)) |
| 630 | if (!(BBI.FalseBB == TBB || BBI.FalseBB == FBB || |
| 631 | (CanFallthrough && getNextBlock(BBI.BB) == BBI.FalseBB))) |
| 632 | BBI.BB->removeSuccessor(BBI.FalseBB); |
| 633 | } |
| 634 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 635 | /// IfConvertSimple - If convert a simple (split, no rejoin) sub-CFG. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 636 | /// |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 637 | bool IfConverter::IfConvertSimple(BBInfo &BBI) { |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 638 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
| 639 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
| 640 | BBInfo *CvtBBI = &TrueBBI; |
| 641 | BBInfo *NextBBI = &FalseBBI; |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 642 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 643 | std::vector<MachineOperand> Cond(BBI.BrCond); |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 644 | if (BBI.Kind == ICSimpleFalse) { |
Evan Cheng | b5a0690 | 2007-06-01 20:29:21 +0000 | [diff] [blame] | 645 | std::swap(CvtBBI, NextBBI); |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 646 | TII->ReverseBranchCondition(Cond); |
Evan Cheng | b5a0690 | 2007-06-01 20:29:21 +0000 | [diff] [blame] | 647 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 648 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 649 | PredicateBlock(*CvtBBI, Cond); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 650 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 651 | // Merge converted block into entry block. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 652 | BBI.NonPredSize -= TII->RemoveBranch(*BBI.BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 653 | MergeBlocks(BBI, *CvtBBI); |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 654 | |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 655 | bool IterIfcvt = true; |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 656 | if (!canFallThroughTo(BBI.BB, NextBBI->BB)) { |
Evan Cheng | c8ed9ba | 2007-05-29 22:31:16 +0000 | [diff] [blame] | 657 | InsertUncondBranch(BBI.BB, NextBBI->BB, TII); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 658 | BBI.HasFallThrough = false; |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 659 | // Now ifcvt'd block will look like this: |
| 660 | // BB: |
| 661 | // ... |
| 662 | // t, f = cmp |
| 663 | // if t op |
| 664 | // b BBf |
| 665 | // |
| 666 | // We cannot further ifcvt this block because the unconditional branch |
| 667 | // will have to be predicated on the new condition, that will not be |
| 668 | // available if cmp executes. |
| 669 | IterIfcvt = false; |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 670 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 671 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 672 | RemoveExtraEdges(BBI); |
| 673 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 674 | // Update block info. BB can be iteratively if-converted. |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 675 | if (IterIfcvt) |
| 676 | BBI.Kind = ICReAnalyze; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 677 | else |
| 678 | BBI.Kind = ICDead; |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 679 | ReTryPreds(BBI.BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 680 | CvtBBI->Kind = ICDead; |
| 681 | |
| 682 | // FIXME: Must maintain LiveIns. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 683 | return true; |
| 684 | } |
| 685 | |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 686 | /// IfConvertTriangle - If convert a triangle sub-CFG. |
| 687 | /// |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 688 | bool IfConverter::IfConvertTriangle(BBInfo &BBI) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 689 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 690 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
| 691 | BBInfo *CvtBBI = &TrueBBI; |
| 692 | BBInfo *NextBBI = &FalseBBI; |
| 693 | |
| 694 | std::vector<MachineOperand> Cond(BBI.BrCond); |
| 695 | if (BBI.Kind == ICTriangleFalse || BBI.Kind == ICTriangleFRev) { |
| 696 | if (BBI.Kind == ICTriangleFRev) { |
| 697 | ReverseBranchCondition(*NextBBI); |
| 698 | // False BB has been changed, modify its predecessors (except for this |
| 699 | // one) so they don't get ifcvt'ed based on bad intel. |
| 700 | for (MachineBasicBlock::pred_iterator PI = NextBBI->BB->pred_begin(), |
| 701 | E = NextBBI->BB->pred_end(); PI != E; ++PI) { |
| 702 | MachineBasicBlock *PBB = *PI; |
| 703 | if (PBB == BBI.BB) |
| 704 | continue; |
| 705 | BBInfo &PBBI = BBAnalysis[PBB->getNumber()]; |
| 706 | PBBI.Kind = ICReAnalyze; |
| 707 | } |
| 708 | } |
| 709 | std::swap(CvtBBI, NextBBI); |
| 710 | TII->ReverseBranchCondition(Cond); |
| 711 | } |
Evan Cheng | 8c52938 | 2007-06-01 07:41:07 +0000 | [diff] [blame] | 712 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 713 | // Predicate the 'true' block after removing its branch. |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 714 | TrueBBI.NonPredSize -= TII->RemoveBranch(*CvtBBI->BB); |
| 715 | PredicateBlock(*CvtBBI, Cond); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 716 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 717 | // 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] | 718 | bool HasEarlyExit = CvtBBI->FalseBB != NULL; |
Evan Cheng | 8ed680c | 2007-06-05 01:31:40 +0000 | [diff] [blame] | 719 | if (HasEarlyExit) { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 720 | std::vector<MachineOperand> RevCond(CvtBBI->BrCond); |
Evan Cheng | 8c52938 | 2007-06-01 07:41:07 +0000 | [diff] [blame] | 721 | if (TII->ReverseBranchCondition(RevCond)) |
| 722 | assert(false && "Unable to reverse branch condition!"); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 723 | TII->InsertBranch(*CvtBBI->BB, CvtBBI->FalseBB, NULL, RevCond); |
Evan Cheng | 8c52938 | 2007-06-01 07:41:07 +0000 | [diff] [blame] | 724 | } |
| 725 | |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 726 | // Now merge the entry of the triangle with the true block. |
| 727 | BBI.NonPredSize -= TII->RemoveBranch(*BBI.BB); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 728 | MergeBlocks(BBI, *CvtBBI); |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 729 | |
| 730 | // Merge in the 'false' block if the 'false' block has no other |
| 731 | // predecessors. Otherwise, add a unconditional branch from to 'false'. |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 732 | bool FalseBBDead = false; |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 733 | bool IterIfcvt = true; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 734 | bool isFallThrough = canFallThroughTo(BBI.BB, NextBBI->BB); |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 735 | if (!isFallThrough) { |
| 736 | // Only merge them if the true block does not fallthrough to the false |
| 737 | // block. By not merging them, we make it possible to iteratively |
| 738 | // ifcvt the blocks. |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 739 | if (!HasEarlyExit && NextBBI->BB->pred_size() == 1) { |
| 740 | MergeBlocks(BBI, *NextBBI); |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 741 | FalseBBDead = true; |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 742 | } else { |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 743 | InsertUncondBranch(BBI.BB, NextBBI->BB, TII); |
| 744 | BBI.HasFallThrough = false; |
Evan Cheng | f476961 | 2007-06-07 08:13:00 +0000 | [diff] [blame] | 745 | } |
Evan Cheng | ac5f142 | 2007-06-08 09:36:04 +0000 | [diff] [blame] | 746 | // Mixed predicated and unpredicated code. This cannot be iteratively |
| 747 | // predicated. |
| 748 | IterIfcvt = false; |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 749 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 750 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 751 | RemoveExtraEdges(BBI); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 752 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 753 | // Update block info. BB can be iteratively if-converted. |
Evan Cheng | 3d6f60e | 2007-06-06 02:08:52 +0000 | [diff] [blame] | 754 | if (IterIfcvt) |
| 755 | BBI.Kind = ICReAnalyze; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 756 | else |
| 757 | BBI.Kind = ICDead; |
Evan Cheng | 8258210 | 2007-05-30 19:49:19 +0000 | [diff] [blame] | 758 | ReTryPreds(BBI.BB); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 759 | CvtBBI->Kind = ICDead; |
Evan Cheng | f5305f9 | 2007-06-05 00:07:37 +0000 | [diff] [blame] | 760 | if (FalseBBDead) |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 761 | NextBBI->Kind = ICDead; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 762 | |
| 763 | // FIXME: Must maintain LiveIns. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 764 | return true; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 765 | } |
| 766 | |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 767 | /// IfConvertDiamond - If convert a diamond sub-CFG. |
| 768 | /// |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 769 | bool IfConverter::IfConvertDiamond(BBInfo &BBI) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 770 | BBInfo &TrueBBI = BBAnalysis[BBI.TrueBB->getNumber()]; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 771 | BBInfo &FalseBBI = BBAnalysis[BBI.FalseBB->getNumber()]; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 772 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 773 | SmallVector<MachineInstr*, 2> Dups; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 774 | if (!BBI.TailBB) { |
| 775 | // No common merge block. Check if the terminators (e.g. return) are |
| 776 | // the same or predicable. |
| 777 | MachineBasicBlock::iterator TT = BBI.TrueBB->getFirstTerminator(); |
| 778 | MachineBasicBlock::iterator FT = BBI.FalseBB->getFirstTerminator(); |
| 779 | while (TT != BBI.TrueBB->end() && FT != BBI.FalseBB->end()) { |
| 780 | if (TT->isIdenticalTo(FT)) |
| 781 | Dups.push_back(TT); // Will erase these later. |
| 782 | else if (!TT->isPredicable() && !FT->isPredicable()) |
| 783 | return false; // Can't if-convert. Abort! |
| 784 | ++TT; |
| 785 | ++FT; |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 786 | } |
Evan Cheng | cf6cc11 | 2007-05-18 18:14:37 +0000 | [diff] [blame] | 787 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 788 | // One of the two pathes have more terminators, make sure they are |
| 789 | // all predicable. |
| 790 | while (TT != BBI.TrueBB->end()) { |
| 791 | if (!TT->isPredicable()) { |
| 792 | return false; // Can't if-convert. Abort! |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 793 | } |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 794 | ++TT; |
| 795 | } |
| 796 | while (FT != BBI.FalseBB->end()) { |
| 797 | if (!FT->isPredicable()) { |
| 798 | return false; // Can't if-convert. Abort! |
| 799 | } |
| 800 | ++FT; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 801 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 802 | } |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 803 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 804 | // Remove the duplicated instructions from the 'true' block. |
| 805 | for (unsigned i = 0, e = Dups.size(); i != e; ++i) { |
| 806 | Dups[i]->eraseFromParent(); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 807 | --TrueBBI.NonPredSize; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 808 | } |
| 809 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 810 | // Merge the 'true' and 'false' blocks by copying the instructions |
| 811 | // from the 'false' block to the 'true' block. That is, unless the true |
| 812 | // block would clobber the predicate, in that case, do the opposite. |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 813 | BBInfo *BBI1 = &TrueBBI; |
| 814 | BBInfo *BBI2 = &FalseBBI; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 815 | std::vector<MachineOperand> RevCond(BBI.BrCond); |
| 816 | TII->ReverseBranchCondition(RevCond); |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 817 | std::vector<MachineOperand> *Cond1 = &BBI.BrCond; |
| 818 | std::vector<MachineOperand> *Cond2 = &RevCond; |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 819 | // Check the 'true' and 'false' blocks if either isn't ended with a branch. |
| 820 | // Either the block fallthrough to another block or it ends with a |
| 821 | // return. If it's the former, add a branch to its successor. |
| 822 | bool NeedBr1 = !BBI1->TrueBB && BBI1->BB->succ_size(); |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 823 | bool NeedBr2 = !BBI2->TrueBB && BBI2->BB->succ_size(); |
| 824 | |
| 825 | if ((TrueBBI.ModifyPredicate && !FalseBBI.ModifyPredicate) || |
| 826 | (!TrueBBI.ModifyPredicate && !FalseBBI.ModifyPredicate && |
| 827 | NeedBr1 && !NeedBr2)) { |
| 828 | std::swap(BBI1, BBI2); |
| 829 | std::swap(Cond1, Cond2); |
| 830 | std::swap(NeedBr1, NeedBr2); |
| 831 | } |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 832 | |
| 833 | // Predicate the 'true' block after removing its branch. |
| 834 | BBI1->NonPredSize -= TII->RemoveBranch(*BBI1->BB); |
| 835 | PredicateBlock(*BBI1, *Cond1); |
| 836 | |
| 837 | // Add an early exit branch if needed. |
| 838 | if (NeedBr1) |
| 839 | TII->InsertBranch(*BBI1->BB, *BBI1->BB->succ_begin(), NULL, *Cond1); |
| 840 | |
| 841 | // Predicate the 'false' block. |
| 842 | PredicateBlock(*BBI2, *Cond2, true); |
| 843 | |
| 844 | // Add an unconditional branch from 'false' to to 'false' successor if it |
| 845 | // will not be the fallthrough block. |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 846 | if (NeedBr2 && !NeedBr1) { |
| 847 | // If BBI2 isn't going to be merged in, then the existing fallthrough |
| 848 | // or branch is fine. |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 849 | if (!canFallThroughTo(BBI.BB, *BBI2->BB->succ_begin())) { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 850 | InsertUncondBranch(BBI2->BB, *BBI2->BB->succ_begin(), TII); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 851 | BBI2->HasFallThrough = false; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 852 | } |
| 853 | } |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 854 | |
| 855 | // Keep them as two separate blocks if there is an early exit. |
| 856 | if (!NeedBr1) |
| 857 | MergeBlocks(*BBI1, *BBI2); |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 858 | |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 859 | // Remove the conditional branch from entry to the blocks. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 860 | BBI.NonPredSize -= TII->RemoveBranch(*BBI.BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 861 | |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 862 | // Merge the combined block into the entry of the diamond. |
| 863 | MergeBlocks(BBI, *BBI1); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 864 | |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 865 | // 'True' and 'false' aren't combined, see if we need to add a unconditional |
| 866 | // branch to the 'false' block. |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 867 | if (NeedBr1 && !canFallThroughTo(BBI.BB, BBI2->BB)) { |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 868 | InsertUncondBranch(BBI.BB, BBI2->BB, TII); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 869 | BBI1->HasFallThrough = false; |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 870 | } |
Evan Cheng | e705213 | 2007-06-06 00:57:55 +0000 | [diff] [blame] | 871 | |
Evan Cheng | 58fbb9f | 2007-05-29 23:37:20 +0000 | [diff] [blame] | 872 | // If the if-converted block fallthrough or unconditionally branch into the |
| 873 | // tail block, and the tail block does not have other predecessors, then |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 874 | // fold the tail block in as well. |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 875 | BBInfo *CvtBBI = NeedBr1 ? BBI2 : &BBI; |
Evan Cheng | 58fbb9f | 2007-05-29 23:37:20 +0000 | [diff] [blame] | 876 | if (BBI.TailBB && |
Evan Cheng | f15d44c | 2007-05-31 20:53:33 +0000 | [diff] [blame] | 877 | BBI.TailBB->pred_size() == 1 && CvtBBI->BB->succ_size() == 1) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 878 | CvtBBI->NonPredSize -= TII->RemoveBranch(*CvtBBI->BB); |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 879 | BBInfo TailBBI = BBAnalysis[BBI.TailBB->getNumber()]; |
| 880 | MergeBlocks(*CvtBBI, TailBBI); |
| 881 | TailBBI.Kind = ICDead; |
| 882 | } |
| 883 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 884 | RemoveExtraEdges(BBI); |
| 885 | |
Evan Cheng | 993fc95 | 2007-06-05 23:46:14 +0000 | [diff] [blame] | 886 | // Update block info. |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 887 | BBI.Kind = ICDead; |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 888 | TrueBBI.Kind = ICDead; |
| 889 | FalseBBI.Kind = ICDead; |
| 890 | |
| 891 | // FIXME: Must maintain LiveIns. |
Evan Cheng | a6b4f43 | 2007-05-21 22:22:58 +0000 | [diff] [blame] | 892 | return true; |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 893 | } |
| 894 | |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 895 | /// PredicateBlock - Predicate every instruction in the block with the specified |
| 896 | /// condition. If IgnoreTerm is true, skip over all terminator instructions. |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 897 | void IfConverter::PredicateBlock(BBInfo &BBI, |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 898 | std::vector<MachineOperand> &Cond, |
| 899 | bool IgnoreTerm) { |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 900 | for (MachineBasicBlock::iterator I = BBI.BB->begin(), E = BBI.BB->end(); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 901 | I != E; ++I) { |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 902 | if (IgnoreTerm && TII->isTerminatorInstr(I->getOpcode())) |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 903 | continue; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 904 | if (TII->isPredicated(I)) |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 905 | continue; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 906 | if (!TII->PredicateInstruction(I, Cond)) { |
Evan Cheng | fe57a7e | 2007-06-01 00:55:26 +0000 | [diff] [blame] | 907 | cerr << "Unable to predicate " << *I << "!\n"; |
Evan Cheng | d6ddc30 | 2007-05-16 21:54:37 +0000 | [diff] [blame] | 908 | abort(); |
| 909 | } |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 910 | } |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 911 | |
| 912 | BBI.NonPredSize = 0; |
Evan Cheng | 2acdbcc | 2007-06-08 19:17:12 +0000 | [diff] [blame] | 913 | std::copy(Cond.begin(), Cond.end(), std::back_inserter(BBI.Predicate)); |
| 914 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 915 | NumIfConvBBs++; |
| 916 | } |
| 917 | |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 918 | /// MergeBlocks - Move all instructions from FromBB to the end of ToBB. |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 919 | /// |
Evan Cheng | 86cbfea | 2007-05-18 00:20:58 +0000 | [diff] [blame] | 920 | void IfConverter::MergeBlocks(BBInfo &ToBBI, BBInfo &FromBBI) { |
| 921 | ToBBI.BB->splice(ToBBI.BB->end(), |
| 922 | FromBBI.BB, FromBBI.BB->begin(), FromBBI.BB->end()); |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 923 | |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 924 | // Redirect all branches to FromBB to ToBB. |
Evan Cheng | a1a9f40 | 2007-06-05 22:03:53 +0000 | [diff] [blame] | 925 | std::vector<MachineBasicBlock *> Preds(FromBBI.BB->pred_begin(), |
| 926 | FromBBI.BB->pred_end()); |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 927 | for (unsigned i = 0, e = Preds.size(); i != e; ++i) { |
| 928 | MachineBasicBlock *Pred = Preds[i]; |
| 929 | if (Pred == ToBBI.BB) |
| 930 | continue; |
| 931 | Pred->ReplaceUsesOfBlockWith(FromBBI.BB, ToBBI.BB); |
| 932 | } |
| 933 | |
| 934 | std::vector<MachineBasicBlock *> Succs(FromBBI.BB->succ_begin(), |
| 935 | FromBBI.BB->succ_end()); |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 936 | MachineBasicBlock *NBB = getNextBlock(FromBBI.BB); |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 937 | MachineBasicBlock *FallThrough = FromBBI.HasFallThrough ? NBB : NULL; |
Evan Cheng | b6665f6 | 2007-06-04 06:47:22 +0000 | [diff] [blame] | 938 | |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 939 | for (unsigned i = 0, e = Succs.size(); i != e; ++i) { |
| 940 | MachineBasicBlock *Succ = Succs[i]; |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 941 | // Fallthrough edge can't be transferred. |
Evan Cheng | d4de6d9 | 2007-06-07 02:12:15 +0000 | [diff] [blame] | 942 | if (Succ == FallThrough) |
| 943 | continue; |
| 944 | FromBBI.BB->removeSuccessor(Succ); |
| 945 | if (!ToBBI.BB->isSuccessor(Succ)) |
| 946 | ToBBI.BB->addSuccessor(Succ); |
| 947 | } |
| 948 | |
Evan Cheng | 7e75ba8 | 2007-06-08 22:01:07 +0000 | [diff] [blame] | 949 | // Now FromBBI always fall through to the next block! |
| 950 | if (NBB) |
| 951 | FromBBI.BB->addSuccessor(NBB); |
| 952 | |
Evan Cheng | a13aa95 | 2007-05-23 07:23:16 +0000 | [diff] [blame] | 953 | ToBBI.NonPredSize += FromBBI.NonPredSize; |
| 954 | FromBBI.NonPredSize = 0; |
Evan Cheng | 7a65547 | 2007-06-06 10:16:17 +0000 | [diff] [blame] | 955 | |
| 956 | ToBBI.ModifyPredicate |= FromBBI.ModifyPredicate; |
Evan Cheng | 1c9f91d | 2007-06-09 01:03:43 +0000 | [diff] [blame] | 957 | ToBBI.HasFallThrough = FromBBI.HasFallThrough; |
Evan Cheng | 2acdbcc | 2007-06-08 19:17:12 +0000 | [diff] [blame] | 958 | |
| 959 | std::copy(FromBBI.Predicate.begin(), FromBBI.Predicate.end(), |
| 960 | std::back_inserter(ToBBI.Predicate)); |
| 961 | FromBBI.Predicate.clear(); |
Evan Cheng | 4e65485 | 2007-05-16 02:00:57 +0000 | [diff] [blame] | 962 | } |