Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2014 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 | |
| 17 | #include <vector> |
| 18 | |
Andreas Gampe | 0b9203e | 2015-01-22 20:39:27 -0800 | [diff] [blame] | 19 | #include "base/logging.h" |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 20 | #include "dataflow_iterator.h" |
| 21 | #include "dataflow_iterator-inl.h" |
Andreas Gampe | 0b9203e | 2015-01-22 20:39:27 -0800 | [diff] [blame] | 22 | #include "dex/compiler_ir.h" |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 23 | #include "dex/mir_field_info.h" |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 24 | #include "gtest/gtest.h" |
| 25 | |
| 26 | namespace art { |
| 27 | |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 28 | class MirOptimizationTest : public testing::Test { |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 29 | protected: |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 30 | struct BBDef { |
| 31 | static constexpr size_t kMaxSuccessors = 4; |
| 32 | static constexpr size_t kMaxPredecessors = 4; |
| 33 | |
| 34 | BBType type; |
| 35 | size_t num_successors; |
| 36 | BasicBlockId successors[kMaxPredecessors]; |
| 37 | size_t num_predecessors; |
| 38 | BasicBlockId predecessors[kMaxPredecessors]; |
| 39 | }; |
| 40 | |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 41 | struct MethodDef { |
| 42 | uint16_t method_idx; |
| 43 | uintptr_t declaring_dex_file; |
| 44 | uint16_t declaring_class_idx; |
| 45 | uint16_t declaring_method_idx; |
| 46 | InvokeType invoke_type; |
| 47 | InvokeType sharp_type; |
| 48 | bool is_referrers_class; |
| 49 | bool is_initialized; |
| 50 | }; |
| 51 | |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 52 | struct MIRDef { |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 53 | BasicBlockId bbid; |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 54 | Instruction::Code opcode; |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 55 | uint32_t field_or_method_info; |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 56 | uint32_t vA; |
| 57 | uint32_t vB; |
| 58 | uint32_t vC; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 59 | }; |
| 60 | |
| 61 | #define DEF_SUCC0() \ |
| 62 | 0u, { } |
| 63 | #define DEF_SUCC1(s1) \ |
| 64 | 1u, { s1 } |
| 65 | #define DEF_SUCC2(s1, s2) \ |
| 66 | 2u, { s1, s2 } |
| 67 | #define DEF_SUCC3(s1, s2, s3) \ |
| 68 | 3u, { s1, s2, s3 } |
| 69 | #define DEF_SUCC4(s1, s2, s3, s4) \ |
| 70 | 4u, { s1, s2, s3, s4 } |
| 71 | #define DEF_PRED0() \ |
| 72 | 0u, { } |
| 73 | #define DEF_PRED1(p1) \ |
| 74 | 1u, { p1 } |
| 75 | #define DEF_PRED2(p1, p2) \ |
| 76 | 2u, { p1, p2 } |
| 77 | #define DEF_PRED3(p1, p2, p3) \ |
| 78 | 3u, { p1, p2, p3 } |
| 79 | #define DEF_PRED4(p1, p2, p3, p4) \ |
| 80 | 4u, { p1, p2, p3, p4 } |
| 81 | #define DEF_BB(type, succ, pred) \ |
| 82 | { type, succ, pred } |
| 83 | |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 84 | #define DEF_SGET_SPUT(bb, opcode, vA, field_info) \ |
| 85 | { bb, opcode, field_info, vA, 0u, 0u } |
| 86 | #define DEF_IGET_IPUT(bb, opcode, vA, vB, field_info) \ |
| 87 | { bb, opcode, field_info, vA, vB, 0u } |
| 88 | #define DEF_AGET_APUT(bb, opcode, vA, vB, vC) \ |
| 89 | { bb, opcode, 0u, vA, vB, vC } |
| 90 | #define DEF_INVOKE(bb, opcode, vC, method_info) \ |
| 91 | { bb, opcode, method_info, 0u, 0u, vC } |
Vladimir Marko | 8b858e1 | 2014-11-27 14:52:37 +0000 | [diff] [blame] | 92 | #define DEF_OTHER0(bb, opcode) \ |
| 93 | { bb, opcode, 0u, 0u, 0u, 0u } |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 94 | #define DEF_OTHER1(bb, opcode, vA) \ |
| 95 | { bb, opcode, 0u, vA, 0u, 0u } |
| 96 | #define DEF_OTHER2(bb, opcode, vA, vB) \ |
| 97 | { bb, opcode, 0u, vA, vB, 0u } |
| 98 | |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 99 | void DoPrepareBasicBlocks(const BBDef* defs, size_t count) { |
| 100 | cu_.mir_graph->block_id_map_.clear(); |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 101 | cu_.mir_graph->block_list_.clear(); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 102 | ASSERT_LT(3u, count); // null, entry, exit and at least one bytecode block. |
| 103 | ASSERT_EQ(kNullBlock, defs[0].type); |
| 104 | ASSERT_EQ(kEntryBlock, defs[1].type); |
| 105 | ASSERT_EQ(kExitBlock, defs[2].type); |
| 106 | for (size_t i = 0u; i != count; ++i) { |
| 107 | const BBDef* def = &defs[i]; |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 108 | BasicBlock* bb = cu_.mir_graph->CreateNewBB(def->type); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 109 | if (def->num_successors <= 2) { |
| 110 | bb->successor_block_list_type = kNotUsed; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 111 | bb->fall_through = (def->num_successors >= 1) ? def->successors[0] : 0u; |
| 112 | bb->taken = (def->num_successors >= 2) ? def->successors[1] : 0u; |
| 113 | } else { |
| 114 | bb->successor_block_list_type = kPackedSwitch; |
| 115 | bb->fall_through = 0u; |
| 116 | bb->taken = 0u; |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 117 | bb->successor_blocks.reserve(def->num_successors); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 118 | for (size_t j = 0u; j != def->num_successors; ++j) { |
| 119 | SuccessorBlockInfo* successor_block_info = |
| 120 | static_cast<SuccessorBlockInfo*>(cu_.arena.Alloc(sizeof(SuccessorBlockInfo), |
| 121 | kArenaAllocSuccessor)); |
| 122 | successor_block_info->block = j; |
| 123 | successor_block_info->key = 0u; // Not used by class init check elimination. |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 124 | bb->successor_blocks.push_back(successor_block_info); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 125 | } |
| 126 | } |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 127 | bb->predecessors.assign(def->predecessors, def->predecessors + def->num_predecessors); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 128 | if (def->type == kDalvikByteCode || def->type == kEntryBlock || def->type == kExitBlock) { |
| 129 | bb->data_flow_info = static_cast<BasicBlockDataFlow*>( |
| 130 | cu_.arena.Alloc(sizeof(BasicBlockDataFlow), kArenaAllocDFInfo)); |
| 131 | } |
| 132 | } |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 133 | ASSERT_EQ(count, cu_.mir_graph->block_list_.size()); |
| 134 | cu_.mir_graph->entry_block_ = cu_.mir_graph->block_list_[1]; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 135 | ASSERT_EQ(kEntryBlock, cu_.mir_graph->entry_block_->block_type); |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 136 | cu_.mir_graph->exit_block_ = cu_.mir_graph->block_list_[2]; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 137 | ASSERT_EQ(kExitBlock, cu_.mir_graph->exit_block_->block_type); |
| 138 | } |
| 139 | |
| 140 | template <size_t count> |
| 141 | void PrepareBasicBlocks(const BBDef (&defs)[count]) { |
| 142 | DoPrepareBasicBlocks(defs, count); |
| 143 | } |
| 144 | |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 145 | void PrepareSingleBlock() { |
| 146 | static const BBDef bbs[] = { |
| 147 | DEF_BB(kNullBlock, DEF_SUCC0(), DEF_PRED0()), |
| 148 | DEF_BB(kEntryBlock, DEF_SUCC1(3), DEF_PRED0()), |
| 149 | DEF_BB(kExitBlock, DEF_SUCC0(), DEF_PRED1(3)), |
| 150 | DEF_BB(kDalvikByteCode, DEF_SUCC1(2), DEF_PRED1(1)), |
| 151 | }; |
| 152 | PrepareBasicBlocks(bbs); |
| 153 | } |
| 154 | |
| 155 | void PrepareDiamond() { |
| 156 | static const BBDef bbs[] = { |
| 157 | DEF_BB(kNullBlock, DEF_SUCC0(), DEF_PRED0()), |
| 158 | DEF_BB(kEntryBlock, DEF_SUCC1(3), DEF_PRED0()), |
| 159 | DEF_BB(kExitBlock, DEF_SUCC0(), DEF_PRED1(6)), |
| 160 | DEF_BB(kDalvikByteCode, DEF_SUCC2(4, 5), DEF_PRED1(1)), |
| 161 | DEF_BB(kDalvikByteCode, DEF_SUCC1(6), DEF_PRED1(3)), |
| 162 | DEF_BB(kDalvikByteCode, DEF_SUCC1(6), DEF_PRED1(3)), |
| 163 | DEF_BB(kDalvikByteCode, DEF_SUCC1(2), DEF_PRED2(4, 5)), |
| 164 | }; |
| 165 | PrepareBasicBlocks(bbs); |
| 166 | } |
| 167 | |
| 168 | void PrepareLoop() { |
| 169 | static const BBDef bbs[] = { |
| 170 | DEF_BB(kNullBlock, DEF_SUCC0(), DEF_PRED0()), |
| 171 | DEF_BB(kEntryBlock, DEF_SUCC1(3), DEF_PRED0()), |
| 172 | DEF_BB(kExitBlock, DEF_SUCC0(), DEF_PRED1(5)), |
| 173 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(1)), |
| 174 | DEF_BB(kDalvikByteCode, DEF_SUCC2(5, 4), DEF_PRED2(3, 4)), // "taken" loops to self. |
| 175 | DEF_BB(kDalvikByteCode, DEF_SUCC1(2), DEF_PRED1(4)), |
| 176 | }; |
| 177 | PrepareBasicBlocks(bbs); |
| 178 | } |
| 179 | |
Vladimir Marko | 8b858e1 | 2014-11-27 14:52:37 +0000 | [diff] [blame] | 180 | void PrepareNestedLoopsWhile_While() { |
| 181 | static const BBDef bbs[] = { |
| 182 | DEF_BB(kNullBlock, DEF_SUCC0(), DEF_PRED0()), |
| 183 | DEF_BB(kEntryBlock, DEF_SUCC1(3), DEF_PRED0()), |
| 184 | DEF_BB(kExitBlock, DEF_SUCC0(), DEF_PRED1(8)), |
| 185 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(1)), |
| 186 | DEF_BB(kDalvikByteCode, DEF_SUCC2(5, 8), DEF_PRED2(3, 7)), // Outer while loop head. |
| 187 | DEF_BB(kDalvikByteCode, DEF_SUCC2(6, 7), DEF_PRED2(4, 6)), // Inner while loop head. |
| 188 | DEF_BB(kDalvikByteCode, DEF_SUCC1(5), DEF_PRED1(5)), // "taken" loops to inner head. |
| 189 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(5)), // "taken" loops to outer head. |
| 190 | DEF_BB(kDalvikByteCode, DEF_SUCC1(2), DEF_PRED1(4)), |
| 191 | }; |
| 192 | PrepareBasicBlocks(bbs); |
| 193 | } |
| 194 | |
| 195 | void PrepareNestedLoopsWhile_WhileWhile() { |
| 196 | static const BBDef bbs[] = { |
| 197 | DEF_BB(kNullBlock, DEF_SUCC0(), DEF_PRED0()), |
| 198 | DEF_BB(kEntryBlock, DEF_SUCC1(3), DEF_PRED0()), |
| 199 | DEF_BB(kExitBlock, DEF_SUCC0(), DEF_PRED1(10)), |
| 200 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(1)), |
| 201 | DEF_BB(kDalvikByteCode, DEF_SUCC2(5, 10), DEF_PRED2(3, 9)), // Outer while loop head. |
| 202 | DEF_BB(kDalvikByteCode, DEF_SUCC2(6, 7), DEF_PRED2(4, 6)), // Inner while loop head 1. |
| 203 | DEF_BB(kDalvikByteCode, DEF_SUCC1(5), DEF_PRED1(5)), // Loops to inner head 1. |
| 204 | DEF_BB(kDalvikByteCode, DEF_SUCC2(8, 9), DEF_PRED2(5, 8)), // Inner while loop head 2. |
| 205 | DEF_BB(kDalvikByteCode, DEF_SUCC1(7), DEF_PRED1(7)), // loops to inner head 2. |
| 206 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(7)), // loops to outer head. |
| 207 | DEF_BB(kDalvikByteCode, DEF_SUCC1(2), DEF_PRED1(4)), |
| 208 | }; |
| 209 | PrepareBasicBlocks(bbs); |
| 210 | } |
| 211 | |
| 212 | void PrepareNestedLoopsWhile_WhileWhile_WithExtraEdge() { |
| 213 | // Extra edge from the first inner loop body to second inner loop body (6u->8u). |
| 214 | static const BBDef bbs[] = { |
| 215 | DEF_BB(kNullBlock, DEF_SUCC0(), DEF_PRED0()), |
| 216 | DEF_BB(kEntryBlock, DEF_SUCC1(3), DEF_PRED0()), |
| 217 | DEF_BB(kExitBlock, DEF_SUCC0(), DEF_PRED1(10)), |
| 218 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(1)), |
| 219 | DEF_BB(kDalvikByteCode, DEF_SUCC2(5, 10), DEF_PRED2(3, 9)), // Outer while loop head. |
| 220 | DEF_BB(kDalvikByteCode, DEF_SUCC2(6, 7), DEF_PRED2(4, 6)), // Inner while loop head 1. |
| 221 | DEF_BB(kDalvikByteCode, DEF_SUCC2(5, 8), DEF_PRED1(5)), // Loops to inner head 1. |
| 222 | DEF_BB(kDalvikByteCode, DEF_SUCC2(8, 9), DEF_PRED2(5, 8)), // Inner while loop head 2. |
| 223 | DEF_BB(kDalvikByteCode, DEF_SUCC1(7), DEF_PRED2(7, 6)), // loops to inner head 2. |
| 224 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(7)), // loops to outer head. |
| 225 | DEF_BB(kDalvikByteCode, DEF_SUCC1(2), DEF_PRED1(4)), |
| 226 | }; |
| 227 | PrepareBasicBlocks(bbs); |
| 228 | } |
| 229 | |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 230 | void PrepareCatch() { |
| 231 | static const BBDef bbs[] = { |
| 232 | DEF_BB(kNullBlock, DEF_SUCC0(), DEF_PRED0()), |
| 233 | DEF_BB(kEntryBlock, DEF_SUCC1(3), DEF_PRED0()), |
| 234 | DEF_BB(kExitBlock, DEF_SUCC0(), DEF_PRED1(6)), |
| 235 | DEF_BB(kDalvikByteCode, DEF_SUCC1(4), DEF_PRED1(1)), // The top. |
| 236 | DEF_BB(kDalvikByteCode, DEF_SUCC1(6), DEF_PRED1(3)), // The throwing insn. |
| 237 | DEF_BB(kDalvikByteCode, DEF_SUCC1(6), DEF_PRED1(3)), // Catch handler. |
| 238 | DEF_BB(kDalvikByteCode, DEF_SUCC1(2), DEF_PRED2(4, 5)), // The merged block. |
| 239 | }; |
| 240 | PrepareBasicBlocks(bbs); |
| 241 | BasicBlock* catch_handler = cu_.mir_graph->GetBasicBlock(5u); |
| 242 | catch_handler->catch_entry = true; |
| 243 | // Add successor block info to the check block. |
| 244 | BasicBlock* check_bb = cu_.mir_graph->GetBasicBlock(3u); |
| 245 | check_bb->successor_block_list_type = kCatch; |
| 246 | SuccessorBlockInfo* successor_block_info = reinterpret_cast<SuccessorBlockInfo*> |
| 247 | (cu_.arena.Alloc(sizeof(SuccessorBlockInfo), kArenaAllocSuccessor)); |
| 248 | successor_block_info->block = catch_handler->id; |
| 249 | check_bb->successor_blocks.push_back(successor_block_info); |
| 250 | } |
| 251 | |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 252 | void DoPrepareMethods(const MethodDef* defs, size_t count) { |
| 253 | cu_.mir_graph->method_lowering_infos_.clear(); |
| 254 | cu_.mir_graph->method_lowering_infos_.reserve(count); |
| 255 | for (size_t i = 0u; i != count; ++i) { |
| 256 | const MethodDef* def = &defs[i]; |
Mathieu Chartier | 2535abe | 2015-02-17 10:38:49 -0800 | [diff] [blame] | 257 | MirMethodLoweringInfo method_info(def->method_idx, def->invoke_type, false); |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 258 | if (def->declaring_dex_file != 0u) { |
| 259 | method_info.declaring_dex_file_ = reinterpret_cast<const DexFile*>(def->declaring_dex_file); |
| 260 | method_info.declaring_class_idx_ = def->declaring_class_idx; |
| 261 | method_info.declaring_method_idx_ = def->declaring_method_idx; |
| 262 | } |
| 263 | ASSERT_EQ(def->invoke_type != kStatic, def->sharp_type != kStatic); |
| 264 | method_info.flags_ = |
| 265 | ((def->invoke_type == kStatic) ? MirMethodLoweringInfo::kFlagIsStatic : 0u) | |
| 266 | MirMethodLoweringInfo::kFlagFastPath | |
| 267 | (static_cast<uint16_t>(def->invoke_type) << MirMethodLoweringInfo::kBitInvokeTypeBegin) | |
| 268 | (static_cast<uint16_t>(def->sharp_type) << MirMethodLoweringInfo::kBitSharpTypeBegin) | |
| 269 | ((def->is_referrers_class) ? MirMethodLoweringInfo::kFlagIsReferrersClass : 0u) | |
| 270 | ((def->is_initialized == kStatic) ? MirMethodLoweringInfo::kFlagClassIsInitialized : 0u); |
| 271 | ASSERT_EQ(def->declaring_dex_file != 0u, method_info.IsResolved()); |
| 272 | cu_.mir_graph->method_lowering_infos_.push_back(method_info); |
| 273 | } |
| 274 | } |
| 275 | |
| 276 | template <size_t count> |
| 277 | void PrepareMethods(const MethodDef (&defs)[count]) { |
| 278 | DoPrepareMethods(defs, count); |
| 279 | } |
| 280 | |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 281 | void DoPrepareMIRs(const MIRDef* defs, size_t count) { |
| 282 | mir_count_ = count; |
Vladimir Marko | e4fcc5b | 2015-02-13 10:28:29 +0000 | [diff] [blame] | 283 | mirs_ = cu_.arena.AllocArray<MIR>(count, kArenaAllocMIR); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 284 | uint64_t merged_df_flags = 0u; |
| 285 | for (size_t i = 0u; i != count; ++i) { |
| 286 | const MIRDef* def = &defs[i]; |
| 287 | MIR* mir = &mirs_[i]; |
| 288 | mir->dalvikInsn.opcode = def->opcode; |
Vladimir Marko | e39c54e | 2014-09-22 14:50:02 +0100 | [diff] [blame] | 289 | ASSERT_LT(def->bbid, cu_.mir_graph->block_list_.size()); |
| 290 | BasicBlock* bb = cu_.mir_graph->block_list_[def->bbid]; |
Jean Christophe Beyler | cdacac4 | 2014-03-13 14:54:59 -0700 | [diff] [blame] | 291 | bb->AppendMIR(mir); |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 292 | if (IsInstructionIGetOrIPut(def->opcode)) { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 293 | ASSERT_LT(def->field_or_method_info, cu_.mir_graph->ifield_lowering_infos_.size()); |
| 294 | mir->meta.ifield_lowering_info = def->field_or_method_info; |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 295 | ASSERT_EQ(cu_.mir_graph->ifield_lowering_infos_[def->field_or_method_info].MemAccessType(), |
| 296 | IGetOrIPutMemAccessType(def->opcode)); |
| 297 | } else if (IsInstructionSGetOrSPut(def->opcode)) { |
| 298 | ASSERT_LT(def->field_or_method_info, cu_.mir_graph->sfield_lowering_infos_.size()); |
| 299 | mir->meta.sfield_lowering_info = def->field_or_method_info; |
| 300 | ASSERT_EQ(cu_.mir_graph->sfield_lowering_infos_[def->field_or_method_info].MemAccessType(), |
| 301 | SGetOrSPutMemAccessType(def->opcode)); |
| 302 | } else if (IsInstructionInvoke(def->opcode)) { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 303 | ASSERT_LT(def->field_or_method_info, cu_.mir_graph->method_lowering_infos_.size()); |
| 304 | mir->meta.method_lowering_info = def->field_or_method_info; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 305 | } |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 306 | mir->dalvikInsn.vA = def->vA; |
| 307 | mir->dalvikInsn.vB = def->vB; |
| 308 | mir->dalvikInsn.vC = def->vC; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 309 | mir->ssa_rep = nullptr; |
| 310 | mir->offset = 2 * i; // All insns need to be at least 2 code units long. |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 311 | mir->optimization_flags = 0u; |
Jean Christophe Beyler | cc794c3 | 2014-05-02 09:34:13 -0700 | [diff] [blame] | 312 | merged_df_flags |= MIRGraph::GetDataFlowAttributes(def->opcode); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 313 | } |
| 314 | cu_.mir_graph->merged_df_flags_ = merged_df_flags; |
| 315 | |
| 316 | code_item_ = static_cast<DexFile::CodeItem*>( |
| 317 | cu_.arena.Alloc(sizeof(DexFile::CodeItem), kArenaAllocMisc)); |
| 318 | memset(code_item_, 0, sizeof(DexFile::CodeItem)); |
| 319 | code_item_->insns_size_in_code_units_ = 2u * count; |
Razvan A Lupusoru | 7503597 | 2014-09-11 15:24:59 -0700 | [diff] [blame] | 320 | cu_.mir_graph->current_code_item_ = code_item_; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 321 | } |
| 322 | |
| 323 | template <size_t count> |
| 324 | void PrepareMIRs(const MIRDef (&defs)[count]) { |
| 325 | DoPrepareMIRs(defs, count); |
| 326 | } |
| 327 | |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 328 | MirOptimizationTest() |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 329 | : pool_(), |
Andreas Gampe | 0b9203e | 2015-01-22 20:39:27 -0800 | [diff] [blame] | 330 | cu_(&pool_, kRuntimeISA, nullptr, nullptr), |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 331 | mir_count_(0u), |
| 332 | mirs_(nullptr), |
| 333 | code_item_(nullptr) { |
| 334 | cu_.mir_graph.reset(new MIRGraph(&cu_, &cu_.arena)); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 335 | cu_.access_flags = kAccStatic; // Don't let "this" interfere with this test. |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 336 | } |
| 337 | |
| 338 | ArenaPool pool_; |
| 339 | CompilationUnit cu_; |
| 340 | size_t mir_count_; |
| 341 | MIR* mirs_; |
| 342 | DexFile::CodeItem* code_item_; |
| 343 | }; |
| 344 | |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 345 | class ClassInitCheckEliminationTest : public MirOptimizationTest { |
| 346 | protected: |
| 347 | struct SFieldDef { |
| 348 | uint16_t field_idx; |
| 349 | uintptr_t declaring_dex_file; |
| 350 | uint16_t declaring_class_idx; |
| 351 | uint16_t declaring_field_idx; |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 352 | DexMemAccessType type; |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 353 | }; |
| 354 | |
| 355 | void DoPrepareSFields(const SFieldDef* defs, size_t count) { |
| 356 | cu_.mir_graph->sfield_lowering_infos_.clear(); |
| 357 | cu_.mir_graph->sfield_lowering_infos_.reserve(count); |
| 358 | for (size_t i = 0u; i != count; ++i) { |
| 359 | const SFieldDef* def = &defs[i]; |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 360 | MirSFieldLoweringInfo field_info(def->field_idx, def->type); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 361 | if (def->declaring_dex_file != 0u) { |
| 362 | field_info.declaring_dex_file_ = reinterpret_cast<const DexFile*>(def->declaring_dex_file); |
| 363 | field_info.declaring_class_idx_ = def->declaring_class_idx; |
| 364 | field_info.declaring_field_idx_ = def->declaring_field_idx; |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 365 | // We don't care about the volatile flag in these tests. |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 366 | } |
| 367 | ASSERT_EQ(def->declaring_dex_file != 0u, field_info.IsResolved()); |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 368 | ASSERT_FALSE(field_info.IsClassInitialized()); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 369 | cu_.mir_graph->sfield_lowering_infos_.push_back(field_info); |
| 370 | } |
| 371 | } |
| 372 | |
| 373 | template <size_t count> |
| 374 | void PrepareSFields(const SFieldDef (&defs)[count]) { |
| 375 | DoPrepareSFields(defs, count); |
| 376 | } |
| 377 | |
| 378 | void PerformClassInitCheckElimination() { |
| 379 | cu_.mir_graph->ComputeDFSOrders(); |
| 380 | bool gate_result = cu_.mir_graph->EliminateClassInitChecksGate(); |
| 381 | ASSERT_TRUE(gate_result); |
| 382 | RepeatingPreOrderDfsIterator iterator(cu_.mir_graph.get()); |
| 383 | bool change = false; |
| 384 | for (BasicBlock* bb = iterator.Next(change); bb != nullptr; bb = iterator.Next(change)) { |
| 385 | change = cu_.mir_graph->EliminateClassInitChecks(bb); |
| 386 | } |
| 387 | cu_.mir_graph->EliminateClassInitChecksEnd(); |
| 388 | } |
| 389 | |
| 390 | ClassInitCheckEliminationTest() |
| 391 | : MirOptimizationTest() { |
| 392 | } |
| 393 | }; |
| 394 | |
| 395 | class NullCheckEliminationTest : public MirOptimizationTest { |
| 396 | protected: |
| 397 | struct IFieldDef { |
| 398 | uint16_t field_idx; |
| 399 | uintptr_t declaring_dex_file; |
| 400 | uint16_t declaring_class_idx; |
| 401 | uint16_t declaring_field_idx; |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 402 | DexMemAccessType type; |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 403 | }; |
| 404 | |
| 405 | void DoPrepareIFields(const IFieldDef* defs, size_t count) { |
| 406 | cu_.mir_graph->ifield_lowering_infos_.clear(); |
| 407 | cu_.mir_graph->ifield_lowering_infos_.reserve(count); |
| 408 | for (size_t i = 0u; i != count; ++i) { |
| 409 | const IFieldDef* def = &defs[i]; |
Mathieu Chartier | 2535abe | 2015-02-17 10:38:49 -0800 | [diff] [blame] | 410 | MirIFieldLoweringInfo field_info(def->field_idx, def->type, false); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 411 | if (def->declaring_dex_file != 0u) { |
| 412 | field_info.declaring_dex_file_ = reinterpret_cast<const DexFile*>(def->declaring_dex_file); |
| 413 | field_info.declaring_class_idx_ = def->declaring_class_idx; |
| 414 | field_info.declaring_field_idx_ = def->declaring_field_idx; |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 415 | // We don't care about the volatile flag in these tests. |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 416 | } |
| 417 | ASSERT_EQ(def->declaring_dex_file != 0u, field_info.IsResolved()); |
| 418 | cu_.mir_graph->ifield_lowering_infos_.push_back(field_info); |
| 419 | } |
| 420 | } |
| 421 | |
| 422 | template <size_t count> |
| 423 | void PrepareIFields(const IFieldDef (&defs)[count]) { |
| 424 | DoPrepareIFields(defs, count); |
| 425 | } |
| 426 | |
| 427 | void PerformNullCheckElimination() { |
| 428 | // Make vregs in range [100, 1000) input registers, i.e. requiring a null check. |
| 429 | code_item_->registers_size_ = 1000; |
| 430 | code_item_->ins_size_ = 900; |
| 431 | |
| 432 | cu_.mir_graph->ComputeDFSOrders(); |
| 433 | bool gate_result = cu_.mir_graph->EliminateNullChecksGate(); |
| 434 | ASSERT_TRUE(gate_result); |
| 435 | RepeatingPreOrderDfsIterator iterator(cu_.mir_graph.get()); |
| 436 | bool change = false; |
| 437 | for (BasicBlock* bb = iterator.Next(change); bb != nullptr; bb = iterator.Next(change)) { |
| 438 | change = cu_.mir_graph->EliminateNullChecks(bb); |
| 439 | } |
| 440 | cu_.mir_graph->EliminateNullChecksEnd(); |
| 441 | } |
| 442 | |
| 443 | NullCheckEliminationTest() |
| 444 | : MirOptimizationTest() { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 445 | static const MethodDef methods[] = { |
| 446 | { 0u, 1u, 0u, 0u, kDirect, kDirect, false, false }, // Dummy. |
| 447 | }; |
| 448 | PrepareMethods(methods); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 449 | } |
| 450 | }; |
| 451 | |
Vladimir Marko | 8b858e1 | 2014-11-27 14:52:37 +0000 | [diff] [blame] | 452 | class SuspendCheckEliminationTest : public MirOptimizationTest { |
| 453 | protected: |
| 454 | bool IsBackEdge(BasicBlockId branch_bb, BasicBlockId target_bb) { |
| 455 | BasicBlock* branch = cu_.mir_graph->GetBasicBlock(branch_bb); |
| 456 | return target_bb != NullBasicBlockId && cu_.mir_graph->IsBackEdge(branch, target_bb); |
| 457 | } |
| 458 | |
| 459 | bool IsSuspendCheckEdge(BasicBlockId branch_bb, BasicBlockId target_bb) { |
| 460 | BasicBlock* branch = cu_.mir_graph->GetBasicBlock(branch_bb); |
| 461 | return cu_.mir_graph->IsSuspendCheckEdge(branch, target_bb); |
| 462 | } |
| 463 | |
| 464 | void PerformSuspendCheckElimination() { |
| 465 | cu_.mir_graph->SSATransformationStart(); |
| 466 | cu_.mir_graph->ComputeDFSOrders(); |
| 467 | cu_.mir_graph->ComputeDominators(); |
| 468 | cu_.mir_graph->ComputeTopologicalSortOrder(); |
| 469 | cu_.mir_graph->SSATransformationEnd(); |
| 470 | bool gate_result = cu_.mir_graph->EliminateSuspendChecksGate(); |
| 471 | ASSERT_TRUE(gate_result); |
| 472 | TopologicalSortIterator iterator(cu_.mir_graph.get()); |
| 473 | bool change = false; |
| 474 | for (BasicBlock* bb = iterator.Next(change); bb != nullptr; bb = iterator.Next(change)) { |
| 475 | change = cu_.mir_graph->EliminateSuspendChecks(bb); |
| 476 | } |
| 477 | cu_.mir_graph->EliminateSuspendChecksEnd(); |
| 478 | } |
| 479 | |
| 480 | SuspendCheckEliminationTest() |
| 481 | : MirOptimizationTest() { |
| 482 | static const MethodDef methods[] = { |
| 483 | { 0u, 1u, 0u, 0u, kDirect, kDirect, false, false }, // Dummy. |
| 484 | }; |
| 485 | PrepareMethods(methods); |
| 486 | } |
| 487 | }; |
| 488 | |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 489 | TEST_F(ClassInitCheckEliminationTest, SingleBlock) { |
| 490 | static const SFieldDef sfields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 491 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 492 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
| 493 | { 2u, 1u, 2u, 2u, kDexMemAccessWord }, |
| 494 | { 3u, 1u, 3u, 3u, kDexMemAccessWord }, // Same declaring class as sfield[4]. |
| 495 | { 4u, 1u, 3u, 4u, kDexMemAccessWord }, // Same declaring class as sfield[3]. |
| 496 | { 5u, 0u, 0u, 0u, kDexMemAccessWord }, // Unresolved. |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 497 | }; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 498 | static const MIRDef mirs[] = { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 499 | DEF_SGET_SPUT(3u, Instruction::SPUT, 0u, 5u), // Unresolved. |
| 500 | DEF_SGET_SPUT(3u, Instruction::SPUT, 0u, 0u), |
| 501 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 1u), |
| 502 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 2u), |
| 503 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 5u), // Unresolved. |
| 504 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 0u), |
| 505 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 1u), |
| 506 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 2u), |
| 507 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 5u), // Unresolved. |
| 508 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 3u), |
| 509 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 4u), |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 510 | }; |
| 511 | static const bool expected_ignore_clinit_check[] = { |
Vladimir Marko | f418f32 | 2014-07-09 14:45:36 +0100 | [diff] [blame] | 512 | false, false, false, false, true, true, true, true, true, false, true |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 513 | }; |
| 514 | |
| 515 | PrepareSFields(sfields); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 516 | PrepareSingleBlock(); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 517 | PrepareMIRs(mirs); |
| 518 | PerformClassInitCheckElimination(); |
| 519 | ASSERT_EQ(arraysize(expected_ignore_clinit_check), mir_count_); |
| 520 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 521 | EXPECT_EQ(expected_ignore_clinit_check[i], |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 522 | (mirs_[i].optimization_flags & MIR_CLASS_IS_INITIALIZED) != 0) << i; |
| 523 | EXPECT_EQ(expected_ignore_clinit_check[i], |
| 524 | (mirs_[i].optimization_flags & MIR_CLASS_IS_IN_DEX_CACHE) != 0) << i; |
| 525 | } |
| 526 | } |
| 527 | |
| 528 | TEST_F(ClassInitCheckEliminationTest, SingleBlockWithInvokes) { |
| 529 | static const SFieldDef sfields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 530 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 531 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
| 532 | { 2u, 1u, 2u, 2u, kDexMemAccessWord }, |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 533 | }; |
| 534 | static const MethodDef methods[] = { |
| 535 | { 0u, 1u, 0u, 0u, kStatic, kStatic, false, false }, |
| 536 | { 1u, 1u, 1u, 1u, kStatic, kStatic, false, false }, |
| 537 | { 2u, 1u, 2u, 2u, kStatic, kStatic, false, false }, |
| 538 | }; |
| 539 | static const MIRDef mirs[] = { |
| 540 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 0u), |
| 541 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u /* dummy */, 0u), |
| 542 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u /* dummy */, 1u), |
| 543 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 1u), |
| 544 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u /* dummy */, 2u), |
| 545 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u /* dummy */, 2u), |
| 546 | }; |
| 547 | static const bool expected_class_initialized[] = { |
| 548 | false, true, false, true, false, true |
| 549 | }; |
| 550 | static const bool expected_class_in_dex_cache[] = { |
| 551 | false, false, false, false, false, false |
| 552 | }; |
| 553 | |
| 554 | PrepareSFields(sfields); |
| 555 | PrepareMethods(methods); |
| 556 | PrepareSingleBlock(); |
| 557 | PrepareMIRs(mirs); |
| 558 | PerformClassInitCheckElimination(); |
| 559 | ASSERT_EQ(arraysize(expected_class_initialized), mir_count_); |
| 560 | ASSERT_EQ(arraysize(expected_class_in_dex_cache), mir_count_); |
| 561 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 562 | EXPECT_EQ(expected_class_initialized[i], |
| 563 | (mirs_[i].optimization_flags & MIR_CLASS_IS_INITIALIZED) != 0) << i; |
| 564 | EXPECT_EQ(expected_class_in_dex_cache[i], |
| 565 | (mirs_[i].optimization_flags & MIR_CLASS_IS_IN_DEX_CACHE) != 0) << i; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 566 | } |
| 567 | } |
| 568 | |
| 569 | TEST_F(ClassInitCheckEliminationTest, Diamond) { |
| 570 | static const SFieldDef sfields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 571 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 572 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
| 573 | { 2u, 1u, 2u, 2u, kDexMemAccessWord }, |
| 574 | { 3u, 1u, 3u, 3u, kDexMemAccessWord }, |
| 575 | { 4u, 1u, 4u, 4u, kDexMemAccessWord }, |
| 576 | { 5u, 1u, 5u, 5u, kDexMemAccessWord }, |
| 577 | { 6u, 1u, 6u, 6u, kDexMemAccessWord }, |
| 578 | { 7u, 1u, 7u, 7u, kDexMemAccessWord }, |
| 579 | { 8u, 1u, 8u, 8u, kDexMemAccessWord }, // Same declaring class as sfield[9]. |
| 580 | { 9u, 1u, 8u, 9u, kDexMemAccessWord }, // Same declaring class as sfield[8]. |
| 581 | { 10u, 0u, 0u, 0u, kDexMemAccessWord }, // Unresolved. |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 582 | }; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 583 | static const MIRDef mirs[] = { |
| 584 | // NOTE: MIRs here are ordered by unique tests. They will be put into appropriate blocks. |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 585 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 10u), // Unresolved. |
| 586 | DEF_SGET_SPUT(3u, Instruction::SPUT, 0u, 10u), // Unresolved. |
| 587 | DEF_SGET_SPUT(3u, Instruction::SPUT, 0u, 0u), |
| 588 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 0u), // Eliminated (BB #3 dominates #6). |
| 589 | DEF_SGET_SPUT(4u, Instruction::SPUT, 0u, 1u), |
| 590 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 1u), // Not eliminated (BB #4 doesn't dominate #6). |
| 591 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 2u), |
| 592 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 2u), // Eliminated (BB #3 dominates #4). |
| 593 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 3u), |
| 594 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 3u), // Eliminated (BB #3 dominates #5). |
| 595 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 4u), |
| 596 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 4u), // Eliminated (BB #3 dominates #6). |
| 597 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 5u), |
| 598 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 5u), // Not eliminated (BB #4 doesn't dominate #6). |
| 599 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 6u), |
| 600 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 6u), // Not eliminated (BB #5 doesn't dominate #6). |
| 601 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 7u), |
| 602 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 7u), |
| 603 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 7u), // Eliminated (initialized in both #3 and #4). |
| 604 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 8u), |
| 605 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 9u), |
| 606 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 8u), // Eliminated (with sfield[9] in BB #5). |
| 607 | DEF_SGET_SPUT(6u, Instruction::SPUT, 0u, 9u), // Eliminated (with sfield[8] in BB #4). |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 608 | }; |
| 609 | static const bool expected_ignore_clinit_check[] = { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 610 | false, true, // Unresolved: sfield[10] |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 611 | false, true, // sfield[0] |
| 612 | false, false, // sfield[1] |
| 613 | false, true, // sfield[2] |
| 614 | false, true, // sfield[3] |
| 615 | false, true, // sfield[4] |
| 616 | false, false, // sfield[5] |
| 617 | false, false, // sfield[6] |
| 618 | false, false, true, // sfield[7] |
| 619 | false, false, true, true, // sfield[8], sfield[9] |
| 620 | }; |
| 621 | |
| 622 | PrepareSFields(sfields); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 623 | PrepareDiamond(); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 624 | PrepareMIRs(mirs); |
| 625 | PerformClassInitCheckElimination(); |
| 626 | ASSERT_EQ(arraysize(expected_ignore_clinit_check), mir_count_); |
| 627 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 628 | EXPECT_EQ(expected_ignore_clinit_check[i], |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 629 | (mirs_[i].optimization_flags & MIR_CLASS_IS_INITIALIZED) != 0) << i; |
| 630 | EXPECT_EQ(expected_ignore_clinit_check[i], |
| 631 | (mirs_[i].optimization_flags & MIR_CLASS_IS_IN_DEX_CACHE) != 0) << i; |
| 632 | } |
| 633 | } |
| 634 | |
| 635 | TEST_F(ClassInitCheckEliminationTest, DiamondWithInvokes) { |
| 636 | static const SFieldDef sfields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 637 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 638 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
| 639 | { 2u, 1u, 2u, 2u, kDexMemAccessWord }, |
| 640 | { 3u, 1u, 3u, 3u, kDexMemAccessWord }, |
| 641 | { 4u, 1u, 4u, 4u, kDexMemAccessWord }, |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 642 | }; |
| 643 | static const MethodDef methods[] = { |
| 644 | { 0u, 1u, 0u, 0u, kStatic, kStatic, false, false }, |
| 645 | { 1u, 1u, 1u, 1u, kStatic, kStatic, false, false }, |
| 646 | { 2u, 1u, 2u, 2u, kStatic, kStatic, false, false }, |
| 647 | { 3u, 1u, 3u, 3u, kStatic, kStatic, false, false }, |
| 648 | { 4u, 1u, 4u, 4u, kStatic, kStatic, false, false }, |
| 649 | }; |
| 650 | static const MIRDef mirs[] = { |
| 651 | // NOTE: MIRs here are ordered by unique tests. They will be put into appropriate blocks. |
| 652 | DEF_SGET_SPUT(3u, Instruction::SPUT, 0u, 0u), |
| 653 | DEF_INVOKE(6u, Instruction::INVOKE_STATIC, 0u /* dummy */, 0u), |
| 654 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u /* dummy */, 1u), |
| 655 | DEF_SGET_SPUT(6u, Instruction::SPUT, 0u, 1u), |
| 656 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 2u), |
| 657 | DEF_INVOKE(5u, Instruction::INVOKE_STATIC, 0u /* dummy */, 2u), |
| 658 | DEF_SGET_SPUT(6u, Instruction::SPUT, 0u, 2u), |
| 659 | DEF_INVOKE(4u, Instruction::INVOKE_STATIC, 0u /* dummy */, 3u), |
| 660 | DEF_SGET_SPUT(5u, Instruction::SPUT, 0u, 3u), |
| 661 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 3u), |
| 662 | DEF_SGET_SPUT(4u, Instruction::SPUT, 0u, 4u), |
| 663 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 4u), |
| 664 | DEF_INVOKE(6u, Instruction::INVOKE_STATIC, 0u /* dummy */, 4u), |
| 665 | }; |
| 666 | static const bool expected_class_initialized[] = { |
| 667 | false, true, // BB #3 SPUT, BB#6 INVOKE_STATIC |
| 668 | false, true, // BB #3 INVOKE_STATIC, BB#6 SPUT |
| 669 | false, false, true, // BB #4 SGET, BB #5 INVOKE_STATIC, BB #6 SPUT |
| 670 | false, false, true, // BB #4 INVOKE_STATIC, BB #5 SPUT, BB #6 SGET |
| 671 | false, false, true, // BB #4 SPUT, BB #5 SGET, BB #6 INVOKE_STATIC |
| 672 | }; |
| 673 | static const bool expected_class_in_dex_cache[] = { |
| 674 | false, false, // BB #3 SPUT, BB#6 INVOKE_STATIC |
| 675 | false, false, // BB #3 INVOKE_STATIC, BB#6 SPUT |
| 676 | false, false, false, // BB #4 SGET, BB #5 INVOKE_STATIC, BB #6 SPUT |
| 677 | false, false, false, // BB #4 INVOKE_STATIC, BB #5 SPUT, BB #6 SGET |
| 678 | false, false, false, // BB #4 SPUT, BB #5 SGET, BB #6 INVOKE_STATIC |
| 679 | }; |
| 680 | |
| 681 | PrepareSFields(sfields); |
| 682 | PrepareMethods(methods); |
| 683 | PrepareDiamond(); |
| 684 | PrepareMIRs(mirs); |
| 685 | PerformClassInitCheckElimination(); |
| 686 | ASSERT_EQ(arraysize(expected_class_initialized), mir_count_); |
| 687 | ASSERT_EQ(arraysize(expected_class_in_dex_cache), mir_count_); |
| 688 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 689 | EXPECT_EQ(expected_class_initialized[i], |
| 690 | (mirs_[i].optimization_flags & MIR_CLASS_IS_INITIALIZED) != 0) << i; |
| 691 | EXPECT_EQ(expected_class_in_dex_cache[i], |
| 692 | (mirs_[i].optimization_flags & MIR_CLASS_IS_IN_DEX_CACHE) != 0) << i; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 693 | } |
| 694 | } |
| 695 | |
| 696 | TEST_F(ClassInitCheckEliminationTest, Loop) { |
| 697 | static const SFieldDef sfields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 698 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 699 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
| 700 | { 2u, 1u, 2u, 2u, kDexMemAccessWord }, |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 701 | }; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 702 | static const MIRDef mirs[] = { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 703 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 0u), |
| 704 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 0u), // Eliminated. |
| 705 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 1u), |
| 706 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 1u), // Eliminated. |
| 707 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 2u), |
| 708 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 2u), // Eliminated. |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 709 | }; |
| 710 | static const bool expected_ignore_clinit_check[] = { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 711 | false, true, false, true, false, true, |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 712 | }; |
| 713 | |
| 714 | PrepareSFields(sfields); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 715 | PrepareLoop(); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 716 | PrepareMIRs(mirs); |
| 717 | PerformClassInitCheckElimination(); |
| 718 | ASSERT_EQ(arraysize(expected_ignore_clinit_check), mir_count_); |
| 719 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 720 | EXPECT_EQ(expected_ignore_clinit_check[i], |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 721 | (mirs_[i].optimization_flags & MIR_CLASS_IS_INITIALIZED) != 0) << i; |
| 722 | EXPECT_EQ(expected_ignore_clinit_check[i], |
| 723 | (mirs_[i].optimization_flags & MIR_CLASS_IS_IN_DEX_CACHE) != 0) << i; |
| 724 | } |
| 725 | } |
| 726 | |
| 727 | TEST_F(ClassInitCheckEliminationTest, LoopWithInvokes) { |
| 728 | static const SFieldDef sfields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 729 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 730 | }; |
| 731 | static const MethodDef methods[] = { |
| 732 | { 0u, 1u, 0u, 0u, kStatic, kStatic, false, false }, |
| 733 | { 1u, 1u, 1u, 1u, kStatic, kStatic, false, false }, |
| 734 | { 2u, 1u, 2u, 2u, kStatic, kStatic, false, false }, |
| 735 | }; |
| 736 | static const MIRDef mirs[] = { |
| 737 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u /* dummy */, 0u), |
| 738 | DEF_INVOKE(4u, Instruction::INVOKE_STATIC, 0u /* dummy */, 0u), |
| 739 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u /* dummy */, 1u), |
| 740 | DEF_INVOKE(5u, Instruction::INVOKE_STATIC, 0u /* dummy */, 1u), |
| 741 | DEF_INVOKE(4u, Instruction::INVOKE_STATIC, 0u /* dummy */, 2u), |
| 742 | DEF_INVOKE(5u, Instruction::INVOKE_STATIC, 0u /* dummy */, 2u), |
| 743 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 0u), |
| 744 | }; |
| 745 | static const bool expected_class_initialized[] = { |
| 746 | false, true, false, true, false, true, true, |
| 747 | }; |
| 748 | static const bool expected_class_in_dex_cache[] = { |
| 749 | false, false, false, false, false, false, false, |
| 750 | }; |
| 751 | |
| 752 | PrepareSFields(sfields); |
| 753 | PrepareMethods(methods); |
| 754 | PrepareLoop(); |
| 755 | PrepareMIRs(mirs); |
| 756 | PerformClassInitCheckElimination(); |
| 757 | ASSERT_EQ(arraysize(expected_class_initialized), mir_count_); |
| 758 | ASSERT_EQ(arraysize(expected_class_in_dex_cache), mir_count_); |
| 759 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 760 | EXPECT_EQ(expected_class_initialized[i], |
| 761 | (mirs_[i].optimization_flags & MIR_CLASS_IS_INITIALIZED) != 0) << i; |
| 762 | EXPECT_EQ(expected_class_in_dex_cache[i], |
| 763 | (mirs_[i].optimization_flags & MIR_CLASS_IS_IN_DEX_CACHE) != 0) << i; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 764 | } |
| 765 | } |
| 766 | |
| 767 | TEST_F(ClassInitCheckEliminationTest, Catch) { |
| 768 | static const SFieldDef sfields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 769 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 770 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
| 771 | { 2u, 1u, 2u, 2u, kDexMemAccessWord }, |
| 772 | { 3u, 1u, 3u, 3u, kDexMemAccessWord }, |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 773 | }; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 774 | static const MIRDef mirs[] = { |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 775 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 0u), // Before the exception edge. |
| 776 | DEF_SGET_SPUT(3u, Instruction::SGET, 0u, 1u), // Before the exception edge. |
| 777 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 2u), // After the exception edge. |
| 778 | DEF_SGET_SPUT(4u, Instruction::SGET, 0u, 3u), // After the exception edge. |
| 779 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 0u), // In catch handler; eliminated. |
| 780 | DEF_SGET_SPUT(5u, Instruction::SGET, 0u, 2u), // In catch handler; not eliminated. |
| 781 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 0u), // Class init check eliminated. |
| 782 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 1u), // Class init check eliminated. |
| 783 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 2u), // Class init check eliminated. |
| 784 | DEF_SGET_SPUT(6u, Instruction::SGET, 0u, 3u), // Class init check not eliminated. |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 785 | }; |
| 786 | static const bool expected_ignore_clinit_check[] = { |
Vladimir Marko | 0a810d2 | 2014-07-11 14:44:36 +0100 | [diff] [blame] | 787 | false, false, false, false, true, false, true, true, true, false |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 788 | }; |
| 789 | |
| 790 | PrepareSFields(sfields); |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 791 | PrepareCatch(); |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 792 | PrepareMIRs(mirs); |
| 793 | PerformClassInitCheckElimination(); |
| 794 | ASSERT_EQ(arraysize(expected_ignore_clinit_check), mir_count_); |
| 795 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 796 | EXPECT_EQ(expected_ignore_clinit_check[i], |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 797 | (mirs_[i].optimization_flags & MIR_CLASS_IS_INITIALIZED) != 0) << i; |
| 798 | EXPECT_EQ(expected_ignore_clinit_check[i], |
| 799 | (mirs_[i].optimization_flags & MIR_CLASS_IS_IN_DEX_CACHE) != 0) << i; |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 800 | } |
| 801 | } |
| 802 | |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 803 | TEST_F(NullCheckEliminationTest, SingleBlock) { |
| 804 | static const IFieldDef ifields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 805 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 806 | { 1u, 1u, 0u, 1u, kDexMemAccessWord }, |
| 807 | { 2u, 1u, 0u, 2u, kDexMemAccessObject }, |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 808 | }; |
| 809 | static const MIRDef mirs[] = { |
| 810 | DEF_IGET_IPUT(3u, Instruction::IGET_OBJECT, 0u, 100u, 2u), |
| 811 | DEF_IGET_IPUT(3u, Instruction::IGET, 1u, 0u, 1u), |
| 812 | DEF_IGET_IPUT(3u, Instruction::IGET_OBJECT, 2u, 100u, 2u), // Differs from 0u (no LVN here). |
| 813 | DEF_IGET_IPUT(3u, Instruction::IGET, 3u, 2u, 1u), |
| 814 | DEF_IGET_IPUT(3u, Instruction::IGET, 4u, 101u, 0u), |
| 815 | DEF_IGET_IPUT(3u, Instruction::IGET, 5u, 102u, 0u), |
| 816 | DEF_IGET_IPUT(3u, Instruction::IGET, 6u, 103u, 0u), |
| 817 | DEF_IGET_IPUT(3u, Instruction::IGET, 7u, 103u, 1u), |
| 818 | DEF_IGET_IPUT(3u, Instruction::IPUT, 8u, 104u, 0u), |
| 819 | DEF_IGET_IPUT(3u, Instruction::IPUT, 9u, 104u, 1u), |
| 820 | DEF_IGET_IPUT(3u, Instruction::IGET, 10u, 105u, 0u), |
| 821 | DEF_IGET_IPUT(3u, Instruction::IPUT, 11u, 105u, 1u), |
| 822 | DEF_IGET_IPUT(3u, Instruction::IPUT, 12u, 106u, 0u), |
| 823 | DEF_IGET_IPUT(3u, Instruction::IGET, 13u, 106u, 1u), |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 824 | DEF_INVOKE(3u, Instruction::INVOKE_DIRECT, 107, 0u /* dummy */), |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 825 | DEF_IGET_IPUT(3u, Instruction::IGET, 15u, 107u, 1u), |
| 826 | DEF_IGET_IPUT(3u, Instruction::IGET, 16u, 108u, 0u), |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 827 | DEF_INVOKE(3u, Instruction::INVOKE_DIRECT, 108, 0u /* dummy */), |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 828 | DEF_AGET_APUT(3u, Instruction::AGET, 18u, 109u, 110u), |
| 829 | DEF_AGET_APUT(3u, Instruction::APUT, 19u, 109u, 111u), |
| 830 | DEF_OTHER2(3u, Instruction::ARRAY_LENGTH, 20u, 112u), |
| 831 | DEF_AGET_APUT(3u, Instruction::AGET, 21u, 112u, 113u), |
| 832 | DEF_OTHER1(3u, Instruction::MONITOR_ENTER, 114u), |
| 833 | DEF_OTHER1(3u, Instruction::MONITOR_EXIT, 114u), |
| 834 | }; |
| 835 | static const bool expected_ignore_null_check[] = { |
| 836 | false, false, true, false /* Not doing LVN. */, |
| 837 | false, true /* Set before running NCE. */, |
| 838 | false, true, // IGET, IGET |
| 839 | false, true, // IPUT, IPUT |
| 840 | false, true, // IGET, IPUT |
| 841 | false, true, // IPUT, IGET |
| 842 | false, true, // INVOKE, IGET |
| 843 | false, true, // IGET, INVOKE |
| 844 | false, true, // AGET, APUT |
| 845 | false, true, // ARRAY_LENGTH, AGET |
| 846 | false, true, // MONITOR_ENTER, MONITOR_EXIT |
| 847 | }; |
| 848 | |
| 849 | PrepareIFields(ifields); |
| 850 | PrepareSingleBlock(); |
| 851 | PrepareMIRs(mirs); |
| 852 | |
| 853 | // Mark IGET 5u as null-checked to test that NCE doesn't clear this flag. |
| 854 | mirs_[5u].optimization_flags |= MIR_IGNORE_NULL_CHECK; |
| 855 | |
| 856 | PerformNullCheckElimination(); |
| 857 | ASSERT_EQ(arraysize(expected_ignore_null_check), mir_count_); |
| 858 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 859 | EXPECT_EQ(expected_ignore_null_check[i], |
| 860 | (mirs_[i].optimization_flags & MIR_IGNORE_NULL_CHECK) != 0) << i; |
| 861 | } |
| 862 | } |
| 863 | |
| 864 | TEST_F(NullCheckEliminationTest, Diamond) { |
| 865 | static const IFieldDef ifields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 866 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 867 | { 1u, 1u, 0u, 1u, kDexMemAccessWord }, |
| 868 | { 2u, 1u, 0u, 2u, kDexMemAccessObject }, // int[]. |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 869 | }; |
| 870 | static const MIRDef mirs[] = { |
| 871 | // NOTE: MIRs here are ordered by unique tests. They will be put into appropriate blocks. |
| 872 | DEF_IGET_IPUT(3u, Instruction::IPUT, 0u, 100u, 0u), |
| 873 | DEF_IGET_IPUT(6u, Instruction::IGET, 1u, 100u, 1u), // Eliminated (BB #3 dominates #6). |
| 874 | DEF_IGET_IPUT(3u, Instruction::IGET, 2u, 101u, 0u), |
| 875 | DEF_IGET_IPUT(4u, Instruction::IPUT, 3u, 101u, 0u), // Eliminated (BB #3 dominates #4). |
| 876 | DEF_IGET_IPUT(3u, Instruction::IGET, 4u, 102u, 0u), |
| 877 | DEF_IGET_IPUT(5u, Instruction::IPUT, 5u, 102u, 1u), // Eliminated (BB #3 dominates #5). |
| 878 | DEF_IGET_IPUT(4u, Instruction::IPUT, 6u, 103u, 0u), |
| 879 | DEF_IGET_IPUT(6u, Instruction::IPUT, 7u, 103u, 1u), // Not eliminated (going through BB #5). |
| 880 | DEF_IGET_IPUT(5u, Instruction::IGET, 8u, 104u, 1u), |
| 881 | DEF_IGET_IPUT(6u, Instruction::IGET, 9u, 104u, 0u), // Not eliminated (going through BB #4). |
Vladimir Marko | 66c6d7b | 2014-10-16 15:41:48 +0100 | [diff] [blame] | 882 | DEF_INVOKE(4u, Instruction::INVOKE_DIRECT, 105u, 0u /* dummy */), |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 883 | DEF_IGET_IPUT(5u, Instruction::IGET, 11u, 105u, 1u), |
| 884 | DEF_IGET_IPUT(6u, Instruction::IPUT, 12u, 105u, 0u), // Eliminated. |
| 885 | DEF_IGET_IPUT(3u, Instruction::IGET_OBJECT, 13u, 106u, 2u), |
| 886 | DEF_OTHER1(3u, Instruction::IF_EQZ, 13u), // Last insn in the BB #3. |
| 887 | DEF_OTHER2(5u, Instruction::NEW_ARRAY, 13u, 107u), |
| 888 | DEF_AGET_APUT(6u, Instruction::AGET, 16u, 13u, 108u), // Eliminated. |
| 889 | }; |
| 890 | static const bool expected_ignore_null_check[] = { |
| 891 | false, true, // BB #3 IPUT, BB #6 IGET |
| 892 | false, true, // BB #3 IGET, BB #4 IPUT |
| 893 | false, true, // BB #3 IGET, BB #5 IPUT |
| 894 | false, false, // BB #4 IPUT, BB #6 IPUT |
| 895 | false, false, // BB #5 IGET, BB #6 IGET |
| 896 | false, false, true, // BB #4 INVOKE, BB #5 IGET, BB #6 IPUT |
| 897 | false, false, // BB #3 IGET_OBJECT & IF_EQZ |
| 898 | false, true, // BB #5 NEW_ARRAY, BB #6 AGET |
| 899 | }; |
| 900 | |
| 901 | PrepareIFields(ifields); |
| 902 | PrepareDiamond(); |
| 903 | PrepareMIRs(mirs); |
| 904 | PerformNullCheckElimination(); |
| 905 | ASSERT_EQ(arraysize(expected_ignore_null_check), mir_count_); |
| 906 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 907 | EXPECT_EQ(expected_ignore_null_check[i], |
| 908 | (mirs_[i].optimization_flags & MIR_IGNORE_NULL_CHECK) != 0) << i; |
| 909 | } |
| 910 | } |
| 911 | |
| 912 | TEST_F(NullCheckEliminationTest, Loop) { |
| 913 | static const IFieldDef ifields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 914 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 915 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 916 | }; |
| 917 | static const MIRDef mirs[] = { |
| 918 | DEF_IGET_IPUT(3u, Instruction::IGET, 0u, 100u, 0u), |
| 919 | DEF_IGET_IPUT(4u, Instruction::IGET, 1u, 101u, 0u), |
| 920 | DEF_IGET_IPUT(5u, Instruction::IGET, 2u, 100u, 1u), // Eliminated. |
| 921 | DEF_IGET_IPUT(5u, Instruction::IGET, 3u, 101u, 1u), // Eliminated. |
| 922 | DEF_IGET_IPUT(3u, Instruction::IGET, 4u, 102u, 0u), |
| 923 | DEF_IGET_IPUT(4u, Instruction::IGET, 5u, 102u, 1u), // Not eliminated (MOVE_OBJECT_16). |
| 924 | DEF_OTHER2(4u, Instruction::MOVE_OBJECT_16, 102u, 103u), |
| 925 | }; |
| 926 | static const bool expected_ignore_null_check[] = { |
| 927 | false, false, true, true, |
| 928 | false, false, false, |
| 929 | }; |
| 930 | |
| 931 | PrepareIFields(ifields); |
| 932 | PrepareLoop(); |
| 933 | PrepareMIRs(mirs); |
| 934 | PerformNullCheckElimination(); |
| 935 | ASSERT_EQ(arraysize(expected_ignore_null_check), mir_count_); |
| 936 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 937 | EXPECT_EQ(expected_ignore_null_check[i], |
| 938 | (mirs_[i].optimization_flags & MIR_IGNORE_NULL_CHECK) != 0) << i; |
| 939 | } |
| 940 | } |
| 941 | |
| 942 | TEST_F(NullCheckEliminationTest, Catch) { |
| 943 | static const IFieldDef ifields[] = { |
Vladimir Marko | af6925b | 2014-10-31 16:37:32 +0000 | [diff] [blame] | 944 | { 0u, 1u, 0u, 0u, kDexMemAccessWord }, |
| 945 | { 1u, 1u, 1u, 1u, kDexMemAccessWord }, |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 946 | }; |
| 947 | static const MIRDef mirs[] = { |
| 948 | DEF_IGET_IPUT(3u, Instruction::IGET, 0u, 100u, 0u), // Before the exception edge. |
| 949 | DEF_IGET_IPUT(3u, Instruction::IGET, 1u, 101u, 0u), // Before the exception edge. |
| 950 | DEF_IGET_IPUT(4u, Instruction::IGET, 2u, 102u, 0u), // After the exception edge. |
| 951 | DEF_IGET_IPUT(4u, Instruction::IGET, 3u, 103u, 0u), // After the exception edge. |
| 952 | DEF_IGET_IPUT(5u, Instruction::IGET, 4u, 100u, 1u), // In catch handler; eliminated. |
| 953 | DEF_IGET_IPUT(5u, Instruction::IGET, 5u, 102u, 1u), // In catch handler; not eliminated. |
| 954 | DEF_IGET_IPUT(6u, Instruction::IGET, 6u, 100u, 0u), // Null check eliminated. |
| 955 | DEF_IGET_IPUT(6u, Instruction::IGET, 6u, 101u, 1u), // Null check eliminated. |
| 956 | DEF_IGET_IPUT(6u, Instruction::IGET, 6u, 102u, 0u), // Null check eliminated. |
| 957 | DEF_IGET_IPUT(6u, Instruction::IGET, 6u, 103u, 1u), // Null check not eliminated. |
| 958 | }; |
| 959 | static const bool expected_ignore_null_check[] = { |
| 960 | false, false, false, false, true, false, true, true, true, false |
| 961 | }; |
| 962 | |
| 963 | PrepareIFields(ifields); |
| 964 | PrepareCatch(); |
| 965 | PrepareMIRs(mirs); |
| 966 | PerformNullCheckElimination(); |
| 967 | ASSERT_EQ(arraysize(expected_ignore_null_check), mir_count_); |
| 968 | for (size_t i = 0u; i != arraysize(mirs); ++i) { |
| 969 | EXPECT_EQ(expected_ignore_null_check[i], |
| 970 | (mirs_[i].optimization_flags & MIR_IGNORE_NULL_CHECK) != 0) << i; |
| 971 | } |
| 972 | } |
| 973 | |
Vladimir Marko | 8b858e1 | 2014-11-27 14:52:37 +0000 | [diff] [blame] | 974 | TEST_F(SuspendCheckEliminationTest, LoopNoElimination) { |
| 975 | static const MIRDef mirs[] = { |
| 976 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u, 0u), // Force the pass to run. |
| 977 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge back. |
| 978 | }; |
| 979 | |
| 980 | PrepareLoop(); |
| 981 | PrepareMIRs(mirs); |
| 982 | PerformSuspendCheckElimination(); |
| 983 | ASSERT_TRUE(IsBackEdge(4u, 4u)); |
| 984 | EXPECT_TRUE(IsSuspendCheckEdge(4u, 4u)); // Suspend point on loop to self. |
| 985 | } |
| 986 | |
| 987 | TEST_F(SuspendCheckEliminationTest, LoopElimination) { |
| 988 | static const MIRDef mirs[] = { |
| 989 | DEF_INVOKE(4u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in the loop. |
| 990 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge back. |
| 991 | }; |
| 992 | |
| 993 | PrepareLoop(); |
| 994 | PrepareMIRs(mirs); |
| 995 | PerformSuspendCheckElimination(); |
| 996 | ASSERT_TRUE(IsBackEdge(4u, 4u)); |
| 997 | EXPECT_FALSE(IsSuspendCheckEdge(4u, 4u)); // No suspend point on loop to self. |
| 998 | } |
| 999 | |
| 1000 | TEST_F(SuspendCheckEliminationTest, While_While_NoElimination) { |
| 1001 | static const MIRDef mirs[] = { |
| 1002 | DEF_INVOKE(3u, Instruction::INVOKE_STATIC, 0u, 0u), // Force the pass to run. |
| 1003 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1004 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop. |
| 1005 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1006 | DEF_OTHER0(7u, Instruction::GOTO), // Edge back to outer loop head. |
| 1007 | }; |
| 1008 | |
| 1009 | PrepareNestedLoopsWhile_While(); |
| 1010 | PrepareMIRs(mirs); |
| 1011 | PerformSuspendCheckElimination(); |
| 1012 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1013 | EXPECT_TRUE(IsSuspendCheckEdge(6u, 5u)); |
| 1014 | ASSERT_TRUE(IsBackEdge(7u, 4u)); |
| 1015 | EXPECT_TRUE(IsSuspendCheckEdge(7u, 4u)); |
| 1016 | } |
| 1017 | |
| 1018 | TEST_F(SuspendCheckEliminationTest, While_While_InvokeInOuterLoopHead) { |
| 1019 | static const MIRDef mirs[] = { |
| 1020 | DEF_INVOKE(4u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in outer loop head. |
| 1021 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1022 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop. |
| 1023 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1024 | DEF_OTHER0(7u, Instruction::GOTO), // Edge back to outer loop head. |
| 1025 | }; |
| 1026 | |
| 1027 | PrepareNestedLoopsWhile_While(); |
| 1028 | PrepareMIRs(mirs); |
| 1029 | PerformSuspendCheckElimination(); |
| 1030 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1031 | EXPECT_TRUE(IsSuspendCheckEdge(6u, 5u)); |
| 1032 | ASSERT_TRUE(IsBackEdge(7u, 4u)); |
| 1033 | EXPECT_FALSE(IsSuspendCheckEdge(7u, 4u)); |
| 1034 | } |
| 1035 | |
| 1036 | TEST_F(SuspendCheckEliminationTest, While_While_InvokeInOuterLoopBody) { |
| 1037 | static const MIRDef mirs[] = { |
| 1038 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1039 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop. |
| 1040 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1041 | DEF_INVOKE(7u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in outer loop body. |
| 1042 | DEF_OTHER0(7u, Instruction::GOTO), // Edge back to outer loop head. |
| 1043 | }; |
| 1044 | |
| 1045 | PrepareNestedLoopsWhile_While(); |
| 1046 | PrepareMIRs(mirs); |
| 1047 | PerformSuspendCheckElimination(); |
| 1048 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1049 | EXPECT_TRUE(IsSuspendCheckEdge(6u, 5u)); |
| 1050 | ASSERT_TRUE(IsBackEdge(7u, 4u)); |
| 1051 | EXPECT_FALSE(IsSuspendCheckEdge(7u, 4u)); |
| 1052 | } |
| 1053 | |
| 1054 | TEST_F(SuspendCheckEliminationTest, While_While_InvokeInInnerLoopHead) { |
| 1055 | static const MIRDef mirs[] = { |
| 1056 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1057 | DEF_INVOKE(5u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in inner loop head. |
| 1058 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop. |
| 1059 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1060 | DEF_OTHER0(7u, Instruction::GOTO), // Edge back to outer loop head. |
| 1061 | }; |
| 1062 | |
| 1063 | PrepareNestedLoopsWhile_While(); |
| 1064 | PrepareMIRs(mirs); |
| 1065 | PerformSuspendCheckElimination(); |
| 1066 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1067 | EXPECT_FALSE(IsSuspendCheckEdge(6u, 5u)); |
| 1068 | ASSERT_TRUE(IsBackEdge(7u, 4u)); |
| 1069 | EXPECT_FALSE(IsSuspendCheckEdge(7u, 4u)); |
| 1070 | } |
| 1071 | |
| 1072 | TEST_F(SuspendCheckEliminationTest, While_While_InvokeInInnerLoopBody) { |
| 1073 | static const MIRDef mirs[] = { |
| 1074 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1075 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop. |
| 1076 | DEF_INVOKE(6u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in inner loop body. |
| 1077 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1078 | DEF_OTHER0(7u, Instruction::GOTO), // Edge back to outer loop head. |
| 1079 | }; |
| 1080 | |
| 1081 | PrepareNestedLoopsWhile_While(); |
| 1082 | PrepareMIRs(mirs); |
| 1083 | PerformSuspendCheckElimination(); |
| 1084 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1085 | EXPECT_FALSE(IsSuspendCheckEdge(6u, 5u)); |
| 1086 | ASSERT_TRUE(IsBackEdge(7u, 4u)); |
| 1087 | EXPECT_TRUE(IsSuspendCheckEdge(7u, 4u)); |
| 1088 | } |
| 1089 | |
| 1090 | TEST_F(SuspendCheckEliminationTest, While_WhileWhile_InvokeInFirstInnerLoopHead) { |
| 1091 | static const MIRDef mirs[] = { |
| 1092 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1093 | DEF_INVOKE(5u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in first inner loop head. |
| 1094 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 1. |
| 1095 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1096 | DEF_OTHER1(7u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 2. |
| 1097 | DEF_OTHER0(8u, Instruction::GOTO), // Edge back to inner loop 2 head. |
| 1098 | DEF_OTHER0(9u, Instruction::GOTO), // Edge back to outer loop head. |
| 1099 | }; |
| 1100 | |
| 1101 | PrepareNestedLoopsWhile_WhileWhile(); |
| 1102 | PrepareMIRs(mirs); |
| 1103 | PerformSuspendCheckElimination(); |
| 1104 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1105 | EXPECT_FALSE(IsSuspendCheckEdge(6u, 5u)); |
| 1106 | ASSERT_TRUE(IsBackEdge(8u, 7u)); |
| 1107 | EXPECT_TRUE(IsSuspendCheckEdge(8u, 7u)); |
| 1108 | ASSERT_TRUE(IsBackEdge(9u, 4u)); |
| 1109 | EXPECT_FALSE(IsSuspendCheckEdge(9u, 4u)); |
| 1110 | } |
| 1111 | |
| 1112 | TEST_F(SuspendCheckEliminationTest, While_WhileWhile_InvokeInFirstInnerLoopBody) { |
| 1113 | static const MIRDef mirs[] = { |
| 1114 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1115 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 1. |
| 1116 | DEF_INVOKE(6u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in first inner loop body. |
| 1117 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1118 | DEF_OTHER1(7u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 2. |
| 1119 | DEF_OTHER0(8u, Instruction::GOTO), // Edge back to inner loop 2 head. |
| 1120 | DEF_OTHER0(9u, Instruction::GOTO), // Edge back to outer loop head. |
| 1121 | }; |
| 1122 | |
| 1123 | PrepareNestedLoopsWhile_WhileWhile(); |
| 1124 | PrepareMIRs(mirs); |
| 1125 | PerformSuspendCheckElimination(); |
| 1126 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1127 | EXPECT_FALSE(IsSuspendCheckEdge(6u, 5u)); |
| 1128 | ASSERT_TRUE(IsBackEdge(8u, 7u)); |
| 1129 | EXPECT_TRUE(IsSuspendCheckEdge(8u, 7u)); |
| 1130 | ASSERT_TRUE(IsBackEdge(9u, 4u)); |
| 1131 | EXPECT_TRUE(IsSuspendCheckEdge(9u, 4u)); |
| 1132 | } |
| 1133 | |
| 1134 | TEST_F(SuspendCheckEliminationTest, While_WhileWhile_WithExtraEdge_InvokeInFirstInnerLoopBody) { |
| 1135 | static const MIRDef mirs[] = { |
| 1136 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1137 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 1. |
| 1138 | DEF_INVOKE(6u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in first inner loop body. |
| 1139 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1140 | DEF_OTHER1(7u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 2. |
| 1141 | DEF_OTHER0(8u, Instruction::GOTO), // Edge back to inner loop 2 head. |
| 1142 | DEF_OTHER0(9u, Instruction::GOTO), // Edge back to outer loop head. |
| 1143 | }; |
| 1144 | |
| 1145 | PrepareNestedLoopsWhile_WhileWhile_WithExtraEdge(); |
| 1146 | PrepareMIRs(mirs); |
| 1147 | PerformSuspendCheckElimination(); |
| 1148 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1149 | EXPECT_FALSE(IsSuspendCheckEdge(6u, 5u)); |
| 1150 | ASSERT_TRUE(IsBackEdge(8u, 7u)); |
| 1151 | EXPECT_TRUE(IsSuspendCheckEdge(8u, 7u)); // Unaffected by the extra edge. |
| 1152 | ASSERT_TRUE(IsBackEdge(9u, 4u)); |
| 1153 | EXPECT_TRUE(IsSuspendCheckEdge(9u, 4u)); |
| 1154 | } |
| 1155 | |
| 1156 | TEST_F(SuspendCheckEliminationTest, While_WhileWhile_WithExtraEdge_InvokeInSecondInnerLoopHead) { |
| 1157 | static const MIRDef mirs[] = { |
| 1158 | DEF_OTHER1(4u, Instruction::IF_NEZ, 1u), // Edge out of outer loop. |
| 1159 | DEF_OTHER1(5u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 1. |
| 1160 | DEF_OTHER0(6u, Instruction::GOTO), // Edge back to inner loop head. |
| 1161 | DEF_INVOKE(7u, Instruction::INVOKE_STATIC, 0u, 0u), // Invoke in second inner loop head. |
| 1162 | DEF_OTHER1(7u, Instruction::IF_NEZ, 2u), // Edge out of inner loop 2. |
| 1163 | DEF_OTHER0(8u, Instruction::GOTO), // Edge back to inner loop 2 head. |
| 1164 | DEF_OTHER0(9u, Instruction::GOTO), // Edge back to outer loop head. |
| 1165 | }; |
| 1166 | |
| 1167 | PrepareNestedLoopsWhile_WhileWhile_WithExtraEdge(); |
| 1168 | PrepareMIRs(mirs); |
| 1169 | PerformSuspendCheckElimination(); |
| 1170 | ASSERT_TRUE(IsBackEdge(6u, 5u)); |
| 1171 | EXPECT_TRUE(IsSuspendCheckEdge(6u, 5u)); |
| 1172 | ASSERT_TRUE(IsBackEdge(8u, 7u)); |
| 1173 | EXPECT_FALSE(IsSuspendCheckEdge(8u, 7u)); // Unaffected by the extra edge. |
| 1174 | ASSERT_TRUE(IsBackEdge(9u, 4u)); |
| 1175 | EXPECT_FALSE(IsSuspendCheckEdge(9u, 4u)); |
| 1176 | } |
Vladimir Marko | 7baa6f8 | 2014-10-09 18:01:24 +0100 | [diff] [blame] | 1177 | |
Vladimir Marko | bfea9c2 | 2014-01-17 17:49:33 +0000 | [diff] [blame] | 1178 | } // namespace art |