GeezerVison is showing github.com — deps: backport 819b40a from V8 upstream · nodejs/node@ab25589
File treeExpand file treeCollapse file tree Open diff view settings Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/ast-numbering.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -553,7 +553,7 @@ bool AstNumberingVisitor::Renumber(FunctionLiteral* node) { | 553 | 553 | Scope* scope = node->scope(); | 554 | 554 |
| 555 | 555 | if (scope->HasIllegalRedeclaration()) { | 556 | | - scope->VisitIllegalRedeclaration(this); | | 556 | + Visit(scope->GetIllegalRedeclaration()); | 557 | 557 | DisableOptimization(kFunctionWithIllegalRedeclaration); | 558 | 558 | return Finish(node); | 559 | 559 | } | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/compiler/ast-graph-builder.cc +1-2Lines changed: 1 addition & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -586,8 +586,7 @@ void AstGraphBuilder::CreateGraphBody(bool stack_check) { | 586 | 586 |
| 587 | 587 | // Visit illegal re-declaration and bail out if it exists. | 588 | 588 | if (scope->HasIllegalRedeclaration()) { | 589 | | - AstEffectContext for_effect(this); | 590 | | - scope->VisitIllegalRedeclaration(this); | | 589 | + VisitForEffect(scope->GetIllegalRedeclaration()); | 591 | 590 | return; | 592 | 591 | } | 593 | 592 |
| |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/compiler/linkage.cc +2-1Lines changed: 2 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -218,10 +218,10 @@ int Linkage::FrameStateInputCount(Runtime::FunctionId function) { | 218 | 218 | switch (function) { | 219 | 219 | case Runtime::kAllocateInTargetSpace: | 220 | 220 | case Runtime::kDateField: | 221 | | - case Runtime::kFinalizeClassDefinition: // TODO(conradw): Is it safe? | 222 | 221 | case Runtime::kDefineClassMethod: // TODO(jarin): Is it safe? | 223 | 222 | case Runtime::kDefineGetterPropertyUnchecked: // TODO(jarin): Is it safe? | 224 | 223 | case Runtime::kDefineSetterPropertyUnchecked: // TODO(jarin): Is it safe? | | 224 | + case Runtime::kFinalizeClassDefinition: // TODO(conradw): Is it safe? | 225 | 225 | case Runtime::kForInDone: | 226 | 226 | case Runtime::kForInStep: | 227 | 227 | case Runtime::kGetOriginalConstructor: | @@ -244,6 +244,7 @@ int Linkage::FrameStateInputCount(Runtime::FunctionId function) { | 244 | 244 | case Runtime::kInlineGetCallerJSFunction: | 245 | 245 | case Runtime::kInlineGetPrototype: | 246 | 246 | case Runtime::kInlineRegExpExec: | | 247 | + case Runtime::kInlineSubString: | 247 | 248 | case Runtime::kInlineToObject: | 248 | 249 | return 1; | 249 | 250 | case Runtime::kInlineDeoptimizeNow: | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/full-codegen/arm/full-codegen-arm.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -338,7 +338,7 @@ void FullCodeGenerator::Generate() { | 338 | 338 | // redeclaration. | 339 | 339 | if (scope()->HasIllegalRedeclaration()) { | 340 | 340 | Comment cmnt(masm_, "[ Declarations"); | 341 | | - scope()->VisitIllegalRedeclaration(this); | | 341 | + VisitForEffect(scope()->GetIllegalRedeclaration()); | 342 | 342 |
| 343 | 343 | } else { | 344 | 344 | PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/full-codegen/arm64/full-codegen-arm64.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -344,7 +344,7 @@ void FullCodeGenerator::Generate() { | 344 | 344 | // redeclaration. | 345 | 345 | if (scope()->HasIllegalRedeclaration()) { | 346 | 346 | Comment cmnt(masm_, "[ Declarations"); | 347 | | - scope()->VisitIllegalRedeclaration(this); | | 347 | + VisitForEffect(scope()->GetIllegalRedeclaration()); | 348 | 348 |
| 349 | 349 | } else { | 350 | 350 | PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/full-codegen/ia32/full-codegen-ia32.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -341,7 +341,7 @@ void FullCodeGenerator::Generate() { | 341 | 341 | // redeclaration. | 342 | 342 | if (scope()->HasIllegalRedeclaration()) { | 343 | 343 | Comment cmnt(masm_, "[ Declarations"); | 344 | | - scope()->VisitIllegalRedeclaration(this); | | 344 | + VisitForEffect(scope()->GetIllegalRedeclaration()); | 345 | 345 |
| 346 | 346 | } else { | 347 | 347 | PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/full-codegen/mips/full-codegen-mips.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -356,7 +356,7 @@ void FullCodeGenerator::Generate() { | 356 | 356 | // redeclaration. | 357 | 357 | if (scope()->HasIllegalRedeclaration()) { | 358 | 358 | Comment cmnt(masm_, "[ Declarations"); | 359 | | - scope()->VisitIllegalRedeclaration(this); | | 359 | + VisitForEffect(scope()->GetIllegalRedeclaration()); | 360 | 360 |
| 361 | 361 | } else { | 362 | 362 | PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/full-codegen/mips64/full-codegen-mips64.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -351,7 +351,7 @@ void FullCodeGenerator::Generate() { | 351 | 351 | // redeclaration. | 352 | 352 | if (scope()->HasIllegalRedeclaration()) { | 353 | 353 | Comment cmnt(masm_, "[ Declarations"); | 354 | | - scope()->VisitIllegalRedeclaration(this); | | 354 | + VisitForEffect(scope()->GetIllegalRedeclaration()); | 355 | 355 |
| 356 | 356 | } else { | 357 | 357 | PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/full-codegen/ppc/full-codegen-ppc.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -349,7 +349,7 @@ void FullCodeGenerator::Generate() { | 349 | 349 | // redeclaration. | 350 | 350 | if (scope()->HasIllegalRedeclaration()) { | 351 | 351 | Comment cmnt(masm_, "[ Declarations"); | 352 | | - scope()->VisitIllegalRedeclaration(this); | | 352 | + VisitForEffect(scope()->GetIllegalRedeclaration()); | 353 | 353 |
| 354 | 354 | } else { | 355 | 355 | PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); | |
Collapse file Copy file name to clipboardExpand all lines: deps/v8/src/full-codegen/x64/full-codegen-x64.cc +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -338,7 +338,7 @@ void FullCodeGenerator::Generate() { | 338 | 338 | // redeclaration. | 339 | 339 | if (scope()->HasIllegalRedeclaration()) { | 340 | 340 | Comment cmnt(masm_, "[ Declarations"); | 341 | | - scope()->VisitIllegalRedeclaration(this); | | 341 | + VisitForEffect(scope()->GetIllegalRedeclaration()); | 342 | 342 |
| 343 | 343 | } else { | 344 | 344 | PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); | |
|