|
|
Remove unneeded variable usage analysis.
A variable usage analysis pass was run on toplevel and lazily-compiled
code but never used. Remove this pass and the data structures it
builds.
The representation of variable usage for Variables has been changed
from a struct containing a (weighted) count of reads and writes to a
simple flag. VariableProxies are always used, as before. The unused
"object uses" is removed.
Committed: http://code.google.com/p/v8/source/detail?r=4052
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+22 lines, -610 lines) |
Patch |
 |
M |
src/SConscript
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
src/ast.h
|
View
|
|
2 chunks |
+0 lines, -6 lines |
0 comments
|
Download
|
|
M |
src/ast.cc
|
View
|
|
2 chunks |
+1 line, -4 lines |
0 comments
|
Download
|
|
M |
src/compiler.cc
|
View
|
|
2 chunks |
+2 lines, -3 lines |
0 comments
|
Download
|
|
M |
src/flag-definitions.h
|
View
|
|
1 chunk |
+0 lines, -3 lines |
0 comments
|
Download
|
|
M |
src/parser.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
src/scopeinfo.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
|
src/scopes.cc
|
View
|
|
5 chunks |
+11 lines, -12 lines |
0 comments
|
Download
|
|
D |
src/usage-analyzer.h
|
View
|
|
1 chunk |
+0 lines, -40 lines |
0 comments
|
Download
|
|
D |
src/usage-analyzer.cc
|
View
|
|
1 chunk |
+0 lines, -426 lines |
0 comments
|
Download
|
|
M |
src/variables.h
|
View
|
|
3 chunks |
+3 lines, -33 lines |
0 comments
|
Download
|
|
M |
src/variables.cc
|
View
|
|
2 chunks |
+1 line, -51 lines |
0 comments
|
Download
|
|
M |
tools/gyp/v8.gyp
|
View
|
|
2 chunks |
+2 lines, -4 lines |
0 comments
|
Download
|
|
M |
tools/visual_studio/v8_base.vcproj
|
View
|
|
1 chunk |
+0 lines, -8 lines |
0 comments
|
Download
|
|
M |
tools/visual_studio/v8_base_arm.vcproj
|
View
|
|
1 chunk |
+0 lines, -8 lines |
0 comments
|
Download
|
|
M |
tools/visual_studio/v8_base_x64.vcproj
|
View
|
|
1 chunk |
+0 lines, -8 lines |
0 comments
|
Download
|
Total messages: 2 (0 generated)
|
Chromium Code Reviews
Issue
669270:
Remove unneeded variable usage analysis. (Closed)
