Permalink
Browse files

Fix issue 962.

SplitBetween (formely known as Split with 3 arguments) should select split position from [start, end] instead of [start, end[. This should also improve allocation quality (remove certain redundant move patterns).

Also some minor renaming and refactoring to make register allocator code more readable.

BUG=v8:962
TEST=test/mjsunit/regress/regress-962.js

Review URL: http://codereview.chromium.org/5720001

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@5969 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
  • Loading branch information...
vegorov@chromium.org
vegorov@chromium.org committed Dec 10, 2010
1 parent 38343f7 commit 65f98b1e7acdd8ba70a599ca38b7a9d1b7193d33
Showing with 313 additions and 171 deletions.
  1. +201 −147 src/lithium-allocator.cc
  2. +59 −24 src/lithium-allocator.h
  3. +53 −0 test/mjsunit/regress/regress-962.js
Oops, something went wrong.

0 comments on commit 65f98b1

Please sign in to comment.