I prefer 1). I will upload a separate CL on StackArgumentsAccessor for X64 for
the recent commits:
  1) https://codereview.chromium.org/23903053
  2) your part2 and part3
Could you and dcarney help me review it next week?

After that, danno could continue reviewing
https://codereview.chromium.org/22267005/ for the other places.

On 2013/09/27 13:14:29, marja wrote:
Alright, so the end result should be that x64 uses StackArgumentAccessor.

haitao.feng: We can do any of these:
1) I land this as is, you rebase your CL (which starts using
StackArgumentAccessor) on top
2) I start using StackArgumentAccessor in this CL (merge parts of your CL into
it)
3) You land your CL first and I rebase on top of it

I won't work on this until next week. What's your preference? If you want to
go
ahead and land your CL first, I won't object.


https://codereview.chromium.org/24488006/

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