AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Remove confusing (and apparently wrong) DCHECK from ScopeInfo::Create

Browse Source 
We should always only have exactly as many heap slots as context locals

R=verwaest@chromium.org,marja@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2280883002
Cr-Commit-Position: refs/heads/master@{#38946}
This commit is contained in:
jochen 2016-08-26 07:11:54 -07:00 committed by Commit bot
parent 9c8f4775bd
commit 64b6557886
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/ast/scopeinfo.cc
View File

@ -193,9 +193,7 @@ Handle<ScopeInfo> ScopeInfo::Create(Isolate* isolate, Zone* zone,
DCHECK(index == scope_info->length());
DCHECK(scope->num_parameters() == scope_info->ParameterCount());
DCHECK(scope->num_heap_slots() == scope_info->ContextLength() ||
(scope->num_heap_slots() == kVariablePartIndex &&
scope_info->ContextLength() == 0));
DCHECK(scope->num_heap_slots() == scope_info->ContextLength());
return scope_info;
}