On 2013/02/21 08:46:26, Jakob wrote:
https://codereview.chromium.org/12281019/diff/1/src/hydrogen-instructions.cc
File src/hydrogen-instructions.cc (right):


https://codereview.chromium.org/12281019/diff/1/src/hydrogen-instructions.cc#newcode2749
src/hydrogen-instructions.cc:2749: new(zone) HConstant(FACTORY->NewNumber(val,
TENURED),                          \
On 2013/02/19 14:39:07, Yang wrote:
> As discussed offline, there is no need to allocate in the double case
either.

Done.

LGTM.

https://chromiumcodereview.appspot.com/12281019/

--
--
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.


Reply via email to