Looking good.
https://codereview.chromium.org/113343003/diff/1/src/arm/full-codegen-arm.cc
File src/arm/full-codegen-arm.cc (right):
https://codereview.chromium.org/113343003/diff/1/src/arm/full-codegen-arm.cc#newcode3705
src/arm/full-codegen-arm.cc:3705: __ CallRuntime(Runtime::kMath_log, 1);
We could directly call %Math_log from math.js and get rid of %_Math_log.
Then again, that would mean that if Math.log is optimized by itself and
not inlined into its caller, it would be very slow.
https://codereview.chromium.org/113343003/diff/1/src/serialize.cc
File src/serialize.cc (right):
https://codereview.chromium.org/113343003/diff/1/src/serialize.cc#newcode434
src/serialize.cc:434: 31,
Do we want to change the numbers here now that 30 has been removed?
https://codereview.chromium.org/113343003/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
