Reviewers: Dmitry Lomov (chromium),
Message:
A WIP to fix the code-load regression. Before the offending patch, a
code-load
run locally took 903M instructions with ia32.release, after about 924M
instructions, which matches the measured regression. Simply commenting out
the
nested classifier in ParseAssignmentExpression removes the regression.
However
it is doing a thing for us :) so it would be nice to keep it.
This patch doesn't fix the problem however, instead it just golfs the count
down
to 918M instructions. Copying around these error is silly anyway; there
should
be a way to avoid doing that. Ideas welcome.
Description:
WIP to fix code-load regression
Regression was here:
https://chromium.googlesource.com/v8/v8/+/e73594c7fb3e6b5834b7ddfe78727fb994bab25f
[email protected]
LOG=N
BUG=
Please review this at https://codereview.chromium.org/1140813008/
Base URL: https://chromium.googlesource.com/v8/v8@master
Affected files (+65, -47 lines):
M src/preparser.h
--
--
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.
