https://codereview.chromium.org/1234213004/diff/1/src/parser.cc
File src/parser.cc (right):
https://codereview.chromium.org/1234213004/diff/1/src/parser.cc#newcode4383
src/parser.cc:4383: if (fvar != nullptr) {
On 2015/07/16 13:21:05, rossberg wrote:
On 2015/07/16 13:03:57, caitp wrote:
> On 2015/07/16 12:53:47, rossberg wrote:
> > Hm, why do you need to move (and duplicate) fvar initialisation
and
> IsConstruct
> > assert?
>
> it seems that these need to happen before the init block. I suppose
it could
> just use `result->insertAt(0)` instead though
Not sure I understand. They were happening before the init block
before, AFAICT?
Why did that need to change at all?
I've refactored this, it's much cleaner now
https://codereview.chromium.org/1234213004/
--
--
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.
