-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
evaluate AST_SymbolRef as parameter #1477
Conversation
(Travis CI timed out.) |
Nice. LGTM |
Discovered bug while testing with |
It would be good to run This one uses real life code rather than synthetic and/or unit tests: |
So I stepped on a landmine which is Given the intertwined nature of this bug, I guess I should fix it alongside this PR. |
Phew, I think this is sorted and ready to go now 😉 |
fix invalid boolean conversion now exposed in `make_node_from_constant()`
00e22be
to
33472c6
Compare
I couldn't find those files in a VCS repository - should I just download a copy of everything from http://browserbench.org/JetStream/in-depth.html and work from there? Edit: #1479 |
I see you've changed Also, you've dropped |
I tried running Jetstream locally with node but errored out because it can't find the source for the function |
|
@alexlamsl Regarding your explanation above - were you able to create a such a failing test case against |
@kzc the cases I run into where I could continue to investigate further to see if other |
fix invalid boolean conversion now exposed in `make_node_from_constant()` closes mishoo#1477
Given the current `OPT()` node: - `optimize(compressor)` works on the same level, and will only `descend()` if returned instance is different - `transform(compressor)` works on child nodes, and will always `descend()` Other fixes - `loopcontrol_target()` should compare against `compressor.self()` since the current node may have already been displaced - remove obsolete optimisation in `AST_Binary` after mishoo#1477 fixes mishoo#1592
- remove obsolete optimisation in `AST_Binary` after mishoo#1477 - improve `TreeWalker.has_directive()` readability and resilience against multiple visits
- remove obsolete optimisation in `AST_Binary` after #1477 - improve `TreeWalker.has_directive()` readability and resilience against multiple visits
@kzc this addresses the quirk you've mentioned in #1473 (comment)