Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(103)

Side by Side Diff: src/crankshaft/arm/lithium-arm.cc

Issue 2451853002: Uniform and precise source positions for inlining (Closed)
Patch Set: fixed gcmole issue Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/crankshaft/arm/lithium-arm.h" 5 #include "src/crankshaft/arm/lithium-arm.h"
6 6
7 #include <sstream> 7 #include <sstream>
8 8
9 #include "src/crankshaft/arm/lithium-codegen-arm.h" 9 #include "src/crankshaft/arm/lithium-codegen-arm.h"
10 #include "src/crankshaft/hydrogen-osr.h" 10 #include "src/crankshaft/hydrogen-osr.h"
(...skipping 2326 matching lines...) Expand 10 before | Expand all | Expand 10 after
2337 HEnvironment* inner = outer->CopyForInlining( 2337 HEnvironment* inner = outer->CopyForInlining(
2338 instr->closure(), instr->arguments_count(), instr->function(), undefined, 2338 instr->closure(), instr->arguments_count(), instr->function(), undefined,
2339 instr->inlining_kind(), instr->syntactic_tail_call_mode()); 2339 instr->inlining_kind(), instr->syntactic_tail_call_mode());
2340 // Only replay binding of arguments object if it wasn't removed from graph. 2340 // Only replay binding of arguments object if it wasn't removed from graph.
2341 if (instr->arguments_var() != NULL && instr->arguments_object()->IsLinked()) { 2341 if (instr->arguments_var() != NULL && instr->arguments_object()->IsLinked()) {
2342 inner->Bind(instr->arguments_var(), instr->arguments_object()); 2342 inner->Bind(instr->arguments_var(), instr->arguments_object());
2343 } 2343 }
2344 inner->BindContext(instr->closure_context()); 2344 inner->BindContext(instr->closure_context());
2345 inner->set_entry(instr); 2345 inner->set_entry(instr);
2346 current_block_->UpdateEnvironment(inner); 2346 current_block_->UpdateEnvironment(inner);
2347 chunk_->AddInlinedFunction(instr->shared());
2348 return NULL; 2347 return NULL;
2349 } 2348 }
2350 2349
2351 2350
2352 LInstruction* LChunkBuilder::DoLeaveInlined(HLeaveInlined* instr) { 2351 LInstruction* LChunkBuilder::DoLeaveInlined(HLeaveInlined* instr) {
2353 LInstruction* pop = NULL; 2352 LInstruction* pop = NULL;
2354 2353
2355 HEnvironment* env = current_block_->last_environment(); 2354 HEnvironment* env = current_block_->last_environment();
2356 2355
2357 if (env->entry()->arguments_pushed()) { 2356 if (env->entry()->arguments_pushed()) {
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
2392 LInstruction* LChunkBuilder::DoLoadFieldByIndex(HLoadFieldByIndex* instr) { 2391 LInstruction* LChunkBuilder::DoLoadFieldByIndex(HLoadFieldByIndex* instr) {
2393 LOperand* object = UseRegister(instr->object()); 2392 LOperand* object = UseRegister(instr->object());
2394 LOperand* index = UseTempRegister(instr->index()); 2393 LOperand* index = UseTempRegister(instr->index());
2395 LLoadFieldByIndex* load = new(zone()) LLoadFieldByIndex(object, index); 2394 LLoadFieldByIndex* load = new(zone()) LLoadFieldByIndex(object, index);
2396 LInstruction* result = DefineSameAsFirst(load); 2395 LInstruction* result = DefineSameAsFirst(load);
2397 return AssignPointerMap(result); 2396 return AssignPointerMap(result);
2398 } 2397 }
2399 2398
2400 } // namespace internal 2399 } // namespace internal
2401 } // namespace v8 2400 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698