blob: e1bc590c5c9ae8ce35372587d8e3698430f320c8 [file] [log] [blame]
Sanjay Patel6fd43912017-09-09 13:38:18 +00001//===- DivRemPairs.cpp - Hoist/decompose division and remainder -*- C++ -*-===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This pass hoists and/or decomposes integer division and remainder
11// instructions to enable CFG improvements and better codegen.
12//
13//===----------------------------------------------------------------------===//
14
15#include "llvm/Transforms/Scalar/DivRemPairs.h"
Geoff Berry2af5f3c2018-04-25 02:17:56 +000016#include "llvm/ADT/DenseMap.h"
17#include "llvm/ADT/MapVector.h"
Sanjay Patel6fd43912017-09-09 13:38:18 +000018#include "llvm/ADT/Statistic.h"
19#include "llvm/Analysis/GlobalsModRef.h"
20#include "llvm/Analysis/TargetTransformInfo.h"
21#include "llvm/IR/Dominators.h"
22#include "llvm/IR/Function.h"
23#include "llvm/Pass.h"
24#include "llvm/Transforms/Scalar.h"
25#include "llvm/Transforms/Utils/BypassSlowDivision.h"
26using namespace llvm;
27
28#define DEBUG_TYPE "div-rem-pairs"
29STATISTIC(NumPairs, "Number of div/rem pairs");
30STATISTIC(NumHoisted, "Number of instructions hoisted");
31STATISTIC(NumDecomposed, "Number of instructions decomposed");
32
33/// Find matching pairs of integer div/rem ops (they have the same numerator,
34/// denominator, and signedness). If they exist in different basic blocks, bring
35/// them together by hoisting or replace the common division operation that is
36/// implicit in the remainder:
37/// X % Y <--> X - ((X / Y) * Y).
38///
39/// We can largely ignore the normal safety and cost constraints on speculation
40/// of these ops when we find a matching pair. This is because we are already
41/// guaranteed that any exceptions and most cost are already incurred by the
42/// first member of the pair.
43///
44/// Note: This transform could be an oddball enhancement to EarlyCSE, GVN, or
45/// SimplifyCFG, but it's split off on its own because it's different enough
46/// that it doesn't quite match the stated objectives of those passes.
47static bool optimizeDivRem(Function &F, const TargetTransformInfo &TTI,
48 const DominatorTree &DT) {
49 bool Changed = false;
50
51 // Insert all divide and remainder instructions into maps keyed by their
52 // operands and opcode (signed or unsigned).
Geoff Berry2af5f3c2018-04-25 02:17:56 +000053 DenseMap<DivRemMapKey, Instruction *> DivMap;
54 // Use a MapVector for RemMap so that instructions are moved/inserted in a
55 // deterministic order.
56 MapVector<DivRemMapKey, Instruction *> RemMap;
Sanjay Patel6fd43912017-09-09 13:38:18 +000057 for (auto &BB : F) {
58 for (auto &I : BB) {
59 if (I.getOpcode() == Instruction::SDiv)
60 DivMap[DivRemMapKey(true, I.getOperand(0), I.getOperand(1))] = &I;
61 else if (I.getOpcode() == Instruction::UDiv)
62 DivMap[DivRemMapKey(false, I.getOperand(0), I.getOperand(1))] = &I;
63 else if (I.getOpcode() == Instruction::SRem)
64 RemMap[DivRemMapKey(true, I.getOperand(0), I.getOperand(1))] = &I;
65 else if (I.getOpcode() == Instruction::URem)
66 RemMap[DivRemMapKey(false, I.getOperand(0), I.getOperand(1))] = &I;
67 }
68 }
69
70 // We can iterate over either map because we are only looking for matched
71 // pairs. Choose remainders for efficiency because they are usually even more
72 // rare than division.
73 for (auto &RemPair : RemMap) {
74 // Find the matching division instruction from the division map.
Geoff Berry2af5f3c2018-04-25 02:17:56 +000075 Instruction *DivInst = DivMap[RemPair.first];
Sanjay Patel6fd43912017-09-09 13:38:18 +000076 if (!DivInst)
77 continue;
78
79 // We have a matching pair of div/rem instructions. If one dominates the
80 // other, hoist and/or replace one.
81 NumPairs++;
Geoff Berry2af5f3c2018-04-25 02:17:56 +000082 Instruction *RemInst = RemPair.second;
Sanjay Patel6fd43912017-09-09 13:38:18 +000083 bool IsSigned = DivInst->getOpcode() == Instruction::SDiv;
84 bool HasDivRemOp = TTI.hasDivRemOp(DivInst->getType(), IsSigned);
85
86 // If the target supports div+rem and the instructions are in the same block
87 // already, there's nothing to do. The backend should handle this. If the
88 // target does not support div+rem, then we will decompose the rem.
89 if (HasDivRemOp && RemInst->getParent() == DivInst->getParent())
90 continue;
91
92 bool DivDominates = DT.dominates(DivInst, RemInst);
93 if (!DivDominates && !DT.dominates(RemInst, DivInst))
94 continue;
95
96 if (HasDivRemOp) {
97 // The target has a single div/rem operation. Hoist the lower instruction
98 // to make the matched pair visible to the backend.
99 if (DivDominates)
100 RemInst->moveAfter(DivInst);
101 else
102 DivInst->moveAfter(RemInst);
103 NumHoisted++;
104 } else {
105 // The target does not have a single div/rem operation. Decompose the
106 // remainder calculation as:
107 // X % Y --> X - ((X / Y) * Y).
108 Value *X = RemInst->getOperand(0);
109 Value *Y = RemInst->getOperand(1);
110 Instruction *Mul = BinaryOperator::CreateMul(DivInst, Y);
111 Instruction *Sub = BinaryOperator::CreateSub(X, Mul);
112
113 // If the remainder dominates, then hoist the division up to that block:
114 //
115 // bb1:
116 // %rem = srem %x, %y
117 // bb2:
118 // %div = sdiv %x, %y
119 // -->
120 // bb1:
121 // %div = sdiv %x, %y
122 // %mul = mul %div, %y
123 // %rem = sub %x, %mul
124 //
125 // If the division dominates, it's already in the right place. The mul+sub
126 // will be in a different block because we don't assume that they are
127 // cheap to speculatively execute:
128 //
129 // bb1:
130 // %div = sdiv %x, %y
131 // bb2:
132 // %rem = srem %x, %y
133 // -->
134 // bb1:
135 // %div = sdiv %x, %y
136 // bb2:
137 // %mul = mul %div, %y
138 // %rem = sub %x, %mul
139 //
140 // If the div and rem are in the same block, we do the same transform,
141 // but any code movement would be within the same block.
142
143 if (!DivDominates)
144 DivInst->moveBefore(RemInst);
145 Mul->insertAfter(RemInst);
146 Sub->insertAfter(Mul);
147
148 // Now kill the explicit remainder. We have replaced it with:
149 // (sub X, (mul (div X, Y), Y)
150 RemInst->replaceAllUsesWith(Sub);
151 RemInst->eraseFromParent();
152 NumDecomposed++;
153 }
154 Changed = true;
155 }
156
157 return Changed;
158}
159
160// Pass manager boilerplate below here.
161
162namespace {
163struct DivRemPairsLegacyPass : public FunctionPass {
164 static char ID;
165 DivRemPairsLegacyPass() : FunctionPass(ID) {
166 initializeDivRemPairsLegacyPassPass(*PassRegistry::getPassRegistry());
167 }
168
169 void getAnalysisUsage(AnalysisUsage &AU) const override {
170 AU.addRequired<DominatorTreeWrapperPass>();
171 AU.addRequired<TargetTransformInfoWrapperPass>();
172 AU.setPreservesCFG();
173 AU.addPreserved<DominatorTreeWrapperPass>();
174 AU.addPreserved<GlobalsAAWrapperPass>();
175 FunctionPass::getAnalysisUsage(AU);
176 }
177
178 bool runOnFunction(Function &F) override {
179 if (skipFunction(F))
180 return false;
181 auto &TTI = getAnalysis<TargetTransformInfoWrapperPass>().getTTI(F);
182 auto &DT = getAnalysis<DominatorTreeWrapperPass>().getDomTree();
183 return optimizeDivRem(F, TTI, DT);
184 }
185};
186}
187
188char DivRemPairsLegacyPass::ID = 0;
189INITIALIZE_PASS_BEGIN(DivRemPairsLegacyPass, "div-rem-pairs",
190 "Hoist/decompose integer division and remainder", false,
191 false)
192INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)
193INITIALIZE_PASS_END(DivRemPairsLegacyPass, "div-rem-pairs",
194 "Hoist/decompose integer division and remainder", false,
195 false)
196FunctionPass *llvm::createDivRemPairsPass() {
197 return new DivRemPairsLegacyPass();
198}
199
200PreservedAnalyses DivRemPairsPass::run(Function &F,
201 FunctionAnalysisManager &FAM) {
202 TargetTransformInfo &TTI = FAM.getResult<TargetIRAnalysis>(F);
203 DominatorTree &DT = FAM.getResult<DominatorTreeAnalysis>(F);
204 if (!optimizeDivRem(F, TTI, DT))
205 return PreservedAnalyses::all();
206 // TODO: This pass just hoists/replaces math ops - all analyses are preserved?
207 PreservedAnalyses PA;
208 PA.preserveSet<CFGAnalyses>();
209 PA.preserve<GlobalsAA>();
210 return PA;
211}