https://codereview.chromium.org/1227163011/diff/20001/src/x64/builtins-x64.cc
File src/x64/builtins-x64.cc (right):
https://codereview.chromium.org/1227163011/diff/20001/src/x64/builtins-x64.cc#newcode181
src/x64/builtins-x64.cc:181: __ Push(rdx);
On 2015/07/10 16:21:51, arv (Not doing code reviews) wrote:
Do we care about the extra cost of the push/pop here?
The push/pop is right before and after a runtime call, which in turn
goes through the CEntryStub that sets up and tears down an entire
exit-frame. Also, slack tracking runs for exactly 8 allocations per per
initial map. Also, Runtime_FinalizeInstanceSize will perform a full
traversal over the entire transition tree of the initial map. So no, I
don't think we care.
https://codereview.chromium.org/1227163011/
--
--
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/d/optout.
