- 10 Aug, 2011 1 commit
-
-
fschneider@chromium.org authored
* Bug fix for range analysis (contributed by Andy Wingo). Ranges of double values have to include negative zero. Original code review: http://codereview.chromium.org/7514040/ * Fix a bug in optimized Math.round on ARM. When emitting minus-zero checks we previously return a wrong result because of incorrect register assignment. * Fix performance problem in IA32 and x64. Refine the checks for minus zero and avoid unnecessary deoptimizations on Math.floor. * Improve mjsunit test for Math.round to make sure we also get the optimized version of the code for each test case. Review URL: http://codereview.chromium.org/7604028 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8877 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 04 Aug, 2011 1 commit
-
-
kmillikin@chromium.org authored
This change reportedly causes a slowdown or inifinite loop on ARM. Revert pending investigation. R=vegorov@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7566040 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8834 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 03 Aug, 2011 1 commit
-
-
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
-
- 14 Apr, 2011 1 commit
-
-
lrn@chromium.org authored
TEST=mjsunit/math-floor Review URL: http://codereview.chromium.org/6835021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7613 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 13 Apr, 2011 1 commit
-
-
lrn@chromium.org authored
Make math-round.js test check both normal and optimized version. Add some cases to the tests. BUG=v8:958 TEST=mjsunit/math-round Review URL: http://codereview.chromium.org/6837018 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7599 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 18 Mar, 2010 1 commit
-
-
oleg@chromium.org authored
Improve Math.round(). Fix the bug in r4146. Further improve performance by checking the exponent instead of comparing doubles. Add several tests for numbers near the limits of SMI and several tests from WebKit. Review URL: http://codereview.chromium.org/1008004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@4180 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 02 Feb, 2010 1 commit
-
-
lrn@chromium.org authored
Review URL: http://codereview.chromium.org/567011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3764 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 01 Feb, 2010 1 commit
-
-
kmillikin@chromium.org authored
Code is specialized to the initial receiver. Review URL: http://codereview.chromium.org/553149 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3760 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 29 Jan, 2010 1 commit
-
-
fschneider@chromium.org authored
This change adds a post-order numbering to AST nodes that are relevant for the fast code generator. It is only invoked together with the fast compiler. Also changed the ast printer to print the numbering for testing purposes if it is present. Review URL: http://codereview.chromium.org/553134 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3738 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 27 Jan, 2010 1 commit
-
-
kmillikin@chromium.org authored
simpler backend for non-toplevel code. Review URL: http://codereview.chromium.org/549158 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3721 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 11 Dec, 2009 1 commit
-
-
erik.corry@gmail.com authored
Review URL: http://codereview.chromium.org/488017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3454 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 05 Nov, 2009 1 commit
-
-
lrn@chromium.org authored
Review URL: http://codereview.chromium.org/361026 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3223 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 25 May, 2009 1 commit
-
-
mikhail.naganov@gmail.com authored
This issue was raised by Brett Wilson while reviewing my changelist for readability. Craig Silverstein (one of C++ SG maintainers) confirmed that we should declare one namespace per line. Our way of namespaces closing seems not violating style guides (there is no clear agreement on it), so I left it intact. Review URL: http://codereview.chromium.org/115756 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@2038 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 09 Sep, 2008 1 commit
-
-
christian.plesner.hansen@gmail.com authored
Added presubmit step to check copyright. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@242 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 03 Jul, 2008 1 commit
-
-
christian.plesner.hansen authored
git-svn-id: http://v8.googlecode.com/svn/trunk@2 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-