Replace math.h with cmath

Replace math.h with cmath

This will make it easier to use other STL headers in the future

Committed: http://code.google.com/p/v8/source/detail?r=14352

[jochen (gone - plz use gerrit), 2013-04-19 10:33:44]

plz review

I checked that it builds on win, linux, and android. I don't have access to
solaris :-/

[danno, 2013-04-19 13:08:33]

generally looks ok, but a comment and question.

https://codereview.chromium.org/14362023/diff/8001/src/assembler.cc
File src/assembler.cc (right):

https://codereview.chromium.org/14362023/diff/8001/src/assembler.cc#newcode1510
src/assembler.cc:1510: return OS::nan_value();
Please put {} around the return.

https://codereview.chromium.org/14362023/diff/8001/src/conversions.cc
File src/conversions.cc (right):

https://codereview.chromium.org/14362023/diff/8001/src/conversions.cc#newcode39
src/conversions.cc:39: using std::fpclassify;
Did you actually test this?

[jochen (gone - plz use gerrit), 2013-04-19 13:12:25]

https://chromiumcodereview.appspot.com/14362023/diff/8001/src/assembler.cc
File src/assembler.cc (right):

https://chromiumcodereview.appspot.com/14362023/diff/8001/src/assembler.cc#ne...
src/assembler.cc:1510: return OS::nan_value();
On 2013/04/19 13:08:33, danno wrote:
> Please put {} around the return.

Done.

https://chromiumcodereview.appspot.com/14362023/diff/8001/src/conversions.cc
File src/conversions.cc (right):

https://chromiumcodereview.appspot.com/14362023/diff/8001/src/conversions.cc#...
src/conversions.cc:39: using std::fpclassify;
On 2013/04/19 13:08:33, danno wrote:
> Did you actually test this?
Yes, golem bails out without this

[danno, 2013-04-19 13:23:29]

lgtm, I'll land this for you

[danno, 2013-04-19 13:27:13]

Committed patchset #6 manually as r14352 (presubmit successful).