Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +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 "inliner.h" |
| 18 | |
Mathieu Chartier | e401d14 | 2015-04-22 13:56:20 -0700 | [diff] [blame] | 19 | #include "art_method-inl.h" |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 20 | #include "builder.h" |
| 21 | #include "class_linker.h" |
| 22 | #include "constant_folding.h" |
| 23 | #include "dead_code_elimination.h" |
| 24 | #include "driver/compiler_driver-inl.h" |
Calin Juravle | ec74835 | 2015-07-29 13:52:12 +0100 | [diff] [blame] | 25 | #include "driver/compiler_options.h" |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 26 | #include "driver/dex_compilation_unit.h" |
| 27 | #include "instruction_simplifier.h" |
Scott Wakeling | d60a1af | 2015-07-22 14:32:44 +0100 | [diff] [blame] | 28 | #include "intrinsics.h" |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 29 | #include "mirror/class_loader.h" |
| 30 | #include "mirror/dex_cache.h" |
| 31 | #include "nodes.h" |
Nicolas Geoffray | 335005e | 2015-06-25 10:01:47 +0100 | [diff] [blame] | 32 | #include "optimizing_compiler.h" |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 33 | #include "reference_type_propagation.h" |
Nicolas Geoffray | 259136f | 2014-12-17 23:21:58 +0000 | [diff] [blame] | 34 | #include "register_allocator.h" |
Vladimir Marko | dc151b2 | 2015-10-15 18:02:30 +0100 | [diff] [blame] | 35 | #include "sharpening.h" |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 36 | #include "ssa_phi_elimination.h" |
| 37 | #include "scoped_thread_state_change.h" |
| 38 | #include "thread.h" |
Calin Juravle | f1c6d9e | 2015-04-13 18:42:21 +0100 | [diff] [blame] | 39 | #include "dex/verified_method.h" |
| 40 | #include "dex/verification_results.h" |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 41 | |
| 42 | namespace art { |
| 43 | |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 44 | static constexpr size_t kMaximumNumberOfHInstructions = 12; |
| 45 | |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 46 | void HInliner::Run() { |
Calin Juravle | 8f96df8 | 2015-07-29 15:58:48 +0100 | [diff] [blame] | 47 | const CompilerOptions& compiler_options = compiler_driver_->GetCompilerOptions(); |
| 48 | if ((compiler_options.GetInlineDepthLimit() == 0) |
| 49 | || (compiler_options.GetInlineMaxCodeUnits() == 0)) { |
| 50 | return; |
| 51 | } |
Nicolas Geoffray | e50b8d2 | 2015-03-13 08:57:42 +0000 | [diff] [blame] | 52 | if (graph_->IsDebuggable()) { |
| 53 | // For simplicity, we currently never inline when the graph is debuggable. This avoids |
| 54 | // doing some logic in the runtime to discover if a method could have been inlined. |
| 55 | return; |
| 56 | } |
Vladimir Marko | fa6b93c | 2015-09-15 10:15:55 +0100 | [diff] [blame] | 57 | const ArenaVector<HBasicBlock*>& blocks = graph_->GetReversePostOrder(); |
| 58 | DCHECK(!blocks.empty()); |
| 59 | HBasicBlock* next_block = blocks[0]; |
| 60 | for (size_t i = 0; i < blocks.size(); ++i) { |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 61 | // Because we are changing the graph when inlining, we need to remember the next block. |
| 62 | // This avoids doing the inlining work again on the inlined blocks. |
Vladimir Marko | fa6b93c | 2015-09-15 10:15:55 +0100 | [diff] [blame] | 63 | if (blocks[i] != next_block) { |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 64 | continue; |
| 65 | } |
| 66 | HBasicBlock* block = next_block; |
Vladimir Marko | fa6b93c | 2015-09-15 10:15:55 +0100 | [diff] [blame] | 67 | next_block = (i == blocks.size() - 1) ? nullptr : blocks[i + 1]; |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 68 | for (HInstruction* instruction = block->GetFirstInstruction(); instruction != nullptr;) { |
| 69 | HInstruction* next = instruction->GetNext(); |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 70 | HInvoke* call = instruction->AsInvoke(); |
Razvan A Lupusoru | 3e90a96 | 2015-03-27 13:44:44 -0700 | [diff] [blame] | 71 | // As long as the call is not intrinsified, it is worth trying to inline. |
| 72 | if (call != nullptr && call->GetIntrinsic() == Intrinsics::kNone) { |
Nicolas Geoffray | 7904129 | 2015-03-26 10:05:54 +0000 | [diff] [blame] | 73 | // We use the original invoke type to ensure the resolution of the called method |
| 74 | // works properly. |
Vladimir Marko | 5815501 | 2015-08-19 12:49:41 +0000 | [diff] [blame] | 75 | if (!TryInline(call)) { |
Nicolas Geoffray | 335005e | 2015-06-25 10:01:47 +0100 | [diff] [blame] | 76 | if (kIsDebugBuild && IsCompilingWithCoreImage()) { |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 77 | std::string callee_name = |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 78 | PrettyMethod(call->GetDexMethodIndex(), *outer_compilation_unit_.GetDexFile()); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 79 | bool should_inline = callee_name.find("$inline$") != std::string::npos; |
| 80 | CHECK(!should_inline) << "Could not inline " << callee_name; |
| 81 | } |
Guillaume "Vermeille" Sanchez | e918d38 | 2015-06-03 15:32:41 +0100 | [diff] [blame] | 82 | } else { |
Nicolas Geoffray | 335005e | 2015-06-25 10:01:47 +0100 | [diff] [blame] | 83 | if (kIsDebugBuild && IsCompilingWithCoreImage()) { |
Guillaume "Vermeille" Sanchez | e918d38 | 2015-06-03 15:32:41 +0100 | [diff] [blame] | 84 | std::string callee_name = |
| 85 | PrettyMethod(call->GetDexMethodIndex(), *outer_compilation_unit_.GetDexFile()); |
| 86 | bool must_not_inline = callee_name.find("$noinline$") != std::string::npos; |
| 87 | CHECK(!must_not_inline) << "Should not have inlined " << callee_name; |
| 88 | } |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 89 | } |
| 90 | } |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 91 | instruction = next; |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 92 | } |
| 93 | } |
| 94 | } |
| 95 | |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 96 | static bool IsMethodOrDeclaringClassFinal(ArtMethod* method) |
Mathieu Chartier | 9044347 | 2015-07-16 20:32:27 -0700 | [diff] [blame] | 97 | SHARED_REQUIRES(Locks::mutator_lock_) { |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 98 | return method->IsFinal() || method->GetDeclaringClass()->IsFinal(); |
| 99 | } |
| 100 | |
| 101 | /** |
| 102 | * Given the `resolved_method` looked up in the dex cache, try to find |
| 103 | * the actual runtime target of an interface or virtual call. |
| 104 | * Return nullptr if the runtime target cannot be proven. |
| 105 | */ |
| 106 | static ArtMethod* FindVirtualOrInterfaceTarget(HInvoke* invoke, ArtMethod* resolved_method) |
Mathieu Chartier | 9044347 | 2015-07-16 20:32:27 -0700 | [diff] [blame] | 107 | SHARED_REQUIRES(Locks::mutator_lock_) { |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 108 | if (IsMethodOrDeclaringClassFinal(resolved_method)) { |
| 109 | // No need to lookup further, the resolved method will be the target. |
| 110 | return resolved_method; |
| 111 | } |
| 112 | |
| 113 | HInstruction* receiver = invoke->InputAt(0); |
| 114 | if (receiver->IsNullCheck()) { |
| 115 | // Due to multiple levels of inlining within the same pass, it might be that |
| 116 | // null check does not have the reference type of the actual receiver. |
| 117 | receiver = receiver->InputAt(0); |
| 118 | } |
| 119 | ReferenceTypeInfo info = receiver->GetReferenceTypeInfo(); |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 120 | DCHECK(info.IsValid()) << "Invalid RTI for " << receiver->DebugName(); |
| 121 | if (!info.IsExact()) { |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 122 | // We currently only support inlining with known receivers. |
| 123 | // TODO: Remove this check, we should be able to inline final methods |
| 124 | // on unknown receivers. |
| 125 | return nullptr; |
| 126 | } else if (info.GetTypeHandle()->IsInterface()) { |
| 127 | // Statically knowing that the receiver has an interface type cannot |
| 128 | // help us find what is the target method. |
| 129 | return nullptr; |
| 130 | } else if (!resolved_method->GetDeclaringClass()->IsAssignableFrom(info.GetTypeHandle().Get())) { |
| 131 | // The method that we're trying to call is not in the receiver's class or super classes. |
| 132 | return nullptr; |
| 133 | } |
| 134 | |
| 135 | ClassLinker* cl = Runtime::Current()->GetClassLinker(); |
| 136 | size_t pointer_size = cl->GetImagePointerSize(); |
| 137 | if (invoke->IsInvokeInterface()) { |
| 138 | resolved_method = info.GetTypeHandle()->FindVirtualMethodForInterface( |
| 139 | resolved_method, pointer_size); |
| 140 | } else { |
| 141 | DCHECK(invoke->IsInvokeVirtual()); |
| 142 | resolved_method = info.GetTypeHandle()->FindVirtualMethodForVirtual( |
| 143 | resolved_method, pointer_size); |
| 144 | } |
| 145 | |
| 146 | if (resolved_method == nullptr) { |
| 147 | // The information we had on the receiver was not enough to find |
| 148 | // the target method. Since we check above the exact type of the receiver, |
| 149 | // the only reason this can happen is an IncompatibleClassChangeError. |
| 150 | return nullptr; |
Alex Light | 9139e00 | 2015-10-09 15:59:48 -0700 | [diff] [blame] | 151 | } else if (!resolved_method->IsInvokable()) { |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 152 | // The information we had on the receiver was not enough to find |
| 153 | // the target method. Since we check above the exact type of the receiver, |
| 154 | // the only reason this can happen is an IncompatibleClassChangeError. |
| 155 | return nullptr; |
| 156 | } else if (IsMethodOrDeclaringClassFinal(resolved_method)) { |
| 157 | // A final method has to be the target method. |
| 158 | return resolved_method; |
| 159 | } else if (info.IsExact()) { |
| 160 | // If we found a method and the receiver's concrete type is statically |
| 161 | // known, we know for sure the target. |
| 162 | return resolved_method; |
| 163 | } else { |
| 164 | // Even if we did find a method, the receiver type was not enough to |
| 165 | // statically find the runtime target. |
| 166 | return nullptr; |
| 167 | } |
| 168 | } |
| 169 | |
| 170 | static uint32_t FindMethodIndexIn(ArtMethod* method, |
| 171 | const DexFile& dex_file, |
| 172 | uint32_t referrer_index) |
Mathieu Chartier | 9044347 | 2015-07-16 20:32:27 -0700 | [diff] [blame] | 173 | SHARED_REQUIRES(Locks::mutator_lock_) { |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 174 | if (method->GetDexFile()->GetLocation().compare(dex_file.GetLocation()) == 0) { |
| 175 | return method->GetDexMethodIndex(); |
| 176 | } else { |
| 177 | return method->FindDexMethodIndexInOtherDexFile(dex_file, referrer_index); |
| 178 | } |
| 179 | } |
| 180 | |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 181 | bool HInliner::TryInline(HInvoke* invoke_instruction) { |
Calin Juravle | 175dc73 | 2015-08-25 15:42:32 +0100 | [diff] [blame] | 182 | if (invoke_instruction->IsInvokeUnresolved()) { |
| 183 | return false; // Don't bother to move further if we know the method is unresolved. |
| 184 | } |
| 185 | |
Vladimir Marko | 5815501 | 2015-08-19 12:49:41 +0000 | [diff] [blame] | 186 | uint32_t method_index = invoke_instruction->GetDexMethodIndex(); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 187 | ScopedObjectAccess soa(Thread::Current()); |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 188 | const DexFile& caller_dex_file = *caller_compilation_unit_.GetDexFile(); |
| 189 | VLOG(compiler) << "Try inlining " << PrettyMethod(method_index, caller_dex_file); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 190 | |
Nicolas Geoffray | 3507105 | 2015-06-09 15:43:38 +0100 | [diff] [blame] | 191 | ClassLinker* class_linker = caller_compilation_unit_.GetClassLinker(); |
| 192 | // We can query the dex cache directly. The verifier has populated it already. |
Vladimir Marko | 5815501 | 2015-08-19 12:49:41 +0000 | [diff] [blame] | 193 | ArtMethod* resolved_method; |
| 194 | if (invoke_instruction->IsInvokeStaticOrDirect()) { |
Nicolas Geoffray | e523423 | 2015-12-02 09:06:11 +0000 | [diff] [blame] | 195 | if (invoke_instruction->AsInvokeStaticOrDirect()->IsStringInit()) { |
| 196 | VLOG(compiler) << "Not inlining a String.<init> method"; |
| 197 | return false; |
| 198 | } |
Vladimir Marko | 5815501 | 2015-08-19 12:49:41 +0000 | [diff] [blame] | 199 | MethodReference ref = invoke_instruction->AsInvokeStaticOrDirect()->GetTargetMethod(); |
Mathieu Chartier | 736b560 | 2015-09-02 14:54:11 -0700 | [diff] [blame] | 200 | mirror::DexCache* const dex_cache = (&caller_dex_file == ref.dex_file) |
| 201 | ? caller_compilation_unit_.GetDexCache().Get() |
| 202 | : class_linker->FindDexCache(soa.Self(), *ref.dex_file); |
| 203 | resolved_method = dex_cache->GetResolvedMethod( |
Vladimir Marko | 5815501 | 2015-08-19 12:49:41 +0000 | [diff] [blame] | 204 | ref.dex_method_index, class_linker->GetImagePointerSize()); |
| 205 | } else { |
Mathieu Chartier | 736b560 | 2015-09-02 14:54:11 -0700 | [diff] [blame] | 206 | resolved_method = caller_compilation_unit_.GetDexCache().Get()->GetResolvedMethod( |
Vladimir Marko | 5815501 | 2015-08-19 12:49:41 +0000 | [diff] [blame] | 207 | method_index, class_linker->GetImagePointerSize()); |
| 208 | } |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 209 | |
Mathieu Chartier | e401d14 | 2015-04-22 13:56:20 -0700 | [diff] [blame] | 210 | if (resolved_method == nullptr) { |
Calin Juravle | 175dc73 | 2015-08-25 15:42:32 +0100 | [diff] [blame] | 211 | // TODO: Can this still happen? |
Nicolas Geoffray | 3507105 | 2015-06-09 15:43:38 +0100 | [diff] [blame] | 212 | // Method cannot be resolved if it is in another dex file we do not have access to. |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 213 | VLOG(compiler) << "Method cannot be resolved " << PrettyMethod(method_index, caller_dex_file); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 214 | return false; |
| 215 | } |
| 216 | |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 217 | if (!invoke_instruction->IsInvokeStaticOrDirect()) { |
| 218 | resolved_method = FindVirtualOrInterfaceTarget(invoke_instruction, resolved_method); |
| 219 | if (resolved_method == nullptr) { |
| 220 | VLOG(compiler) << "Interface or virtual call to " |
| 221 | << PrettyMethod(method_index, caller_dex_file) |
| 222 | << " could not be statically determined"; |
| 223 | return false; |
| 224 | } |
| 225 | // We have found a method, but we need to find where that method is for the caller's |
| 226 | // dex file. |
| 227 | method_index = FindMethodIndexIn(resolved_method, caller_dex_file, method_index); |
| 228 | if (method_index == DexFile::kDexNoIndex) { |
| 229 | VLOG(compiler) << "Interface or virtual call to " |
| 230 | << PrettyMethod(resolved_method) |
| 231 | << " cannot be inlined because unaccessible to caller"; |
| 232 | return false; |
| 233 | } |
| 234 | } |
| 235 | |
Vladimir Marko | 5815501 | 2015-08-19 12:49:41 +0000 | [diff] [blame] | 236 | bool same_dex_file = |
| 237 | IsSameDexFile(*outer_compilation_unit_.GetDexFile(), *resolved_method->GetDexFile()); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 238 | |
| 239 | const DexFile::CodeItem* code_item = resolved_method->GetCodeItem(); |
| 240 | |
| 241 | if (code_item == nullptr) { |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 242 | VLOG(compiler) << "Method " << PrettyMethod(method_index, caller_dex_file) |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 243 | << " is not inlined because it is native"; |
| 244 | return false; |
| 245 | } |
| 246 | |
Calin Juravle | ec74835 | 2015-07-29 13:52:12 +0100 | [diff] [blame] | 247 | size_t inline_max_code_units = compiler_driver_->GetCompilerOptions().GetInlineMaxCodeUnits(); |
| 248 | if (code_item->insns_size_in_code_units_ > inline_max_code_units) { |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 249 | VLOG(compiler) << "Method " << PrettyMethod(method_index, caller_dex_file) |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 250 | << " is too big to inline"; |
| 251 | return false; |
| 252 | } |
| 253 | |
| 254 | if (code_item->tries_size_ != 0) { |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 255 | VLOG(compiler) << "Method " << PrettyMethod(method_index, caller_dex_file) |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 256 | << " is not inlined because of try block"; |
| 257 | return false; |
| 258 | } |
| 259 | |
Nicolas Geoffray | 481303b | 2015-10-02 12:38:40 +0100 | [diff] [blame] | 260 | if (!resolved_method->GetDeclaringClass()->IsVerified()) { |
Nicolas Geoffray | ccc6197 | 2015-10-01 14:34:20 +0100 | [diff] [blame] | 261 | uint16_t class_def_idx = resolved_method->GetDeclaringClass()->GetDexClassDefIndex(); |
| 262 | if (!compiler_driver_->IsMethodVerifiedWithoutFailures( |
| 263 | resolved_method->GetDexMethodIndex(), class_def_idx, *resolved_method->GetDexFile())) { |
| 264 | VLOG(compiler) << "Method " << PrettyMethod(method_index, caller_dex_file) |
| 265 | << " couldn't be verified, so it cannot be inlined"; |
| 266 | return false; |
| 267 | } |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 268 | } |
| 269 | |
Roland Levillain | 4c0eb42 | 2015-04-24 16:43:49 +0100 | [diff] [blame] | 270 | if (invoke_instruction->IsInvokeStaticOrDirect() && |
| 271 | invoke_instruction->AsInvokeStaticOrDirect()->IsStaticWithImplicitClinitCheck()) { |
| 272 | // Case of a static method that cannot be inlined because it implicitly |
| 273 | // requires an initialization check of its declaring class. |
| 274 | VLOG(compiler) << "Method " << PrettyMethod(method_index, caller_dex_file) |
| 275 | << " is not inlined because it is static and requires a clinit" |
| 276 | << " check that cannot be emitted due to Dex cache limitations"; |
| 277 | return false; |
| 278 | } |
| 279 | |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 280 | if (!TryBuildAndInline(resolved_method, invoke_instruction, same_dex_file)) { |
Nicolas Geoffray | c0365b1 | 2015-03-18 18:31:52 +0000 | [diff] [blame] | 281 | return false; |
| 282 | } |
| 283 | |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 284 | VLOG(compiler) << "Successfully inlined " << PrettyMethod(method_index, caller_dex_file); |
Nicolas Geoffray | c0365b1 | 2015-03-18 18:31:52 +0000 | [diff] [blame] | 285 | MaybeRecordStat(kInlinedInvoke); |
| 286 | return true; |
| 287 | } |
| 288 | |
Mathieu Chartier | e401d14 | 2015-04-22 13:56:20 -0700 | [diff] [blame] | 289 | bool HInliner::TryBuildAndInline(ArtMethod* resolved_method, |
Nicolas Geoffray | c0365b1 | 2015-03-18 18:31:52 +0000 | [diff] [blame] | 290 | HInvoke* invoke_instruction, |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 291 | bool same_dex_file) { |
Nicolas Geoffray | c0365b1 | 2015-03-18 18:31:52 +0000 | [diff] [blame] | 292 | ScopedObjectAccess soa(Thread::Current()); |
| 293 | const DexFile::CodeItem* code_item = resolved_method->GetCodeItem(); |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 294 | const DexFile& callee_dex_file = *resolved_method->GetDexFile(); |
| 295 | uint32_t method_index = resolved_method->GetDexMethodIndex(); |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 296 | ClassLinker* class_linker = caller_compilation_unit_.GetClassLinker(); |
Mathieu Chartier | 736b560 | 2015-09-02 14:54:11 -0700 | [diff] [blame] | 297 | Handle<mirror::DexCache> dex_cache(handles_->NewHandle(resolved_method->GetDexCache())); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 298 | DexCompilationUnit dex_compilation_unit( |
| 299 | nullptr, |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 300 | caller_compilation_unit_.GetClassLoader(), |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 301 | class_linker, |
Nicolas Geoffray | 8dbf0cf | 2015-08-11 02:14:38 +0000 | [diff] [blame] | 302 | callee_dex_file, |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 303 | code_item, |
| 304 | resolved_method->GetDeclaringClass()->GetDexClassDefIndex(), |
Nicolas Geoffray | 8dbf0cf | 2015-08-11 02:14:38 +0000 | [diff] [blame] | 305 | method_index, |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 306 | resolved_method->GetAccessFlags(), |
Mathieu Chartier | 736b560 | 2015-09-02 14:54:11 -0700 | [diff] [blame] | 307 | compiler_driver_->GetVerifiedMethod(&callee_dex_file, method_index), |
| 308 | dex_cache); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 309 | |
Calin Juravle | 3cd4fc8 | 2015-05-14 15:15:42 +0100 | [diff] [blame] | 310 | bool requires_ctor_barrier = false; |
| 311 | |
| 312 | if (dex_compilation_unit.IsConstructor()) { |
| 313 | // If it's a super invocation and we already generate a barrier there's no need |
| 314 | // to generate another one. |
| 315 | // We identify super calls by looking at the "this" pointer. If its value is the |
| 316 | // same as the local "this" pointer then we must have a super invocation. |
| 317 | bool is_super_invocation = invoke_instruction->InputAt(0)->IsParameterValue() |
| 318 | && invoke_instruction->InputAt(0)->AsParameterValue()->IsThis(); |
| 319 | if (is_super_invocation && graph_->ShouldGenerateConstructorBarrier()) { |
| 320 | requires_ctor_barrier = false; |
| 321 | } else { |
| 322 | Thread* self = Thread::Current(); |
| 323 | requires_ctor_barrier = compiler_driver_->RequiresConstructorBarrier(self, |
| 324 | dex_compilation_unit.GetDexFile(), |
| 325 | dex_compilation_unit.GetClassDefIndex()); |
| 326 | } |
| 327 | } |
| 328 | |
Nicolas Geoffray | 3507105 | 2015-06-09 15:43:38 +0100 | [diff] [blame] | 329 | InvokeType invoke_type = invoke_instruction->GetOriginalInvokeType(); |
| 330 | if (invoke_type == kInterface) { |
| 331 | // We have statically resolved the dispatch. To please the class linker |
| 332 | // at runtime, we change this call as if it was a virtual call. |
| 333 | invoke_type = kVirtual; |
| 334 | } |
Nicolas Geoffray | e0fe7ae | 2015-03-09 10:02:49 +0000 | [diff] [blame] | 335 | HGraph* callee_graph = new (graph_->GetArena()) HGraph( |
Nicolas Geoffray | 0a23d74 | 2015-05-07 11:57:35 +0100 | [diff] [blame] | 336 | graph_->GetArena(), |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 337 | callee_dex_file, |
Nicolas Geoffray | 0a23d74 | 2015-05-07 11:57:35 +0100 | [diff] [blame] | 338 | method_index, |
Calin Juravle | 3cd4fc8 | 2015-05-14 15:15:42 +0100 | [diff] [blame] | 339 | requires_ctor_barrier, |
Mathieu Chartier | e401d14 | 2015-04-22 13:56:20 -0700 | [diff] [blame] | 340 | compiler_driver_->GetInstructionSet(), |
Nicolas Geoffray | 3507105 | 2015-06-09 15:43:38 +0100 | [diff] [blame] | 341 | invoke_type, |
Nicolas Geoffray | 0a23d74 | 2015-05-07 11:57:35 +0100 | [diff] [blame] | 342 | graph_->IsDebuggable(), |
| 343 | graph_->GetCurrentInstructionId()); |
David Brazdil | 5e8b137 | 2015-01-23 14:39:08 +0000 | [diff] [blame] | 344 | |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 345 | OptimizingCompilerStats inline_stats; |
David Brazdil | 5e8b137 | 2015-01-23 14:39:08 +0000 | [diff] [blame] | 346 | HGraphBuilder builder(callee_graph, |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 347 | &dex_compilation_unit, |
| 348 | &outer_compilation_unit_, |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 349 | resolved_method->GetDexFile(), |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 350 | compiler_driver_, |
Nicolas Geoffray | 9523a3e | 2015-07-17 11:51:28 +0000 | [diff] [blame] | 351 | &inline_stats, |
Mathieu Chartier | 736b560 | 2015-09-02 14:54:11 -0700 | [diff] [blame] | 352 | resolved_method->GetQuickenedInfo(), |
| 353 | dex_cache); |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 354 | |
David Brazdil | 5e8b137 | 2015-01-23 14:39:08 +0000 | [diff] [blame] | 355 | if (!builder.BuildGraph(*code_item)) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 356 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 357 | << " could not be built, so cannot be inlined"; |
| 358 | return false; |
| 359 | } |
| 360 | |
Nicolas Geoffray | 259136f | 2014-12-17 23:21:58 +0000 | [diff] [blame] | 361 | if (!RegisterAllocator::CanAllocateRegistersFor(*callee_graph, |
| 362 | compiler_driver_->GetInstructionSet())) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 363 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | 259136f | 2014-12-17 23:21:58 +0000 | [diff] [blame] | 364 | << " cannot be inlined because of the register allocator"; |
| 365 | return false; |
| 366 | } |
| 367 | |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 368 | if (!callee_graph->TryBuildingSsa()) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 369 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 370 | << " could not be transformed to SSA"; |
| 371 | return false; |
| 372 | } |
| 373 | |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 374 | size_t parameter_index = 0; |
| 375 | for (HInstructionIterator instructions(callee_graph->GetEntryBlock()->GetInstructions()); |
| 376 | !instructions.Done(); |
| 377 | instructions.Advance()) { |
| 378 | HInstruction* current = instructions.Current(); |
| 379 | if (current->IsParameterValue()) { |
| 380 | HInstruction* argument = invoke_instruction->InputAt(parameter_index++); |
| 381 | if (argument->IsNullConstant()) { |
| 382 | current->ReplaceWith(callee_graph->GetNullConstant()); |
| 383 | } else if (argument->IsIntConstant()) { |
| 384 | current->ReplaceWith(callee_graph->GetIntConstant(argument->AsIntConstant()->GetValue())); |
| 385 | } else if (argument->IsLongConstant()) { |
| 386 | current->ReplaceWith(callee_graph->GetLongConstant(argument->AsLongConstant()->GetValue())); |
| 387 | } else if (argument->IsFloatConstant()) { |
| 388 | current->ReplaceWith( |
| 389 | callee_graph->GetFloatConstant(argument->AsFloatConstant()->GetValue())); |
| 390 | } else if (argument->IsDoubleConstant()) { |
| 391 | current->ReplaceWith( |
| 392 | callee_graph->GetDoubleConstant(argument->AsDoubleConstant()->GetValue())); |
| 393 | } else if (argument->GetType() == Primitive::kPrimNot) { |
| 394 | current->SetReferenceTypeInfo(argument->GetReferenceTypeInfo()); |
| 395 | current->AsParameterValue()->SetCanBeNull(argument->CanBeNull()); |
| 396 | } |
| 397 | } |
| 398 | } |
| 399 | |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 400 | // Run simple optimizations on the graph. |
Calin Juravle | 7a9c885 | 2015-04-21 14:07:50 +0100 | [diff] [blame] | 401 | HDeadCodeElimination dce(callee_graph, stats_); |
Nicolas Geoffray | e34648d | 2015-11-23 08:59:07 +0000 | [diff] [blame] | 402 | HConstantFolding fold(callee_graph); |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 403 | ReferenceTypePropagation type_propagation(callee_graph, handles_); |
Vladimir Marko | dc151b2 | 2015-10-15 18:02:30 +0100 | [diff] [blame] | 404 | HSharpening sharpening(callee_graph, codegen_, dex_compilation_unit, compiler_driver_); |
Calin Juravle | acf735c | 2015-02-12 15:25:22 +0000 | [diff] [blame] | 405 | InstructionSimplifier simplify(callee_graph, stats_); |
Nicolas Geoffray | e34648d | 2015-11-23 08:59:07 +0000 | [diff] [blame] | 406 | IntrinsicsRecognizer intrinsics(callee_graph, compiler_driver_); |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 407 | |
| 408 | HOptimization* optimizations[] = { |
Scott Wakeling | d60a1af | 2015-07-22 14:32:44 +0100 | [diff] [blame] | 409 | &intrinsics, |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 410 | &type_propagation, |
Vladimir Marko | dc151b2 | 2015-10-15 18:02:30 +0100 | [diff] [blame] | 411 | &sharpening, |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 412 | &simplify, |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 413 | &fold, |
Vladimir Marko | 9e23df5 | 2015-11-10 17:14:35 +0000 | [diff] [blame] | 414 | &dce, |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 415 | }; |
| 416 | |
| 417 | for (size_t i = 0; i < arraysize(optimizations); ++i) { |
| 418 | HOptimization* optimization = optimizations[i]; |
| 419 | optimization->Run(); |
| 420 | } |
| 421 | |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 422 | size_t number_of_instructions_budget = kMaximumNumberOfHInstructions; |
Calin Juravle | ec74835 | 2015-07-29 13:52:12 +0100 | [diff] [blame] | 423 | if (depth_ + 1 < compiler_driver_->GetCompilerOptions().GetInlineDepthLimit()) { |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 424 | HInliner inliner(callee_graph, |
Vladimir Marko | dc151b2 | 2015-10-15 18:02:30 +0100 | [diff] [blame] | 425 | codegen_, |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 426 | outer_compilation_unit_, |
| 427 | dex_compilation_unit, |
| 428 | compiler_driver_, |
Nicolas Geoffray | 454a481 | 2015-06-09 10:37:32 +0100 | [diff] [blame] | 429 | handles_, |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 430 | stats_, |
| 431 | depth_ + 1); |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 432 | inliner.Run(); |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 433 | number_of_instructions_budget += inliner.number_of_inlined_instructions_; |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 434 | } |
| 435 | |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 436 | // TODO: We should abort only if all predecessors throw. However, |
| 437 | // HGraph::InlineInto currently does not handle an exit block with |
| 438 | // a throw predecessor. |
| 439 | HBasicBlock* exit_block = callee_graph->GetExitBlock(); |
| 440 | if (exit_block == nullptr) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 441 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 442 | << " could not be inlined because it has an infinite loop"; |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 443 | return false; |
| 444 | } |
| 445 | |
| 446 | bool has_throw_predecessor = false; |
Vladimir Marko | 6058455 | 2015-09-03 13:35:12 +0000 | [diff] [blame] | 447 | for (HBasicBlock* predecessor : exit_block->GetPredecessors()) { |
| 448 | if (predecessor->GetLastInstruction()->IsThrow()) { |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 449 | has_throw_predecessor = true; |
| 450 | break; |
| 451 | } |
| 452 | } |
| 453 | if (has_throw_predecessor) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 454 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 455 | << " could not be inlined because one branch always throws"; |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 456 | return false; |
| 457 | } |
| 458 | |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 459 | HReversePostOrderIterator it(*callee_graph); |
Nicolas Geoffray | ef87c5d | 2015-01-30 12:41:14 +0000 | [diff] [blame] | 460 | it.Advance(); // Past the entry block, it does not contain instructions that prevent inlining. |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 461 | size_t number_of_instructions = 0; |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 462 | for (; !it.Done(); it.Advance()) { |
| 463 | HBasicBlock* block = it.Current(); |
| 464 | if (block->IsLoopHeader()) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 465 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 466 | << " could not be inlined because it contains a loop"; |
| 467 | return false; |
| 468 | } |
| 469 | |
| 470 | for (HInstructionIterator instr_it(block->GetInstructions()); |
| 471 | !instr_it.Done(); |
| 472 | instr_it.Advance()) { |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 473 | if (number_of_instructions++ == number_of_instructions_budget) { |
| 474 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
| 475 | << " could not be inlined because it is too big."; |
| 476 | return false; |
| 477 | } |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 478 | HInstruction* current = instr_it.Current(); |
Nicolas Geoffray | 276d9da | 2015-02-02 18:24:11 +0000 | [diff] [blame] | 479 | |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 480 | if (current->IsInvokeInterface()) { |
| 481 | // Disable inlining of interface calls. The cost in case of entering the |
| 482 | // resolution conflict is currently too high. |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 483 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 484 | << " could not be inlined because it has an interface call."; |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 485 | return false; |
| 486 | } |
| 487 | |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 488 | if (!same_dex_file && current->NeedsEnvironment()) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 489 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 490 | << " could not be inlined because " << current->DebugName() |
Nicolas Geoffray | d23eeef | 2015-05-18 22:31:29 +0100 | [diff] [blame] | 491 | << " needs an environment and is in a different dex file"; |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 492 | return false; |
| 493 | } |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 494 | |
Vladimir Marko | dc151b2 | 2015-10-15 18:02:30 +0100 | [diff] [blame] | 495 | if (!same_dex_file && current->NeedsDexCacheOfDeclaringClass()) { |
Guillaume "Vermeille" Sanchez | ae09d2d | 2015-05-29 10:52:55 +0100 | [diff] [blame] | 496 | VLOG(compiler) << "Method " << PrettyMethod(method_index, callee_dex_file) |
Nicolas Geoffray | 9437b78 | 2015-03-25 10:08:51 +0000 | [diff] [blame] | 497 | << " could not be inlined because " << current->DebugName() |
| 498 | << " it is in a different dex file and requires access to the dex cache"; |
| 499 | return false; |
| 500 | } |
Nicolas Geoffray | d930929 | 2015-10-31 22:21:31 +0000 | [diff] [blame] | 501 | |
| 502 | if (current->IsNewInstance() && |
| 503 | (current->AsNewInstance()->GetEntrypoint() == kQuickAllocObjectWithAccessCheck)) { |
| 504 | // Allocation entrypoint does not handle inlined frames. |
| 505 | return false; |
| 506 | } |
| 507 | |
| 508 | if (current->IsNewArray() && |
| 509 | (current->AsNewArray()->GetEntrypoint() == kQuickAllocArrayWithAccessCheck)) { |
| 510 | // Allocation entrypoint does not handle inlined frames. |
| 511 | return false; |
| 512 | } |
| 513 | |
| 514 | if (current->IsUnresolvedStaticFieldGet() || |
| 515 | current->IsUnresolvedInstanceFieldGet() || |
| 516 | current->IsUnresolvedStaticFieldSet() || |
| 517 | current->IsUnresolvedInstanceFieldSet()) { |
| 518 | // Entrypoint for unresolved fields does not handle inlined frames. |
| 519 | return false; |
| 520 | } |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 521 | } |
| 522 | } |
Nicolas Geoffray | e418dda | 2015-08-11 20:03:09 -0700 | [diff] [blame] | 523 | number_of_inlined_instructions_ += number_of_instructions; |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 524 | |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 525 | HInstruction* return_replacement = callee_graph->InlineInto(graph_, invoke_instruction); |
Calin Juravle | 214bbcd | 2015-10-20 14:54:07 +0100 | [diff] [blame] | 526 | if (return_replacement != nullptr) { |
| 527 | DCHECK_EQ(graph_, return_replacement->GetBlock()->GetGraph()); |
| 528 | } |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 529 | |
| 530 | // When merging the graph we might create a new NullConstant in the caller graph which does |
| 531 | // not have the chance to be typed. We assign the correct type here so that we can keep the |
| 532 | // assertion that every reference has a valid type. This also simplifies checks along the way. |
| 533 | HNullConstant* null_constant = graph_->GetNullConstant(); |
| 534 | if (!null_constant->GetReferenceTypeInfo().IsValid()) { |
| 535 | ReferenceTypeInfo::TypeHandle obj_handle = |
| 536 | handles_->NewHandle(class_linker->GetClassRoot(ClassLinker::kJavaLangObject)); |
| 537 | null_constant->SetReferenceTypeInfo( |
| 538 | ReferenceTypeInfo::Create(obj_handle, false /* is_exact */)); |
| 539 | } |
| 540 | |
Calin Juravle | cdfed3d | 2015-10-26 14:05:01 +0000 | [diff] [blame] | 541 | // Check the integrity of reference types and run another type propagation if needed. |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 542 | if ((return_replacement != nullptr) |
| 543 | && (return_replacement->GetType() == Primitive::kPrimNot)) { |
| 544 | if (!return_replacement->GetReferenceTypeInfo().IsValid()) { |
| 545 | // Make sure that we have a valid type for the return. We may get an invalid one when |
| 546 | // we inline invokes with multiple branches and create a Phi for the result. |
| 547 | // TODO: we could be more precise by merging the phi inputs but that requires |
| 548 | // some functionality from the reference type propagation. |
| 549 | DCHECK(return_replacement->IsPhi()); |
Vladimir Marko | 05792b9 | 2015-08-03 11:56:49 +0100 | [diff] [blame] | 550 | size_t pointer_size = Runtime::Current()->GetClassLinker()->GetImagePointerSize(); |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 551 | ReferenceTypeInfo::TypeHandle return_handle = |
Calin Juravle | cdfed3d | 2015-10-26 14:05:01 +0000 | [diff] [blame] | 552 | handles_->NewHandle(resolved_method->GetReturnType(true /* resolve */, pointer_size)); |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 553 | return_replacement->SetReferenceTypeInfo(ReferenceTypeInfo::Create( |
David Brazdil | baf89b8 | 2015-09-15 11:36:54 +0100 | [diff] [blame] | 554 | return_handle, return_handle->CannotBeAssignedFromOtherTypes() /* is_exact */)); |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 555 | } |
Calin Juravle | cdfed3d | 2015-10-26 14:05:01 +0000 | [diff] [blame] | 556 | |
| 557 | // If the return type is a refinement of the declared type run the type propagation again. |
| 558 | ReferenceTypeInfo return_rti = return_replacement->GetReferenceTypeInfo(); |
| 559 | ReferenceTypeInfo invoke_rti = invoke_instruction->GetReferenceTypeInfo(); |
| 560 | if (invoke_rti.IsStrictSupertypeOf(return_rti) |
| 561 | || (return_rti.IsExact() && !invoke_rti.IsExact()) |
| 562 | || !return_replacement->CanBeNull()) { |
| 563 | ReferenceTypePropagation rtp_fixup(graph_, handles_); |
| 564 | rtp_fixup.Run(); |
| 565 | } |
Calin Juravle | 2e76830 | 2015-07-28 14:41:11 +0000 | [diff] [blame] | 566 | } |
Nicolas Geoffray | 7c5367b | 2014-12-17 10:13:46 +0000 | [diff] [blame] | 567 | |
Nicolas Geoffray | e53798a | 2014-12-01 10:31:54 +0000 | [diff] [blame] | 568 | return true; |
| 569 | } |
| 570 | |
| 571 | } // namespace art |