cdiggins / myna-parser

Myna Parsing Library
https://cdiggins.github.io/myna-parser
MIT License
78 stars 15 forks source link

Possible performance degradation under Chrome Canary 61 #11

Closed bd82 closed 7 years ago

bd82 commented 7 years ago

Looks like Myna only runs much slower on Chrome Canary 61 (which includes the new TurboFan V8 compiler and Ignition interpreter).

I don't know if this is a bug in Chrome that will be fixed by the time an official version of Chrome is released or if it is a a real issue.

Just an heads up...

Also Note:

cdiggins commented 7 years ago

Thanks for letting me kno! Closing this issue, since there isn't much I can do about it now. I will be looking into the performance improvements suggested in #13 .

bd82 commented 7 years ago

You can reproduce it now on latest Chrome stable version (59). There is a new JavaScript optimizing compiler there (Ignition + turbofan)

I guess the new V8 is much slower for some code patterns used in Myna. But I have no idea what / which / why.

Perhaps you need to compare CPU profiling results on different Chrome versions to figure this out.

Sounds like a pain 😢