blob: 17db2780ced967fd65082cde923ee67364df7bfc [file] [log] [blame]
Aart Bik281c6812016-08-26 11:31:48 -07001/*
2 * Copyright (C) 2016 The Android Open Source Project
3 *
4 * Licensed under the Apache License, Version 2.0 (the "License");
5 * you may not use this file except in compliance with the License.
6 * You may obtain a copy of the License at
7 *
8 * http://www.apache.org/licenses/LICENSE-2.0
9 *
10 * Unless required by applicable law or agreed to in writing, software
11 * distributed under the License is distributed on an "AS IS" BASIS,
12 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13 * See the License for the specific language governing permissions and
14 * limitations under the License.
15 */
16
Vladimir Markoe2727152019-10-10 10:46:42 +010017#include "base/macros.h"
Aart Bik281c6812016-08-26 11:31:48 -070018#include "loop_optimization.h"
19#include "optimizing_unit_test.h"
20
Vladimir Markoe2727152019-10-10 10:46:42 +010021namespace art HIDDEN {
Aart Bik281c6812016-08-26 11:31:48 -070022
23/**
24 * Fixture class for the loop optimization tests. These unit tests focus
25 * constructing the loop hierarchy. Actual optimizations are tested
26 * through the checker tests.
27 */
Vladimir Markoca6fff82017-10-03 14:49:14 +010028class LoopOptimizationTest : public OptimizingUnitTest {
Aart Bik281c6812016-08-26 11:31:48 -070029 public:
30 LoopOptimizationTest()
Vladimir Markoca6fff82017-10-03 14:49:14 +010031 : graph_(CreateGraph()),
32 iva_(new (GetAllocator()) HInductionVarAnalysis(graph_)),
Vladimir Markoa0431112018-06-25 09:32:54 +010033 loop_opt_(new (GetAllocator()) HLoopOptimization(
Andreas Gampe3db70682018-12-26 15:12:03 -080034 graph_, /* compiler_options= */ nullptr, iva_, /* stats= */ nullptr)) {
Aart Bik281c6812016-08-26 11:31:48 -070035 BuildGraph();
36 }
37
38 ~LoopOptimizationTest() { }
39
40 /** Constructs bare minimum graph. */
41 void BuildGraph() {
42 graph_->SetNumberOfVRegs(1);
Vladimir Markoca6fff82017-10-03 14:49:14 +010043 entry_block_ = new (GetAllocator()) HBasicBlock(graph_);
44 return_block_ = new (GetAllocator()) HBasicBlock(graph_);
45 exit_block_ = new (GetAllocator()) HBasicBlock(graph_);
Aart Bik281c6812016-08-26 11:31:48 -070046 graph_->AddBlock(entry_block_);
47 graph_->AddBlock(return_block_);
48 graph_->AddBlock(exit_block_);
49 graph_->SetEntryBlock(entry_block_);
50 graph_->SetExitBlock(exit_block_);
Vladimir Markoca6fff82017-10-03 14:49:14 +010051 parameter_ = new (GetAllocator()) HParameterValue(graph_->GetDexFile(),
52 dex::TypeIndex(0),
53 0,
54 DataType::Type::kInt32);
Aart Bik281c6812016-08-26 11:31:48 -070055 entry_block_->AddInstruction(parameter_);
Vladimir Markoca6fff82017-10-03 14:49:14 +010056 return_block_->AddInstruction(new (GetAllocator()) HReturnVoid());
57 exit_block_->AddInstruction(new (GetAllocator()) HExit());
Aart Bik281c6812016-08-26 11:31:48 -070058 entry_block_->AddSuccessor(return_block_);
59 return_block_->AddSuccessor(exit_block_);
60 }
61
62 /** Adds a loop nest at given position before successor. */
63 HBasicBlock* AddLoop(HBasicBlock* position, HBasicBlock* successor) {
Vladimir Markoca6fff82017-10-03 14:49:14 +010064 HBasicBlock* header = new (GetAllocator()) HBasicBlock(graph_);
65 HBasicBlock* body = new (GetAllocator()) HBasicBlock(graph_);
Aart Bik281c6812016-08-26 11:31:48 -070066 graph_->AddBlock(header);
67 graph_->AddBlock(body);
68 // Control flow.
69 position->ReplaceSuccessor(successor, header);
70 header->AddSuccessor(body);
71 header->AddSuccessor(successor);
Vladimir Markoca6fff82017-10-03 14:49:14 +010072 header->AddInstruction(new (GetAllocator()) HIf(parameter_));
Aart Bik281c6812016-08-26 11:31:48 -070073 body->AddSuccessor(header);
Vladimir Markoca6fff82017-10-03 14:49:14 +010074 body->AddInstruction(new (GetAllocator()) HGoto());
Aart Bik281c6812016-08-26 11:31:48 -070075 return header;
76 }
77
78 /** Performs analysis. */
79 void PerformAnalysis() {
80 graph_->BuildDominatorTree();
81 iva_->Run();
Aart Bik96202302016-10-04 17:33:56 -070082 // Do not release the loop hierarchy.
Vladimir Markoca6fff82017-10-03 14:49:14 +010083 ScopedArenaAllocator loop_allocator(GetArenaStack());
84 loop_opt_->loop_allocator_ = &loop_allocator;
Aart Bik96202302016-10-04 17:33:56 -070085 loop_opt_->LocalRun();
Aart Bik281c6812016-08-26 11:31:48 -070086 }
87
88 /** Constructs string representation of computed loop hierarchy. */
89 std::string LoopStructure() {
90 return LoopStructureRecurse(loop_opt_->top_loop_);
91 }
92
93 // Helper method
94 std::string LoopStructureRecurse(HLoopOptimization::LoopNode* node) {
95 std::string s;
96 for ( ; node != nullptr; node = node->next) {
97 s.append("[");
98 s.append(LoopStructureRecurse(node->inner));
99 s.append("]");
100 }
101 return s;
102 }
103
104 // General building fields.
Aart Bik281c6812016-08-26 11:31:48 -0700105 HGraph* graph_;
106 HInductionVarAnalysis* iva_;
107 HLoopOptimization* loop_opt_;
108
109 HBasicBlock* entry_block_;
110 HBasicBlock* return_block_;
111 HBasicBlock* exit_block_;
112
113 HInstruction* parameter_;
114};
115
116//
117// The actual tests.
118//
119
120TEST_F(LoopOptimizationTest, NoLoops) {
121 PerformAnalysis();
122 EXPECT_EQ("", LoopStructure());
123}
124
125TEST_F(LoopOptimizationTest, SingleLoop) {
126 AddLoop(entry_block_, return_block_);
127 PerformAnalysis();
128 EXPECT_EQ("[]", LoopStructure());
129}
130
131TEST_F(LoopOptimizationTest, LoopNest10) {
132 HBasicBlock* b = entry_block_;
133 HBasicBlock* s = return_block_;
134 for (int i = 0; i < 10; i++) {
135 s = AddLoop(b, s);
136 b = s->GetSuccessors()[0];
137 }
138 PerformAnalysis();
139 EXPECT_EQ("[[[[[[[[[[]]]]]]]]]]", LoopStructure());
140}
141
142TEST_F(LoopOptimizationTest, LoopSequence10) {
143 HBasicBlock* b = entry_block_;
144 HBasicBlock* s = return_block_;
145 for (int i = 0; i < 10; i++) {
146 b = AddLoop(b, s);
147 s = b->GetSuccessors()[1];
148 }
149 PerformAnalysis();
150 EXPECT_EQ("[][][][][][][][][][]", LoopStructure());
151}
152
153TEST_F(LoopOptimizationTest, LoopSequenceOfNests) {
154 HBasicBlock* b = entry_block_;
155 HBasicBlock* s = return_block_;
156 for (int i = 0; i < 10; i++) {
157 b = AddLoop(b, s);
158 s = b->GetSuccessors()[1];
159 HBasicBlock* bi = b->GetSuccessors()[0];
160 HBasicBlock* si = b;
161 for (int j = 0; j < i; j++) {
162 si = AddLoop(bi, si);
163 bi = si->GetSuccessors()[0];
164 }
165 }
166 PerformAnalysis();
167 EXPECT_EQ("[]"
168 "[[]]"
169 "[[[]]]"
170 "[[[[]]]]"
171 "[[[[[]]]]]"
172 "[[[[[[]]]]]]"
173 "[[[[[[[]]]]]]]"
174 "[[[[[[[[]]]]]]]]"
175 "[[[[[[[[[]]]]]]]]]"
176 "[[[[[[[[[[]]]]]]]]]]",
177 LoopStructure());
178}
179
180TEST_F(LoopOptimizationTest, LoopNestWithSequence) {
181 HBasicBlock* b = entry_block_;
182 HBasicBlock* s = return_block_;
183 for (int i = 0; i < 10; i++) {
184 s = AddLoop(b, s);
185 b = s->GetSuccessors()[0];
186 }
187 b = s;
188 s = b->GetSuccessors()[1];
189 for (int i = 0; i < 9; i++) {
190 b = AddLoop(b, s);
191 s = b->GetSuccessors()[1];
192 }
193 PerformAnalysis();
194 EXPECT_EQ("[[[[[[[[[[][][][][][][][][][]]]]]]]]]]", LoopStructure());
195}
196
Artem Serovc73ee372017-07-31 15:08:40 +0100197// Check that SimplifyLoop() doesn't invalidate data flow when ordering loop headers'
198// predecessors.
Artem Serov09faaea2017-12-07 14:36:01 +0000199//
200// This is a test for nodes.cc functionality - HGraph::SimplifyLoop.
201TEST_F(LoopOptimizationTest, SimplifyLoopReoderPredecessors) {
Artem Serovc73ee372017-07-31 15:08:40 +0100202 // Can't use AddLoop as we want special order for blocks predecessors.
Vladimir Markoca6fff82017-10-03 14:49:14 +0100203 HBasicBlock* header = new (GetAllocator()) HBasicBlock(graph_);
204 HBasicBlock* body = new (GetAllocator()) HBasicBlock(graph_);
Artem Serovc73ee372017-07-31 15:08:40 +0100205 graph_->AddBlock(header);
206 graph_->AddBlock(body);
207
208 // Control flow: make a loop back edge first in the list of predecessors.
209 entry_block_->RemoveSuccessor(return_block_);
210 body->AddSuccessor(header);
211 entry_block_->AddSuccessor(header);
212 header->AddSuccessor(body);
213 header->AddSuccessor(return_block_);
214 DCHECK(header->GetSuccessors()[1] == return_block_);
215
216 // Data flow.
Vladimir Markoca6fff82017-10-03 14:49:14 +0100217 header->AddInstruction(new (GetAllocator()) HIf(parameter_));
218 body->AddInstruction(new (GetAllocator()) HGoto());
Artem Serovc73ee372017-07-31 15:08:40 +0100219
Vladimir Markoca6fff82017-10-03 14:49:14 +0100220 HPhi* phi = new (GetAllocator()) HPhi(GetAllocator(), 0, 0, DataType::Type::kInt32);
221 HInstruction* add = new (GetAllocator()) HAdd(DataType::Type::kInt32, phi, parameter_);
Artem Serovc73ee372017-07-31 15:08:40 +0100222 header->AddPhi(phi);
223 body->AddInstruction(add);
224
225 phi->AddInput(add);
226 phi->AddInput(parameter_);
227
228 graph_->ClearLoopInformation();
229 graph_->ClearDominanceInformation();
230 graph_->BuildDominatorTree();
231
Artem Serov02eebcf2017-12-13 19:48:31 +0000232 // BuildDominatorTree inserts a block beetween loop header and entry block.
233 EXPECT_EQ(header->GetPredecessors()[0]->GetSinglePredecessor(), entry_block_);
234
Artem Serovc73ee372017-07-31 15:08:40 +0100235 // Check that after optimizations in BuildDominatorTree()/SimplifyCFG() phi inputs
236 // are still mapped correctly to the block predecessors.
237 for (size_t i = 0, e = phi->InputCount(); i < e; i++) {
238 HInstruction* input = phi->InputAt(i);
Artem Serov02eebcf2017-12-13 19:48:31 +0000239 EXPECT_TRUE(input->GetBlock()->Dominates(header->GetPredecessors()[i]));
Artem Serovc73ee372017-07-31 15:08:40 +0100240 }
241}
Artem Serov09faaea2017-12-07 14:36:01 +0000242
243// Test that SimplifyLoop() processes the multiple-preheaders loops correctly.
244//
245// This is a test for nodes.cc functionality - HGraph::SimplifyLoop.
246TEST_F(LoopOptimizationTest, SimplifyLoopSinglePreheader) {
247 HBasicBlock* header = AddLoop(entry_block_, return_block_);
248
249 header->InsertInstructionBefore(
250 new (GetAllocator()) HSuspendCheck(), header->GetLastInstruction());
251
252 // Insert an if construct before the loop so it will have two preheaders.
253 HBasicBlock* if_block = new (GetAllocator()) HBasicBlock(graph_);
254 HBasicBlock* preheader0 = new (GetAllocator()) HBasicBlock(graph_);
255 HBasicBlock* preheader1 = new (GetAllocator()) HBasicBlock(graph_);
256
257 graph_->AddBlock(if_block);
258 graph_->AddBlock(preheader0);
259 graph_->AddBlock(preheader1);
260
261 // Fix successors/predecessors.
262 entry_block_->ReplaceSuccessor(header, if_block);
263 if_block->AddSuccessor(preheader0);
264 if_block->AddSuccessor(preheader1);
265 preheader0->AddSuccessor(header);
266 preheader1->AddSuccessor(header);
267
268 if_block->AddInstruction(new (GetAllocator()) HIf(parameter_));
269 preheader0->AddInstruction(new (GetAllocator()) HGoto());
270 preheader1->AddInstruction(new (GetAllocator()) HGoto());
271
272 HBasicBlock* body = header->GetSuccessors()[0];
273 DCHECK(body != return_block_);
274
275 // Add some data flow.
276 HIntConstant* const_0 = graph_->GetIntConstant(0);
277 HIntConstant* const_1 = graph_->GetIntConstant(1);
278 HIntConstant* const_2 = graph_->GetIntConstant(2);
279
280 HAdd* preheader0_add = new (GetAllocator()) HAdd(DataType::Type::kInt32, parameter_, const_0);
281 preheader0->AddInstruction(preheader0_add);
282 HAdd* preheader1_add = new (GetAllocator()) HAdd(DataType::Type::kInt32, parameter_, const_1);
283 preheader1->AddInstruction(preheader1_add);
284
285 HPhi* header_phi = new (GetAllocator()) HPhi(GetAllocator(), 0, 0, DataType::Type::kInt32);
286 header->AddPhi(header_phi);
287
288 HAdd* body_add = new (GetAllocator()) HAdd(DataType::Type::kInt32, parameter_, const_2);
289 body->AddInstruction(body_add);
290
291 DCHECK(header->GetPredecessors()[0] == body);
292 DCHECK(header->GetPredecessors()[1] == preheader0);
293 DCHECK(header->GetPredecessors()[2] == preheader1);
294
295 header_phi->AddInput(body_add);
296 header_phi->AddInput(preheader0_add);
297 header_phi->AddInput(preheader1_add);
298
299 graph_->ClearLoopInformation();
300 graph_->ClearDominanceInformation();
301 graph_->BuildDominatorTree();
302
303 EXPECT_EQ(header->GetPredecessors().size(), 2u);
304 EXPECT_EQ(header->GetPredecessors()[1], body);
305
306 HBasicBlock* new_preheader = header->GetLoopInformation()->GetPreHeader();
307 EXPECT_EQ(preheader0->GetSingleSuccessor(), new_preheader);
308 EXPECT_EQ(preheader1->GetSingleSuccessor(), new_preheader);
309
310 EXPECT_EQ(new_preheader->GetPhis().CountSize(), 1u);
311 HPhi* new_preheader_phi = new_preheader->GetFirstPhi()->AsPhi();
312 EXPECT_EQ(new_preheader_phi->InputCount(), 2u);
313 EXPECT_EQ(new_preheader_phi->InputAt(0), preheader0_add);
314 EXPECT_EQ(new_preheader_phi->InputAt(1), preheader1_add);
315
316 EXPECT_EQ(header_phi->InputCount(), 2u);
317 EXPECT_EQ(header_phi->InputAt(0), new_preheader_phi);
318 EXPECT_EQ(header_phi->InputAt(1), body_add);
319}
320
Aart Bik281c6812016-08-26 11:31:48 -0700321} // namespace art