Extend JSBuiltinReducer to cover Math.fround as well.

Extend JSBuiltinReducer to cover Math.fround as well.

R=bmeurer@chromium.org
TEST=compiler-unittests/JSBuiltinReducerTest.MathFround

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

[Benedikt Meurer, 2014-09-24 11:37:51]

LGTM with nit and assuming that there's a system test for this. :-)

https://codereview.chromium.org/594183004/diff/1/src/compiler/js-builtin-redu...
File src/compiler/js-builtin-reducer-unittest.cc (right):

https://codereview.chromium.org/594183004/diff/1/src/compiler/js-builtin-redu...
src/compiler/js-builtin-reducer-unittest.cc:196: EXPECT_TRUE(r.Changed());
Nit: ASSERT_TRUE

[Michael Starzinger, 2014-09-24 13:12:25]

Addressed comments. Landing.

https://codereview.chromium.org/594183004/diff/1/src/compiler/js-builtin-redu...
File src/compiler/js-builtin-reducer-unittest.cc (right):

https://codereview.chromium.org/594183004/diff/1/src/compiler/js-builtin-redu...
src/compiler/js-builtin-reducer-unittest.cc:196: EXPECT_TRUE(r.Changed());
On 2014/09/24 11:37:51, Benedikt Meurer wrote:
> Nit: ASSERT_TRUE

Done.

[Michael Starzinger, 2014-09-24 14:55:26]

Committed patchset #2 (id:20001) manually as 24187 (presubmit successful).

[chunyang.dai, 2014-09-25 10:00:39]

hello. Michael.
This test case will fail on turbofan unsupported platform. For example, X87.
( linkage.cc::141  UNIMPLEMETED code ).

Is it possible to disable it for these platform?

thanks.

[Michael Starzinger, 2014-09-25 10:58:26]

On 2014/09/25 10:00:39, chunyang.dai wrote:
> hello. Michael.
> This test case will fail on turbofan unsupported platform. For example, X87.
> ( linkage.cc::141  UNIMPLEMETED code ).
> 
> Is it possible to disable it for these platform?
> 
> thanks.

Instead of disabling this test I would propose the following fix:
https://codereview.chromium.org/603923002/