Extend JSBuiltinReducer to cover Math.abs as well.

Extend JSBuiltinReducer to cover Math.abs as well.

R=titzer@chromium.org
TEST=compiler-unittests/JSBuiltinReducerTest.MathAbs

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

[Michael Starzinger, 2014-09-26 13:32:25]

mstarzinger@chromium.org changed reviewers:
+ titzer@chromium.org
- bmeurer@chromium.org

[titzer, 2014-09-26 13:35:14]

lgtm with comment

https://codereview.chromium.org/605123004/diff/1/test/mjsunit/asm/math-abs.js
File test/mjsunit/asm/math-abs.js (right):

https://codereview.chromium.org/605123004/diff/1/test/mjsunit/asm/math-abs.js...
test/mjsunit/asm/math-abs.js:14: }
How about an int32 and uint32 case too?

[Michael Starzinger, 2014-09-26 13:59:39]

Addressed comments. Landing.

https://codereview.chromium.org/605123004/diff/1/test/mjsunit/asm/math-abs.js
File test/mjsunit/asm/math-abs.js (right):

https://codereview.chromium.org/605123004/diff/1/test/mjsunit/asm/math-abs.js...
test/mjsunit/asm/math-abs.js:14: }
On 2014/09/26 13:35:13, titzer wrote:
> How about an int32 and uint32 case too?

Done.

[Michael Starzinger, 2014-09-26 14:07:12]

Committed patchset #2 (id:20001) manually as 24255 (tree was closed).