Don't use an explicit s0 in ClampDoubleToUint8.

Don't use an explicit s0 in ClampDoubleToUint8.

R=fschneider@chromium.org
BUG=v8:2004
TEST=test/mjsunit/pixel-array-rounding.js

Committed: https://code.google.com/p/v8/source/detail?r=11053

[fschneider, 2012-03-15 09:17:05]

It seems that the tests in test-api/PixelArray do not exercise the optimizing
compiler well enough.

I'd strongly prefer changing those tests so that they all hit the optimizing
compiler (including the one extracted test case). The easiest is probably to
move them from cctest to a mjsunit test since we now have support for allocating
typed arrays in d8.

LGTM, otherwise.

[danno, 2012-03-15 09:51:12]

Agreed. The test are all pre-d8 typed array support and even
pre-OptimizeFunctionOnNextCall. However, I am going to save that for another CL,
since it's a chunk of refactoring that I don't want to couple with this fix.

On 2012/03/15 09:17:05, fschneider wrote:
> It seems that the tests in test-api/PixelArray do not exercise the optimizing
> compiler well enough.
> 
> I'd strongly prefer changing those tests so that they all hit the optimizing
> compiler (including the one extracted test case). The easiest is probably to
> move them from cctest to a mjsunit test since we now have support for
allocating
> typed arrays in d8.
> 
> LGTM, otherwise.