Performance javascript grammar about peggy HOT 7 CLOSED

If you turn on tracing, you might see the problem, with the same piece of input being parsed a lot.

from peggy.

Nod. You should still take a look at your grammar for ReDoS-style issues then. Places where you have (a*)+ for example.

from peggy.

Can you isolate this to a small grammar input that shows the same behavior? These sorts of things are sometimes ReDoS-style issues.

from peggy.

Yes that might be the case although is rather complex to reduce the grammar we have CallExpressions https://github.com/peggyjs/peggy/blob/main/examples/javascript.pegjs#L631MethodExpressions etc. Maybe creating a MultiCallExpression might be able to help.

from peggy.

@hildjj Fixed it on my end, but if anyone encounters this adding caching (or the lack of) was one of the culprits, it might have been on the migration from peg that the setting was removed. Thank you very much.

from peggy.

Did it work better with caching on, or with caching off? We might not have quite enough thought given to how caching interacts with the newer features.

from peggy.

Caching on, it made a huge difference.

from peggy.