https://codereview.chromium.org/1272673003/diff/150001/src/preparser.h
File src/preparser.h (right):
https://codereview.chromium.org/1272673003/diff/150001/src/preparser.h#newcode34
src/preparser.h:34: mutable int rest_array_literal_index = -1;
On 2015/09/01 15:17:24, caitp wrote:
On 2015/09/01 14:41:20, wingo wrote:
> Is mutable needed? I think not, according to uses in this file.
It is needed by the pattern rewriter, because I was asked to remove
the AST node
(although bmeurer seems to prefer the AST node --- I'm getting
confused by
conflicting requests D:).
To be clear about the conflicting requests: I only objected to the use
of an AST node given the use of that node. If we go down the road of
supporting this in codegen (again), then clearly an AST node is
required.
https://codereview.chromium.org/1272673003/
--
--
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.
