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

Side by Side Diff: src/full-codegen/ia32/full-codegen-ia32.cc

Issue 2105023003: [turbofan] Also verify lazy bailout points in graph builder. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix object literals with duplicate computed properties. Created 4 years, 5 months 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 #if V8_TARGET_ARCH_IA32 5 #if V8_TARGET_ARCH_IA32
6 6
7 #include "src/ast/scopes.h" 7 #include "src/ast/scopes.h"
8 #include "src/code-factory.h" 8 #include "src/code-factory.h"
9 #include "src/code-stubs.h" 9 #include "src/code-stubs.h"
10 #include "src/codegen.h" 10 #include "src/codegen.h"
(...skipping 3265 matching lines...) Expand 10 before | Expand all | Expand 10 after
3276 if (!context()->IsEffect()) { 3276 if (!context()->IsEffect()) {
3277 context()->PlugTOS(); 3277 context()->PlugTOS();
3278 } 3278 }
3279 } else { 3279 } else {
3280 context()->Plug(eax); 3280 context()->Plug(eax);
3281 } 3281 }
3282 break; 3282 break;
3283 } 3283 }
3284 case NAMED_SUPER_PROPERTY: { 3284 case NAMED_SUPER_PROPERTY: {
3285 EmitNamedSuperPropertyStore(prop); 3285 EmitNamedSuperPropertyStore(prop);
3286 PrepareForBailoutForId(expr->AssignmentId(), BailoutState::TOS_REGISTER);
3286 if (expr->is_postfix()) { 3287 if (expr->is_postfix()) {
3287 if (!context()->IsEffect()) { 3288 if (!context()->IsEffect()) {
3288 context()->PlugTOS(); 3289 context()->PlugTOS();
3289 } 3290 }
3290 } else { 3291 } else {
3291 context()->Plug(eax); 3292 context()->Plug(eax);
3292 } 3293 }
3293 break; 3294 break;
3294 } 3295 }
3295 case KEYED_SUPER_PROPERTY: { 3296 case KEYED_SUPER_PROPERTY: {
3296 EmitKeyedSuperPropertyStore(prop); 3297 EmitKeyedSuperPropertyStore(prop);
3298 PrepareForBailoutForId(expr->AssignmentId(), BailoutState::TOS_REGISTER);
3297 if (expr->is_postfix()) { 3299 if (expr->is_postfix()) {
3298 if (!context()->IsEffect()) { 3300 if (!context()->IsEffect()) {
3299 context()->PlugTOS(); 3301 context()->PlugTOS();
3300 } 3302 }
3301 } else { 3303 } else {
3302 context()->Plug(eax); 3304 context()->Plug(eax);
3303 } 3305 }
3304 break; 3306 break;
3305 } 3307 }
3306 case KEYED_PROPERTY: { 3308 case KEYED_PROPERTY: {
(...skipping 378 matching lines...) Expand 10 before | Expand all | Expand 10 after
3685 isolate->builtins()->OnStackReplacement()->entry(), 3687 isolate->builtins()->OnStackReplacement()->entry(),
3686 Assembler::target_address_at(call_target_address, unoptimized_code)); 3688 Assembler::target_address_at(call_target_address, unoptimized_code));
3687 return ON_STACK_REPLACEMENT; 3689 return ON_STACK_REPLACEMENT;
3688 } 3690 }
3689 3691
3690 3692
3691 } // namespace internal 3693 } // namespace internal
3692 } // namespace v8 3694 } // namespace v8
3693 3695
3694 #endif // V8_TARGET_ARCH_IA32 3696 #endif // V8_TARGET_ARCH_IA32
OLDNEW
« no previous file with comments | « src/full-codegen/arm64/full-codegen-arm64.cc ('k') | src/full-codegen/mips/full-codegen-mips.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698