On 2013/08/24 21:04:10, danno wrote:
And like I said in my original review, the ARM-specific changes for
handling
constants are not OK, they definitely don't look good to me. Please wait
to
even
consider landing this until I'm back from vacation on September 11th. I
also
+1
the splitting up of this CL, at first blush it looks possible.
We are going to rewrite and break down this patch. We will start by
uploading
patches improving lithium instructions (not corrupting input or generating
better code sequence). These changes are not affected by the issues
mentioned
above. We'll then proposed patches to address the issues discussed above.
https://codereview.chromium.org/23156006/
--
--
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.
