- 03 Aug, 2011 8 commits
-
-
danno@chromium.org authored
Advantage is that it's much easier to add new element types (like FAST_SMI_ELEMENTS), and that handling logic for each element kind is (more) consolidated. Currently, only GetElementsWithReceiver uses the new encapsulation, but the goal is to move much more element functionality into the class incrementally. BUG=none TEST=none Review URL: http://codereview.chromium.org/7527001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8810 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
R=ricow@chromium.org Review URL: http://codereview.chromium.org/7465079 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8805 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
* src/hydrogen-instructions.cc (HValue::InferRange): Only mark values with int32 representation as never being -0. Always return a non-NULL value; callers should check for representation().IsNone() if that's their concern. In practice these invariants were not violated by callers, but they were sometimes two calls away, which seems brittle. BUG= TEST=tests pass, modulo http://code.google.com/p/v8/issues/detail?id=1572 Review URL: http://codereview.chromium.org/7514040 Patch from Andy Wingo <wingo@igalia.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8804 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
This allows to handle code like: var a = arguments; while (smth) a[i]; without bailout. R=kmillikin@chromium.org Review URL: http://codereview.chromium.org/7465078 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8803 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Ported r8713 (6bdadc4) BUG= TEST= Review URL: http://codereview.chromium.org/7564003 Patch from Paul Lind <plind44@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8802 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Ported r8781 (ca7888b) BUG= TEST= Review URL: http://codereview.chromium.org/7563002 Patch from Paul Lind <plind44@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8801 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
This reverts r8766. TEST=WebKit LayoutTests green again. Review URL: http://codereview.chromium.org/7562005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8800 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
Reapply r8783 with an additional fix. Because the preparser and parser do not use the same scope analysis to determine if a function can be lazily compiled, the parser can have false positives. Rather than treating this as a parse error, treat the preparser as authoritative and eagerly compile the function. R=lrn@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7565003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8797 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 02 Aug, 2011 11 commits
-
-
vegorov@chromium.org authored
Space should be reserved twice before each deserialization. R=erik.corry@gmail.com Review URL: http://codereview.chromium.org/7550009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8795 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
This reverts r8783. R=vegorov@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7550013 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8794 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
Patch by Bert Belder. Review URL: http://codereview.chromium.org/7549009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8793 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
Patch by Bert Belder. Review URL: http://codereview.chromium.org/7553011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8791 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
The condition code for branching on the result of vcmp was wrong, effectively swapping the true/false branches. TEST=regress-lbranch-double.js Review URL: http://codereview.chromium.org/7553010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8784 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
Function declarations inside catch are hoisted to the nearest enclosing function scope, but we compiled their bodies as if occurring inside the catch scope. BUG=chrome:91120 TEST=regress/regress-91120 attached Review URL: http://codereview.chromium.org/7548011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8783 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=jkummerow@chromium.org BUG=91008 TEST=regress-91008.js Review URL: http://codereview.chromium.org/7542008 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8782 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
BUG=chromium:91013 TEST=test/mjsunit/regress/regress-91013.js Review URL: http://codereview.chromium.org/7551009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8781 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
BUG=91245 Review URL: http://codereview.chromium.org/7538028 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8780 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
HGraphBuilder::TryArgumentsAccess does not emit any uses for receiver and will generate incorrect code when receiver for a property access is defined by a phi that returns either arguments object or something else. BUG=v8:1582 TEST=test/mjsunit/regress/regress-1582.js Review URL: http://codereview.chromium.org/7553006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8774 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
BUG=91013 TEST=regress91013.js Review URL: http://codereview.chromium.org/7551004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8773 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 01 Aug, 2011 5 commits
-
-
yangguo@chromium.org authored
BUG=v8:1579 Review URL: http://codereview.chromium.org/7537023 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8771 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Review URL: http://codereview.chromium.org/7541020 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8769 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Handle oddballs on ia32 via root indices, similar to other platforms. Added a special case for Smi types on ia32 to make lithium code generation on both Intel platforms more similar. Review URL: http://codereview.chromium.org/7544012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8767 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
as per ES5. BUG=89490 TEST=es5conform 15.1.*, 15.2.3.*; mjsunit/undeletable-functions Review URL: http://codereview.chromium.org/7538016 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8766 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
Review URL: http://codereview.chromium.org/7538017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8764 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 29 Jul, 2011 2 commits
-
-
yangguo@chromium.org authored
Review URL: http://codereview.chromium.org/7523052 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8763 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
It caused a regressions in compile time in the optimizing compiler. I'm reverting until the reason for this is identified and fixed. TBR=whesse@chromium.org Review URL: http://codereview.chromium.org/7453024 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8762 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 28 Jul, 2011 10 commits
-
-
mstarzinger@chromium.org authored
R=vitalyr@chromium.org TEST=cctest/test-dictionary Review URL: http://codereview.chromium.org/7385006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8761 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Review URL: http://codereview.chromium.org/7524025 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8760 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Review URL: http://codereview.chromium.org/7482044 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8759 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
To do this, the Branch instruction needs to carry around a temporary register, but only when the crankshafted code will make a map access. When the crankshafted code sees an object of a type it hasn't encountered before, it will always trigger a deopt. Another option in theses cases would be calling a ToBooleanStub which can handle all types, but then one has to be careful to *not* trigger a GC (which is currently a bit tricky to achieve). Const-corrected ToBoolean::Types. Moved the NeedsMap logic into ToBoolean::Types itself, where it belongs. This patch improves a lot of benchmarks, crypto-orig even by 16.7%, but slows down others. The slowdown has to be investigated, but I'd like to get this patch out first to fix the flakiness problems we currently have due to the previous crankshafted ToBoolean. Review URL: http://codereview.chromium.org/7461107 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8758 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
Review URL: http://codereview.chromium.org/7482042 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8757 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
keuchel@chromium.org authored
BUG= TEST=mjsunit/scope-calls-eval.js Review URL: http://codereview.chromium.org/7464030 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8756 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
whesse@chromium.org authored
BUG= TEST= Review URL: http://codereview.chromium.org/7097011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8755 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
TBR=ricow@chromium.org Review URL: http://codereview.chromium.org/7523025 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8754 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Landing two patches contributed by Andy Wingo: http://codereview.chromium.org/7514040/ and http://codereview.chromium.org/7516001/ Review URL: http://codereview.chromium.org/7520022 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8753 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ricow@chromium.org authored
Review URL: http://codereview.chromium.org/7522015 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8750 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 27 Jul, 2011 4 commits
-
-
danno@chromium.org authored
R=svenpanne@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org/7477047 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8749 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=ricow@chromium.org BUG=non TEST=running urls from reliability bots Review URL: http://codereview.chromium.org/7497010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8748 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
This fixes the issues when GC happens when invoking ToBooleanStub. Review URL: http://codereview.chromium.org/7465066 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8747 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
whesse@chromium.org authored
BUG=v8:1572 TEST=mjsunit/regress/regress-crbug-l84186.js Review URL: http://codereview.chromium.org/7480044 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8746 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-