• Home
  • Line#
  • Scopes#
  • Navigate#
  • Raw
  • Download
1 // Copyright (c) 2018 Google LLC
2 //
3 // Licensed under the Apache License, Version 2.0 (the "License");
4 // you may not use this file except in compliance with the License.
5 // You may obtain a copy of the License at
6 //
7 //     http://www.apache.org/licenses/LICENSE-2.0
8 //
9 // Unless required by applicable law or agreed to in writing, software
10 // distributed under the License is distributed on an "AS IS" BASIS,
11 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12 // See the License for the specific language governing permissions and
13 // limitations under the License.
14 
15 #include "source/opt/if_conversion.h"
16 
17 #include <memory>
18 #include <vector>
19 
20 #include "source/opt/value_number_table.h"
21 
22 namespace spvtools {
23 namespace opt {
24 
Process()25 Pass::Status IfConversion::Process() {
26   if (!context()->get_feature_mgr()->HasCapability(SpvCapabilityShader)) {
27     return Status::SuccessWithoutChange;
28   }
29 
30   const ValueNumberTable& vn_table = *context()->GetValueNumberTable();
31   bool modified = false;
32   std::vector<Instruction*> to_kill;
33   for (auto& func : *get_module()) {
34     DominatorAnalysis* dominators = context()->GetDominatorAnalysis(&func);
35     for (auto& block : func) {
36       // Check if it is possible for |block| to have phis that can be
37       // transformed.
38       BasicBlock* common = nullptr;
39       if (!CheckBlock(&block, dominators, &common)) continue;
40 
41       // Get an insertion point.
42       auto iter = block.begin();
43       while (iter != block.end() && iter->opcode() == SpvOpPhi) {
44         ++iter;
45       }
46 
47       InstructionBuilder builder(
48           context(), &*iter,
49           IRContext::kAnalysisDefUse | IRContext::kAnalysisInstrToBlockMapping);
50       block.ForEachPhiInst([this, &builder, &modified, &common, &to_kill,
51                             dominators, &block, &vn_table](Instruction* phi) {
52         // This phi is not compatible, but subsequent phis might be.
53         if (!CheckType(phi->type_id())) return;
54 
55         // We cannot transform cases where the phi is used by another phi in the
56         // same block due to instruction ordering restrictions.
57         // TODO(alan-baker): If all inappropriate uses could also be
58         // transformed, we could still remove this phi.
59         if (!CheckPhiUsers(phi, &block)) return;
60 
61         // Identify the incoming values associated with the true and false
62         // branches. If |then_block| dominates |inc0| or if the true edge
63         // branches straight to this block and |common| is |inc0|, then |inc0|
64         // is on the true branch. Otherwise the |inc1| is on the true branch.
65         BasicBlock* inc0 = GetIncomingBlock(phi, 0u);
66         Instruction* branch = common->terminator();
67         uint32_t condition = branch->GetSingleWordInOperand(0u);
68         BasicBlock* then_block = GetBlock(branch->GetSingleWordInOperand(1u));
69         Instruction* true_value = nullptr;
70         Instruction* false_value = nullptr;
71         if ((then_block == &block && inc0 == common) ||
72             dominators->Dominates(then_block, inc0)) {
73           true_value = GetIncomingValue(phi, 0u);
74           false_value = GetIncomingValue(phi, 1u);
75         } else {
76           true_value = GetIncomingValue(phi, 1u);
77           false_value = GetIncomingValue(phi, 0u);
78         }
79 
80         BasicBlock* true_def_block = context()->get_instr_block(true_value);
81         BasicBlock* false_def_block = context()->get_instr_block(false_value);
82 
83         uint32_t true_vn = vn_table.GetValueNumber(true_value);
84         uint32_t false_vn = vn_table.GetValueNumber(false_value);
85         if (true_vn != 0 && true_vn == false_vn) {
86           Instruction* inst_to_use = nullptr;
87 
88           // Try to pick an instruction that is not in a side node.  If we can't
89           // pick either the true for false branch as long as they can be
90           // legally moved.
91           if (!true_def_block ||
92               dominators->Dominates(true_def_block, &block)) {
93             inst_to_use = true_value;
94           } else if (!false_def_block ||
95                      dominators->Dominates(false_def_block, &block)) {
96             inst_to_use = false_value;
97           } else if (CanHoistInstruction(true_value, common, dominators)) {
98             inst_to_use = true_value;
99           } else if (CanHoistInstruction(false_value, common, dominators)) {
100             inst_to_use = false_value;
101           }
102 
103           if (inst_to_use != nullptr) {
104             modified = true;
105             HoistInstruction(inst_to_use, common, dominators);
106             context()->KillNamesAndDecorates(phi);
107             context()->ReplaceAllUsesWith(phi->result_id(),
108                                           inst_to_use->result_id());
109           }
110           return;
111         }
112 
113         // If either incoming value is defined in a block that does not dominate
114         // this phi, then we cannot eliminate the phi with a select.
115         // TODO(alan-baker): Perform code motion where it makes sense to enable
116         // the transform in this case.
117         if (true_def_block && !dominators->Dominates(true_def_block, &block))
118           return;
119 
120         if (false_def_block && !dominators->Dominates(false_def_block, &block))
121           return;
122 
123         analysis::Type* data_ty =
124             context()->get_type_mgr()->GetType(true_value->type_id());
125         if (analysis::Vector* vec_data_ty = data_ty->AsVector()) {
126           condition = SplatCondition(vec_data_ty, condition, &builder);
127         }
128 
129         Instruction* select = builder.AddSelect(phi->type_id(), condition,
130                                                 true_value->result_id(),
131                                                 false_value->result_id());
132         select->UpdateDebugInfoFrom(phi);
133         context()->ReplaceAllUsesWith(phi->result_id(), select->result_id());
134         to_kill.push_back(phi);
135         modified = true;
136 
137         return;
138       });
139     }
140   }
141 
142   for (auto inst : to_kill) {
143     context()->KillInst(inst);
144   }
145 
146   return modified ? Status::SuccessWithChange : Status::SuccessWithoutChange;
147 }
148 
CheckBlock(BasicBlock * block,DominatorAnalysis * dominators,BasicBlock ** common)149 bool IfConversion::CheckBlock(BasicBlock* block, DominatorAnalysis* dominators,
150                               BasicBlock** common) {
151   const std::vector<uint32_t>& preds = cfg()->preds(block->id());
152 
153   // TODO(alan-baker): Extend to more than two predecessors
154   if (preds.size() != 2) return false;
155 
156   BasicBlock* inc0 = context()->get_instr_block(preds[0]);
157   if (dominators->Dominates(block, inc0)) return false;
158 
159   BasicBlock* inc1 = context()->get_instr_block(preds[1]);
160   if (dominators->Dominates(block, inc1)) return false;
161 
162   // All phis will have the same common dominator, so cache the result
163   // for this block. If there is no common dominator, then we cannot transform
164   // any phi in this basic block.
165   *common = dominators->CommonDominator(inc0, inc1);
166   if (!*common || cfg()->IsPseudoEntryBlock(*common)) return false;
167   Instruction* branch = (*common)->terminator();
168   if (branch->opcode() != SpvOpBranchConditional) return false;
169   auto merge = (*common)->GetMergeInst();
170   if (!merge || merge->opcode() != SpvOpSelectionMerge) return false;
171   if ((*common)->MergeBlockIdIfAny() != block->id()) return false;
172 
173   return true;
174 }
175 
CheckPhiUsers(Instruction * phi,BasicBlock * block)176 bool IfConversion::CheckPhiUsers(Instruction* phi, BasicBlock* block) {
177   return get_def_use_mgr()->WhileEachUser(phi, [block,
178                                                 this](Instruction* user) {
179     if (user->opcode() == SpvOpPhi && context()->get_instr_block(user) == block)
180       return false;
181     return true;
182   });
183 }
184 
SplatCondition(analysis::Vector * vec_data_ty,uint32_t cond,InstructionBuilder * builder)185 uint32_t IfConversion::SplatCondition(analysis::Vector* vec_data_ty,
186                                       uint32_t cond,
187                                       InstructionBuilder* builder) {
188   // If the data inputs to OpSelect are vectors, the condition for
189   // OpSelect must be a boolean vector with the same number of
190   // components. So splat the condition for the branch into a vector
191   // type.
192   analysis::Bool bool_ty;
193   analysis::Vector bool_vec_ty(&bool_ty, vec_data_ty->element_count());
194   uint32_t bool_vec_id =
195       context()->get_type_mgr()->GetTypeInstruction(&bool_vec_ty);
196   std::vector<uint32_t> ids(vec_data_ty->element_count(), cond);
197   return builder->AddCompositeConstruct(bool_vec_id, ids)->result_id();
198 }
199 
CheckType(uint32_t id)200 bool IfConversion::CheckType(uint32_t id) {
201   Instruction* type = get_def_use_mgr()->GetDef(id);
202   SpvOp op = type->opcode();
203   if (spvOpcodeIsScalarType(op) || op == SpvOpTypePointer ||
204       op == SpvOpTypeVector)
205     return true;
206   return false;
207 }
208 
GetBlock(uint32_t id)209 BasicBlock* IfConversion::GetBlock(uint32_t id) {
210   return context()->get_instr_block(get_def_use_mgr()->GetDef(id));
211 }
212 
GetIncomingBlock(Instruction * phi,uint32_t predecessor)213 BasicBlock* IfConversion::GetIncomingBlock(Instruction* phi,
214                                            uint32_t predecessor) {
215   uint32_t in_index = 2 * predecessor + 1;
216   return GetBlock(phi->GetSingleWordInOperand(in_index));
217 }
218 
GetIncomingValue(Instruction * phi,uint32_t predecessor)219 Instruction* IfConversion::GetIncomingValue(Instruction* phi,
220                                             uint32_t predecessor) {
221   uint32_t in_index = 2 * predecessor;
222   return get_def_use_mgr()->GetDef(phi->GetSingleWordInOperand(in_index));
223 }
224 
HoistInstruction(Instruction * inst,BasicBlock * target_block,DominatorAnalysis * dominators)225 void IfConversion::HoistInstruction(Instruction* inst, BasicBlock* target_block,
226                                     DominatorAnalysis* dominators) {
227   BasicBlock* inst_block = context()->get_instr_block(inst);
228   if (!inst_block) {
229     // This is in the header, and dominates everything.
230     return;
231   }
232 
233   if (dominators->Dominates(inst_block, target_block)) {
234     // Already in position.  No work to do.
235     return;
236   }
237 
238   assert(inst->IsOpcodeCodeMotionSafe() &&
239          "Trying to move an instruction that is not safe to move.");
240 
241   // First hoist all instructions it depends on.
242   analysis::DefUseManager* def_use_mgr = context()->get_def_use_mgr();
243   inst->ForEachInId(
244       [this, target_block, def_use_mgr, dominators](uint32_t* id) {
245         Instruction* operand_inst = def_use_mgr->GetDef(*id);
246         HoistInstruction(operand_inst, target_block, dominators);
247       });
248 
249   Instruction* insertion_pos = target_block->terminator();
250   if ((insertion_pos)->PreviousNode()->opcode() == SpvOpSelectionMerge) {
251     insertion_pos = insertion_pos->PreviousNode();
252   }
253   inst->RemoveFromList();
254   insertion_pos->InsertBefore(std::unique_ptr<Instruction>(inst));
255   context()->set_instr_block(inst, target_block);
256 }
257 
CanHoistInstruction(Instruction * inst,BasicBlock * target_block,DominatorAnalysis * dominators)258 bool IfConversion::CanHoistInstruction(Instruction* inst,
259                                        BasicBlock* target_block,
260                                        DominatorAnalysis* dominators) {
261   BasicBlock* inst_block = context()->get_instr_block(inst);
262   if (!inst_block) {
263     // This is in the header, and dominates everything.
264     return true;
265   }
266 
267   if (dominators->Dominates(inst_block, target_block)) {
268     // Already in position.  No work to do.
269     return true;
270   }
271 
272   if (!inst->IsOpcodeCodeMotionSafe()) {
273     return false;
274   }
275 
276   // Check all instruction |inst| depends on.
277   analysis::DefUseManager* def_use_mgr = context()->get_def_use_mgr();
278   return inst->WhileEachInId(
279       [this, target_block, def_use_mgr, dominators](uint32_t* id) {
280         Instruction* operand_inst = def_use_mgr->GetDef(*id);
281         return CanHoistInstruction(operand_inst, target_block, dominators);
282       });
283 }
284 
285 }  // namespace opt
286 }  // namespace spvtools
287