|
Do not look for the slot in the ScopeInfo's global range when it's not necessary.
This fixes the Runtime_DeclareGlobals performance regression caused by a huge number of global var declarations mentioned in chromium:517778.
BUG= chromium:517778
LOG=N
Committed: https://crrev.com/a1c1e2bd44ca4f5381c47b34be7648e94bf775d7
Cr-Commit-Position: refs/heads/master@{#30679}
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+82 lines, -91 lines) |
Patch |
|
M |
src/contexts.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
src/contexts.cc
|
View
|
|
2 chunks |
+4 lines, -5 lines |
0 comments
|
Download
|
|
M |
src/debug/debug-evaluate.cc
|
View
|
|
1 chunk |
+3 lines, -4 lines |
0 comments
|
Download
|
|
M |
src/debug/debug-frames.cc
|
View
|
|
1 chunk |
+2 lines, -3 lines |
0 comments
|
Download
|
|
M |
src/debug/debug-scopes.cc
|
View
|
|
1 chunk |
+2 lines, -4 lines |
0 comments
|
Download
|
|
M |
src/objects.h
|
View
|
|
1 chunk |
+13 lines, -4 lines |
0 comments
|
Download
|
|
M |
src/runtime/runtime-debug.cc
|
View
|
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
|
M |
src/scopeinfo.h
|
View
|
|
5 chunks |
+6 lines, -30 lines |
0 comments
|
Download
|
|
M |
src/scopeinfo.cc
|
View
|
1
|
7 chunks |
+43 lines, -33 lines |
0 comments
|
Download
|
|
M |
src/scopes.cc
|
View
|
|
1 chunk |
+8 lines, -4 lines |
0 comments
|
Download
|
Total messages: 9 (4 generated)
|