blob: 49e3c0418f89869993eb1d698f02a9b275f8abf2 [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
Vladimír Marko434d9682022-11-04 14:04:17 +000017#include "base/macros.h"
Artem Serovc8150b52019-07-31 18:28:00 +010018#include "code_generator.h"
Vladimir Markof91fc122020-05-13 09:21:00 +010019#include "driver/compiler_options.h"
Aart Bik281c6812016-08-26 11:31:48 -070020#include "loop_optimization.h"
21#include "optimizing_unit_test.h"
22
Vladimír Marko434d9682022-11-04 14:04:17 +000023namespace art HIDDEN {
Aart Bik281c6812016-08-26 11:31:48 -070024
25/**
26 * Fixture class for the loop optimization tests. These unit tests focus
27 * constructing the loop hierarchy. Actual optimizations are tested
28 * through the checker tests.
29 */
Vladimir Markoca6fff82017-10-03 14:49:14 +010030class LoopOptimizationTest : public OptimizingUnitTest {
Artem Serovc8150b52019-07-31 18:28:00 +010031 protected:
32 void SetUp() override {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +000033 TEST_SETUP_DISABLED_FOR_RISCV64();
Artem Serovc8150b52019-07-31 18:28:00 +010034 OptimizingUnitTest::SetUp();
35
36 graph_ = CreateGraph();
Aart Bik281c6812016-08-26 11:31:48 -070037 BuildGraph();
Artem Serovc8150b52019-07-31 18:28:00 +010038 iva_ = new (GetAllocator()) HInductionVarAnalysis(graph_);
Vladimir Markof91fc122020-05-13 09:21:00 +010039 compiler_options_ = CommonCompilerTest::CreateCompilerOptions(kRuntimeISA, "default");
Artem Serovc8150b52019-07-31 18:28:00 +010040 DCHECK(compiler_options_ != nullptr);
41 codegen_ = CodeGenerator::Create(graph_, *compiler_options_);
42 DCHECK(codegen_.get() != nullptr);
43 loop_opt_ = new (GetAllocator()) HLoopOptimization(
44 graph_, *codegen_.get(), iva_, /* stats= */ nullptr);
Aart Bik281c6812016-08-26 11:31:48 -070045 }
46
Artem Serovc8150b52019-07-31 18:28:00 +010047 void TearDown() override {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +000048 TEST_TEARDOWN_DISABLED_FOR_RISCV64();
Artem Serovc8150b52019-07-31 18:28:00 +010049 codegen_.reset();
Vladimir Markof91fc122020-05-13 09:21:00 +010050 compiler_options_.reset();
Artem Serovc8150b52019-07-31 18:28:00 +010051 graph_ = nullptr;
52 ResetPoolAndAllocator();
53 OptimizingUnitTest::TearDown();
54 }
55
56 virtual ~LoopOptimizationTest() {}
Aart Bik281c6812016-08-26 11:31:48 -070057
58 /** Constructs bare minimum graph. */
59 void BuildGraph() {
60 graph_->SetNumberOfVRegs(1);
Vladimir Markoca6fff82017-10-03 14:49:14 +010061 entry_block_ = new (GetAllocator()) HBasicBlock(graph_);
62 return_block_ = new (GetAllocator()) HBasicBlock(graph_);
63 exit_block_ = new (GetAllocator()) HBasicBlock(graph_);
Aart Bik281c6812016-08-26 11:31:48 -070064 graph_->AddBlock(entry_block_);
65 graph_->AddBlock(return_block_);
66 graph_->AddBlock(exit_block_);
67 graph_->SetEntryBlock(entry_block_);
68 graph_->SetExitBlock(exit_block_);
Vladimir Markoca6fff82017-10-03 14:49:14 +010069 parameter_ = new (GetAllocator()) HParameterValue(graph_->GetDexFile(),
70 dex::TypeIndex(0),
71 0,
72 DataType::Type::kInt32);
Aart Bik281c6812016-08-26 11:31:48 -070073 entry_block_->AddInstruction(parameter_);
Vladimir Markoca6fff82017-10-03 14:49:14 +010074 return_block_->AddInstruction(new (GetAllocator()) HReturnVoid());
75 exit_block_->AddInstruction(new (GetAllocator()) HExit());
Aart Bik281c6812016-08-26 11:31:48 -070076 entry_block_->AddSuccessor(return_block_);
77 return_block_->AddSuccessor(exit_block_);
78 }
79
80 /** Adds a loop nest at given position before successor. */
81 HBasicBlock* AddLoop(HBasicBlock* position, HBasicBlock* successor) {
Vladimir Markoca6fff82017-10-03 14:49:14 +010082 HBasicBlock* header = new (GetAllocator()) HBasicBlock(graph_);
83 HBasicBlock* body = new (GetAllocator()) HBasicBlock(graph_);
Aart Bik281c6812016-08-26 11:31:48 -070084 graph_->AddBlock(header);
85 graph_->AddBlock(body);
86 // Control flow.
87 position->ReplaceSuccessor(successor, header);
88 header->AddSuccessor(body);
89 header->AddSuccessor(successor);
Vladimir Markoca6fff82017-10-03 14:49:14 +010090 header->AddInstruction(new (GetAllocator()) HIf(parameter_));
Aart Bik281c6812016-08-26 11:31:48 -070091 body->AddSuccessor(header);
Vladimir Markoca6fff82017-10-03 14:49:14 +010092 body->AddInstruction(new (GetAllocator()) HGoto());
Aart Bik281c6812016-08-26 11:31:48 -070093 return header;
94 }
95
96 /** Performs analysis. */
97 void PerformAnalysis() {
98 graph_->BuildDominatorTree();
99 iva_->Run();
Santiago Aboy Solanes74da6682022-12-16 19:28:47 +0000100 loop_opt_->Run();
Aart Bik281c6812016-08-26 11:31:48 -0700101 }
102
103 /** Constructs string representation of computed loop hierarchy. */
104 std::string LoopStructure() {
105 return LoopStructureRecurse(loop_opt_->top_loop_);
106 }
107
108 // Helper method
109 std::string LoopStructureRecurse(HLoopOptimization::LoopNode* node) {
110 std::string s;
111 for ( ; node != nullptr; node = node->next) {
112 s.append("[");
113 s.append(LoopStructureRecurse(node->inner));
114 s.append("]");
115 }
116 return s;
117 }
118
119 // General building fields.
Aart Bik281c6812016-08-26 11:31:48 -0700120 HGraph* graph_;
Artem Serovc8150b52019-07-31 18:28:00 +0100121
Vladimir Markof91fc122020-05-13 09:21:00 +0100122 std::unique_ptr<CompilerOptions> compiler_options_;
Artem Serovc8150b52019-07-31 18:28:00 +0100123 std::unique_ptr<CodeGenerator> codegen_;
Aart Bik281c6812016-08-26 11:31:48 -0700124 HInductionVarAnalysis* iva_;
125 HLoopOptimization* loop_opt_;
126
127 HBasicBlock* entry_block_;
128 HBasicBlock* return_block_;
129 HBasicBlock* exit_block_;
130
131 HInstruction* parameter_;
132};
133
134//
135// The actual tests.
136//
137
138TEST_F(LoopOptimizationTest, NoLoops) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000139 TEST_DISABLED_FOR_RISCV64();
Aart Bik281c6812016-08-26 11:31:48 -0700140 PerformAnalysis();
141 EXPECT_EQ("", LoopStructure());
142}
143
144TEST_F(LoopOptimizationTest, SingleLoop) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000145 TEST_DISABLED_FOR_RISCV64();
Aart Bik281c6812016-08-26 11:31:48 -0700146 AddLoop(entry_block_, return_block_);
147 PerformAnalysis();
148 EXPECT_EQ("[]", LoopStructure());
149}
150
151TEST_F(LoopOptimizationTest, LoopNest10) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000152 TEST_DISABLED_FOR_RISCV64();
Aart Bik281c6812016-08-26 11:31:48 -0700153 HBasicBlock* b = entry_block_;
154 HBasicBlock* s = return_block_;
155 for (int i = 0; i < 10; i++) {
156 s = AddLoop(b, s);
157 b = s->GetSuccessors()[0];
158 }
159 PerformAnalysis();
160 EXPECT_EQ("[[[[[[[[[[]]]]]]]]]]", LoopStructure());
161}
162
163TEST_F(LoopOptimizationTest, LoopSequence10) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000164 TEST_DISABLED_FOR_RISCV64();
Aart Bik281c6812016-08-26 11:31:48 -0700165 HBasicBlock* b = entry_block_;
166 HBasicBlock* s = return_block_;
167 for (int i = 0; i < 10; i++) {
168 b = AddLoop(b, s);
169 s = b->GetSuccessors()[1];
170 }
171 PerformAnalysis();
172 EXPECT_EQ("[][][][][][][][][][]", LoopStructure());
173}
174
175TEST_F(LoopOptimizationTest, LoopSequenceOfNests) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000176 TEST_DISABLED_FOR_RISCV64();
Aart Bik281c6812016-08-26 11:31:48 -0700177 HBasicBlock* b = entry_block_;
178 HBasicBlock* s = return_block_;
179 for (int i = 0; i < 10; i++) {
180 b = AddLoop(b, s);
181 s = b->GetSuccessors()[1];
182 HBasicBlock* bi = b->GetSuccessors()[0];
183 HBasicBlock* si = b;
184 for (int j = 0; j < i; j++) {
185 si = AddLoop(bi, si);
186 bi = si->GetSuccessors()[0];
187 }
188 }
189 PerformAnalysis();
190 EXPECT_EQ("[]"
191 "[[]]"
192 "[[[]]]"
193 "[[[[]]]]"
194 "[[[[[]]]]]"
195 "[[[[[[]]]]]]"
196 "[[[[[[[]]]]]]]"
197 "[[[[[[[[]]]]]]]]"
198 "[[[[[[[[[]]]]]]]]]"
199 "[[[[[[[[[[]]]]]]]]]]",
200 LoopStructure());
201}
202
203TEST_F(LoopOptimizationTest, LoopNestWithSequence) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000204 TEST_DISABLED_FOR_RISCV64();
Aart Bik281c6812016-08-26 11:31:48 -0700205 HBasicBlock* b = entry_block_;
206 HBasicBlock* s = return_block_;
207 for (int i = 0; i < 10; i++) {
208 s = AddLoop(b, s);
209 b = s->GetSuccessors()[0];
210 }
211 b = s;
212 s = b->GetSuccessors()[1];
213 for (int i = 0; i < 9; i++) {
214 b = AddLoop(b, s);
215 s = b->GetSuccessors()[1];
216 }
217 PerformAnalysis();
218 EXPECT_EQ("[[[[[[[[[[][][][][][][][][][]]]]]]]]]]", LoopStructure());
219}
220
Artem Serovc73ee372017-07-31 15:08:40 +0100221// Check that SimplifyLoop() doesn't invalidate data flow when ordering loop headers'
222// predecessors.
Artem Serov09faaea2017-12-07 14:36:01 +0000223//
224// This is a test for nodes.cc functionality - HGraph::SimplifyLoop.
225TEST_F(LoopOptimizationTest, SimplifyLoopReoderPredecessors) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000226 TEST_DISABLED_FOR_RISCV64();
Artem Serovc73ee372017-07-31 15:08:40 +0100227 // Can't use AddLoop as we want special order for blocks predecessors.
Vladimir Markoca6fff82017-10-03 14:49:14 +0100228 HBasicBlock* header = new (GetAllocator()) HBasicBlock(graph_);
229 HBasicBlock* body = new (GetAllocator()) HBasicBlock(graph_);
Artem Serovc73ee372017-07-31 15:08:40 +0100230 graph_->AddBlock(header);
231 graph_->AddBlock(body);
232
233 // Control flow: make a loop back edge first in the list of predecessors.
234 entry_block_->RemoveSuccessor(return_block_);
235 body->AddSuccessor(header);
236 entry_block_->AddSuccessor(header);
237 header->AddSuccessor(body);
238 header->AddSuccessor(return_block_);
239 DCHECK(header->GetSuccessors()[1] == return_block_);
240
241 // Data flow.
Vladimir Markoca6fff82017-10-03 14:49:14 +0100242 header->AddInstruction(new (GetAllocator()) HIf(parameter_));
243 body->AddInstruction(new (GetAllocator()) HGoto());
Artem Serovc73ee372017-07-31 15:08:40 +0100244
Vladimir Markoca6fff82017-10-03 14:49:14 +0100245 HPhi* phi = new (GetAllocator()) HPhi(GetAllocator(), 0, 0, DataType::Type::kInt32);
246 HInstruction* add = new (GetAllocator()) HAdd(DataType::Type::kInt32, phi, parameter_);
Artem Serovc73ee372017-07-31 15:08:40 +0100247 header->AddPhi(phi);
248 body->AddInstruction(add);
249
250 phi->AddInput(add);
251 phi->AddInput(parameter_);
252
253 graph_->ClearLoopInformation();
254 graph_->ClearDominanceInformation();
255 graph_->BuildDominatorTree();
256
Artem Serov02eebcf2017-12-13 19:48:31 +0000257 // BuildDominatorTree inserts a block beetween loop header and entry block.
258 EXPECT_EQ(header->GetPredecessors()[0]->GetSinglePredecessor(), entry_block_);
259
Artem Serovc73ee372017-07-31 15:08:40 +0100260 // Check that after optimizations in BuildDominatorTree()/SimplifyCFG() phi inputs
261 // are still mapped correctly to the block predecessors.
262 for (size_t i = 0, e = phi->InputCount(); i < e; i++) {
263 HInstruction* input = phi->InputAt(i);
Artem Serov02eebcf2017-12-13 19:48:31 +0000264 EXPECT_TRUE(input->GetBlock()->Dominates(header->GetPredecessors()[i]));
Artem Serovc73ee372017-07-31 15:08:40 +0100265 }
266}
Artem Serov09faaea2017-12-07 14:36:01 +0000267
268// Test that SimplifyLoop() processes the multiple-preheaders loops correctly.
269//
270// This is a test for nodes.cc functionality - HGraph::SimplifyLoop.
271TEST_F(LoopOptimizationTest, SimplifyLoopSinglePreheader) {
Ulya Trafimovich01e6b562023-01-17 13:13:55 +0000272 TEST_DISABLED_FOR_RISCV64();
Artem Serov09faaea2017-12-07 14:36:01 +0000273 HBasicBlock* header = AddLoop(entry_block_, return_block_);
274
275 header->InsertInstructionBefore(
276 new (GetAllocator()) HSuspendCheck(), header->GetLastInstruction());
277
278 // Insert an if construct before the loop so it will have two preheaders.
279 HBasicBlock* if_block = new (GetAllocator()) HBasicBlock(graph_);
280 HBasicBlock* preheader0 = new (GetAllocator()) HBasicBlock(graph_);
281 HBasicBlock* preheader1 = new (GetAllocator()) HBasicBlock(graph_);
282
283 graph_->AddBlock(if_block);
284 graph_->AddBlock(preheader0);
285 graph_->AddBlock(preheader1);
286
287 // Fix successors/predecessors.
288 entry_block_->ReplaceSuccessor(header, if_block);
289 if_block->AddSuccessor(preheader0);
290 if_block->AddSuccessor(preheader1);
291 preheader0->AddSuccessor(header);
292 preheader1->AddSuccessor(header);
293
294 if_block->AddInstruction(new (GetAllocator()) HIf(parameter_));
295 preheader0->AddInstruction(new (GetAllocator()) HGoto());
296 preheader1->AddInstruction(new (GetAllocator()) HGoto());
297
298 HBasicBlock* body = header->GetSuccessors()[0];
299 DCHECK(body != return_block_);
300
301 // Add some data flow.
302 HIntConstant* const_0 = graph_->GetIntConstant(0);
303 HIntConstant* const_1 = graph_->GetIntConstant(1);
304 HIntConstant* const_2 = graph_->GetIntConstant(2);
305
306 HAdd* preheader0_add = new (GetAllocator()) HAdd(DataType::Type::kInt32, parameter_, const_0);
307 preheader0->AddInstruction(preheader0_add);
308 HAdd* preheader1_add = new (GetAllocator()) HAdd(DataType::Type::kInt32, parameter_, const_1);
309 preheader1->AddInstruction(preheader1_add);
310
311 HPhi* header_phi = new (GetAllocator()) HPhi(GetAllocator(), 0, 0, DataType::Type::kInt32);
312 header->AddPhi(header_phi);
313
314 HAdd* body_add = new (GetAllocator()) HAdd(DataType::Type::kInt32, parameter_, const_2);
315 body->AddInstruction(body_add);
316
317 DCHECK(header->GetPredecessors()[0] == body);
318 DCHECK(header->GetPredecessors()[1] == preheader0);
319 DCHECK(header->GetPredecessors()[2] == preheader1);
320
321 header_phi->AddInput(body_add);
322 header_phi->AddInput(preheader0_add);
323 header_phi->AddInput(preheader1_add);
324
325 graph_->ClearLoopInformation();
326 graph_->ClearDominanceInformation();
327 graph_->BuildDominatorTree();
328
329 EXPECT_EQ(header->GetPredecessors().size(), 2u);
330 EXPECT_EQ(header->GetPredecessors()[1], body);
331
332 HBasicBlock* new_preheader = header->GetLoopInformation()->GetPreHeader();
333 EXPECT_EQ(preheader0->GetSingleSuccessor(), new_preheader);
334 EXPECT_EQ(preheader1->GetSingleSuccessor(), new_preheader);
335
336 EXPECT_EQ(new_preheader->GetPhis().CountSize(), 1u);
Vladimir Markocde64972023-04-25 16:40:06 +0000337 HPhi* new_preheader_phi = new_preheader->GetFirstPhi()->AsPhi();
Artem Serov09faaea2017-12-07 14:36:01 +0000338 EXPECT_EQ(new_preheader_phi->InputCount(), 2u);
339 EXPECT_EQ(new_preheader_phi->InputAt(0), preheader0_add);
340 EXPECT_EQ(new_preheader_phi->InputAt(1), preheader1_add);
341
342 EXPECT_EQ(header_phi->InputCount(), 2u);
343 EXPECT_EQ(header_phi->InputAt(0), new_preheader_phi);
344 EXPECT_EQ(header_phi->InputAt(1), body_add);
345}
346
Aart Bik281c6812016-08-26 11:31:48 -0700347} // namespace art