ACK regarding splitting the patchset.
https://codereview.chromium.org/660083005/diff/1/src/ast-numbering.cc
File src/ast-numbering.cc (right):
https://codereview.chromium.org/660083005/diff/1/src/ast-numbering.cc#newcode559
src/ast-numbering.cc:559: Handle<SharedFunctionInfo> shared, Zone* zone)
{
On 2014/10/22 13:08:52, Sven Panne wrote:
I don't think this actually belongs here: It just copies from the
FunctionLiteral to the SharedFunctionInfo. Perhaps this should live in
SharedFunctionInfo or even in some yet-to-be-implemented part of the
compilation
pipeline (frontend). Anyway, having another Renumber method is just
confusing.
ACK. Unfortunately the fix is a bit more involved, as this copying
usually should happen after Renumber is run, but Renumber is run from a
few places, so we need a central place to call Renumber --
yet-to-be-implemented, as you say.
https://codereview.chromium.org/660083005/
--
--
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.
