- 12 Sep, 2011 2 commits
-
-
kmillikin@chromium.org authored
Refactor Context::Lookup so it is more obvious. Change the comment in contexts.h so it no longer indicates that it can return an arguments object (it can't) and clean up the call sites that had leftover dead code. BUG= TEST= Review URL: http://codereview.chromium.org/7782030 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9223 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
It seems more appropriate as LiteralScope is declared in scanner.h R=lrn@chromium.org Signed-off-by: Thiago Farina <tfarina@chromium.org> Review URL: http://codereview.chromium.org/7864024 Patch from Thiago Farina <tfarina@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9222 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 09 Sep, 2011 8 commits
-
-
mikhail.naganov@gmail.com authored
As pointed out in: http://codereview.chromium.org/7754007/#msg5 "SmartPointer should have been named SmartArrayPointer as it expects an input allocated using new[] and deallocates it using delete[]. Using it as a simple scoped pointer for a single object is incorrect." R=mnaganov@chromium.org Review URL: http://codereview.chromium.org/7860011 Patch from Thiago Farina <tfarina@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9215 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=jkummrow@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org/7787007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9214 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=ricow@chromium.org BUG=95920 TEST=test/mjsunit/regress/regress-95920.js Review URL: http://codereview.chromium.org/7858031 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9213 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
When we propagate the information that a value is not convertible to integer, there is no need for a separate fixed-point computation, we can do things the "Millikin way" (tm), folding as much computation as possible into a single pass: ;-) We already have the phi node reachability information, so we can easily propagate this while doing the representation histogram computation. Review URL: http://codereview.chromium.org/7754010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9212 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Review URL: http://codereview.chromium.org/7859030 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9211 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=v8:796 Review URL: http://codereview.chromium.org/7754018 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9209 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=svenpanne@chromium.org BUG=none TEST=all Review URL: http://codereview.chromium.org/7849017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9208 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Review URL: http://codereview.chromium.org/7857031 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9207 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 08 Sep, 2011 19 commits
-
-
vitalyr@chromium.org authored
Review URL: http://codereview.chromium.org/7787013 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9206 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vitalyr@chromium.org authored
As pointed out in: http://codereview.chromium.org/7754007/ SmartPointer expects an input allocated using new[] and deallocates it using delete[]. So using SmartPointer for deleting T* here is incorrect. Fix it now. R=vitalyr@chromium.org Review URL: http://codereview.chromium.org/7846022 Patch from Thiago Farina <tfarina@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9205 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
This is all blank line before/after linting errors. Review URL: http://codereview.chromium.org/7754022 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9204 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
This is a renaming only change. R=erik.corry@gmail.com BUG= TEST= Review URL: http://codereview.chromium.org/7849020 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9200 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
Review URL: http://codereview.chromium.org/7849019 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9199 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mikhail.naganov@gmail.com authored
We were not disposing the semaphores and the thread used in SourceGroup. R=mnaganov@chromium.org Signed-off-by: Thiago Farina <tfarina@chromium.org> Review URL: http://codereview.chromium.org/7754007/ git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9198 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
Small cleanups in preparser. TEST=cctest/test-utils/SequenceCollectorRegression Review URL: http://codereview.chromium.org/7754014 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9197 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
TEST=compile with component=shared_library and soname_version=1.2.3 and find "libv8-1.2.3.so" in the output. Review URL: http://codereview.chromium.org/7848017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9196 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
R=lrn@chromium.org Signed-off-by: Thiago Farina <tfarina@chromium.org> Review URL: http://codereview.chromium.org/7739020 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9195 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Review URL: http://codereview.chromium.org/7847019 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9192 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Review URL: http://codereview.chromium.org/7851013 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9191 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
optimized string split. Review URL: http://codereview.chromium.org/7849012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9190 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
All the constructs that used it are now properly bracketed in the AST and we handle abrupt exits without try/finally. We can treat normal context exit as occurring implicitly at the end of a body. Review URL: http://codereview.chromium.org/7837025 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9189 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Review URL: http://codereview.chromium.org/7851011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9188 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
Port r9162 (2215df8). BUG= TEST= Review URL: http://codereview.chromium.org/7847015 Patch from Paul Lind <plind44@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9187 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Review URL: http://codereview.chromium.org/7835035 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9186 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
The use-def relation between phis is mainly "forwards" (i.e. from phis with smaller IDs to ones with larger IDs), so the fixed point computation terminates faster when iterate through the phis in a "backwards" manner. This is quite visible in complex Mandreel-generated code, where a few hundred phis with non-trivial use-def chains are generated. Review URL: http://codereview.chromium.org/7848012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9185 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
There is an optional parameter to the function, with default value being a null handle. We then check that this is a flat string. Review URL: http://codereview.chromium.org/7850011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9184 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
that eventuality. Review URL: http://codereview.chromium.org/7840031 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9183 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 07 Sep, 2011 11 commits
-
-
mikhail.naganov@gmail.com authored
This is so we can ignore libv8_g.so and libv8preparser_g.so when running git status. TEST=git status. You should not see the above libraries in the output. R=mnaganov@chromium.org Signed-off-by: Thiago Farina <tfarina@chromium.org> Review URL: http://codereview.chromium.org/7839036/ git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9182 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Review URL: http://codereview.chromium.org/7839031 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9181 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=v8:1662 Review URL: http://codereview.chromium.org/7782028 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9180 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Review URL: http://codereview.chromium.org/7795050 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9179 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Review URL: http://codereview.chromium.org/7739021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9178 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: http://codereview.chromium.org/7841035 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9177 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: http://codereview.chromium.org/7837026 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9176 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: http://codereview.chromium.org/7840029 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9175 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
Duplicate identifier detection must be an early syntax error in strict code, so errors in otherwise lazily compiled functions must be caught in the preparser. Originally introduced in r8541 and reverted in r8542. Now really compiles on Windows. Review URL: http://codereview.chromium.org/7782023 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9172 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
R=lrn@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7841034 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9171 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
The prototype of builtin functions is already unwritable, so we don't have to make it so (the default map for functions changes after builtins are initialized). We no longer need to make the prototype non-extensible, since all properties that are ever read by the bultins code has been added and frozen already. Adding properties to the prototype, or changing its __proto__, cannot affect code. Removing these two pieces of initialization code reduces the snapshot size by a few Kb. Review URL: http://codereview.chromium.org/7839028 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9166 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-