Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 1 | /* |
| 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 Marko | 434d968 | 2022-11-04 14:04:17 +0000 | [diff] [blame] | 17 | #include "base/macros.h" |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 18 | #include "code_generator.h" |
Vladimir Marko | f91fc12 | 2020-05-13 09:21:00 +0100 | [diff] [blame] | 19 | #include "driver/compiler_options.h" |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 20 | #include "loop_optimization.h" |
| 21 | #include "optimizing_unit_test.h" |
| 22 | |
Vladimír Marko | 434d968 | 2022-11-04 14:04:17 +0000 | [diff] [blame] | 23 | namespace art HIDDEN { |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 24 | |
| 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 Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 30 | class LoopOptimizationTest : public OptimizingUnitTest { |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 31 | protected: |
| 32 | void SetUp() override { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 33 | TEST_SETUP_DISABLED_FOR_RISCV64(); |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 34 | OptimizingUnitTest::SetUp(); |
| 35 | |
| 36 | graph_ = CreateGraph(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 37 | BuildGraph(); |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 38 | iva_ = new (GetAllocator()) HInductionVarAnalysis(graph_); |
Vladimir Marko | f91fc12 | 2020-05-13 09:21:00 +0100 | [diff] [blame] | 39 | compiler_options_ = CommonCompilerTest::CreateCompilerOptions(kRuntimeISA, "default"); |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 40 | 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 Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 45 | } |
| 46 | |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 47 | void TearDown() override { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 48 | TEST_TEARDOWN_DISABLED_FOR_RISCV64(); |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 49 | codegen_.reset(); |
Vladimir Marko | f91fc12 | 2020-05-13 09:21:00 +0100 | [diff] [blame] | 50 | compiler_options_.reset(); |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 51 | graph_ = nullptr; |
| 52 | ResetPoolAndAllocator(); |
| 53 | OptimizingUnitTest::TearDown(); |
| 54 | } |
| 55 | |
| 56 | virtual ~LoopOptimizationTest() {} |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 57 | |
| 58 | /** Constructs bare minimum graph. */ |
| 59 | void BuildGraph() { |
| 60 | graph_->SetNumberOfVRegs(1); |
Vladimir Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 61 | entry_block_ = new (GetAllocator()) HBasicBlock(graph_); |
| 62 | return_block_ = new (GetAllocator()) HBasicBlock(graph_); |
| 63 | exit_block_ = new (GetAllocator()) HBasicBlock(graph_); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 64 | 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 Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 69 | parameter_ = new (GetAllocator()) HParameterValue(graph_->GetDexFile(), |
| 70 | dex::TypeIndex(0), |
| 71 | 0, |
| 72 | DataType::Type::kInt32); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 73 | entry_block_->AddInstruction(parameter_); |
Vladimir Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 74 | return_block_->AddInstruction(new (GetAllocator()) HReturnVoid()); |
| 75 | exit_block_->AddInstruction(new (GetAllocator()) HExit()); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 76 | 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 Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 82 | HBasicBlock* header = new (GetAllocator()) HBasicBlock(graph_); |
| 83 | HBasicBlock* body = new (GetAllocator()) HBasicBlock(graph_); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 84 | graph_->AddBlock(header); |
| 85 | graph_->AddBlock(body); |
| 86 | // Control flow. |
| 87 | position->ReplaceSuccessor(successor, header); |
| 88 | header->AddSuccessor(body); |
| 89 | header->AddSuccessor(successor); |
Vladimir Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 90 | header->AddInstruction(new (GetAllocator()) HIf(parameter_)); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 91 | body->AddSuccessor(header); |
Vladimir Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 92 | body->AddInstruction(new (GetAllocator()) HGoto()); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 93 | return header; |
| 94 | } |
| 95 | |
| 96 | /** Performs analysis. */ |
| 97 | void PerformAnalysis() { |
| 98 | graph_->BuildDominatorTree(); |
| 99 | iva_->Run(); |
Santiago Aboy Solanes | 74da668 | 2022-12-16 19:28:47 +0000 | [diff] [blame] | 100 | loop_opt_->Run(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 101 | } |
| 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 Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 120 | HGraph* graph_; |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 121 | |
Vladimir Marko | f91fc12 | 2020-05-13 09:21:00 +0100 | [diff] [blame] | 122 | std::unique_ptr<CompilerOptions> compiler_options_; |
Artem Serov | c8150b5 | 2019-07-31 18:28:00 +0100 | [diff] [blame] | 123 | std::unique_ptr<CodeGenerator> codegen_; |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 124 | 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 | |
| 138 | TEST_F(LoopOptimizationTest, NoLoops) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 139 | TEST_DISABLED_FOR_RISCV64(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 140 | PerformAnalysis(); |
| 141 | EXPECT_EQ("", LoopStructure()); |
| 142 | } |
| 143 | |
| 144 | TEST_F(LoopOptimizationTest, SingleLoop) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 145 | TEST_DISABLED_FOR_RISCV64(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 146 | AddLoop(entry_block_, return_block_); |
| 147 | PerformAnalysis(); |
| 148 | EXPECT_EQ("[]", LoopStructure()); |
| 149 | } |
| 150 | |
| 151 | TEST_F(LoopOptimizationTest, LoopNest10) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 152 | TEST_DISABLED_FOR_RISCV64(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 153 | 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 | |
| 163 | TEST_F(LoopOptimizationTest, LoopSequence10) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 164 | TEST_DISABLED_FOR_RISCV64(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 165 | 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 | |
| 175 | TEST_F(LoopOptimizationTest, LoopSequenceOfNests) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 176 | TEST_DISABLED_FOR_RISCV64(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 177 | 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 | |
| 203 | TEST_F(LoopOptimizationTest, LoopNestWithSequence) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 204 | TEST_DISABLED_FOR_RISCV64(); |
Aart Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 205 | 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 Serov | c73ee37 | 2017-07-31 15:08:40 +0100 | [diff] [blame] | 221 | // Check that SimplifyLoop() doesn't invalidate data flow when ordering loop headers' |
| 222 | // predecessors. |
Artem Serov | 09faaea | 2017-12-07 14:36:01 +0000 | [diff] [blame] | 223 | // |
| 224 | // This is a test for nodes.cc functionality - HGraph::SimplifyLoop. |
| 225 | TEST_F(LoopOptimizationTest, SimplifyLoopReoderPredecessors) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 226 | TEST_DISABLED_FOR_RISCV64(); |
Artem Serov | c73ee37 | 2017-07-31 15:08:40 +0100 | [diff] [blame] | 227 | // Can't use AddLoop as we want special order for blocks predecessors. |
Vladimir Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 228 | HBasicBlock* header = new (GetAllocator()) HBasicBlock(graph_); |
| 229 | HBasicBlock* body = new (GetAllocator()) HBasicBlock(graph_); |
Artem Serov | c73ee37 | 2017-07-31 15:08:40 +0100 | [diff] [blame] | 230 | 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 Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 242 | header->AddInstruction(new (GetAllocator()) HIf(parameter_)); |
| 243 | body->AddInstruction(new (GetAllocator()) HGoto()); |
Artem Serov | c73ee37 | 2017-07-31 15:08:40 +0100 | [diff] [blame] | 244 | |
Vladimir Marko | ca6fff8 | 2017-10-03 14:49:14 +0100 | [diff] [blame] | 245 | HPhi* phi = new (GetAllocator()) HPhi(GetAllocator(), 0, 0, DataType::Type::kInt32); |
| 246 | HInstruction* add = new (GetAllocator()) HAdd(DataType::Type::kInt32, phi, parameter_); |
Artem Serov | c73ee37 | 2017-07-31 15:08:40 +0100 | [diff] [blame] | 247 | 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 Serov | 02eebcf | 2017-12-13 19:48:31 +0000 | [diff] [blame] | 257 | // BuildDominatorTree inserts a block beetween loop header and entry block. |
| 258 | EXPECT_EQ(header->GetPredecessors()[0]->GetSinglePredecessor(), entry_block_); |
| 259 | |
Artem Serov | c73ee37 | 2017-07-31 15:08:40 +0100 | [diff] [blame] | 260 | // 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 Serov | 02eebcf | 2017-12-13 19:48:31 +0000 | [diff] [blame] | 264 | EXPECT_TRUE(input->GetBlock()->Dominates(header->GetPredecessors()[i])); |
Artem Serov | c73ee37 | 2017-07-31 15:08:40 +0100 | [diff] [blame] | 265 | } |
| 266 | } |
Artem Serov | 09faaea | 2017-12-07 14:36:01 +0000 | [diff] [blame] | 267 | |
| 268 | // Test that SimplifyLoop() processes the multiple-preheaders loops correctly. |
| 269 | // |
| 270 | // This is a test for nodes.cc functionality - HGraph::SimplifyLoop. |
| 271 | TEST_F(LoopOptimizationTest, SimplifyLoopSinglePreheader) { |
Ulya Trafimovich | 01e6b56 | 2023-01-17 13:13:55 +0000 | [diff] [blame] | 272 | TEST_DISABLED_FOR_RISCV64(); |
Artem Serov | 09faaea | 2017-12-07 14:36:01 +0000 | [diff] [blame] | 273 | 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 Marko | cde6497 | 2023-04-25 16:40:06 +0000 | [diff] [blame] | 337 | HPhi* new_preheader_phi = new_preheader->GetFirstPhi()->AsPhi(); |
Artem Serov | 09faaea | 2017-12-07 14:36:01 +0000 | [diff] [blame] | 338 | 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 Bik | 281c681 | 2016-08-26 11:31:48 -0700 | [diff] [blame] | 347 | } // namespace art |