LGTMThere is some more cleanup that can be done but it can be done in a follow up if
you prefer.
https://codereview.chromium.org/1007783002/diff/80001/src/parser.cc File src/parser.cc (left): https://codereview.chromium.org/1007783002/diff/80001/src/parser.cc#oldcode2251 src/parser.cc:2251: ReportMessage("strict_const"); Can strict_const be removed now? Fine to do in a followup. https://codereview.chromium.org/1007783002/diff/80001/test/mjsunit/es6/block-conflicts.js File test/mjsunit/es6/block-conflicts.js (right): https://codereview.chromium.org/1007783002/diff/80001/test/mjsunit/es6/block-conflicts.js#newcode5 test/mjsunit/es6/block-conflicts.js:5: // Flags: Remove line? https://codereview.chromium.org/1007783002/diff/80001/test/mjsunit/es6/block-early-errors.js File test/mjsunit/es6/block-early-errors.js (right): https://codereview.chromium.org/1007783002/diff/80001/test/mjsunit/es6/block-early-errors.js#newcode28 test/mjsunit/es6/block-early-errors.js:28: // Flags: remove https://codereview.chromium.org/1007783002/diff/80001/test/mjsunit/es6/block-for.js File test/mjsunit/es6/block-for.js (right): https://codereview.chromium.org/1007783002/diff/80001/test/mjsunit/es6/block-for.js#newcode28 test/mjsunit/es6/block-for.js:28: // Flags: etc https://codereview.chromium.org/1007783002/ -- -- 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.
