made maximum AST depth configurable#6893
Conversation
ac15185 to
1761974
Compare
|
What is the use case for this? |
Getting rid of "hidden" thresholds and bailouts (and too many branches). There is also something going on with |
|
I don't really see the idea right now neither. If the constant is only used in 1 file it does not hurt that it is defined there. you also made it non-const and there is no good reason for that right now. I don't feel convinced that this should be a runtime configuration option. |
|
The current limit is way too high and still might have significant impact on performance. So some people might need to lower that. Making it configurable will make it easier to test for a more reasonable value. But the threshold might mot be helpful because in case it will hit something beforehand will already have consumed a considerable amount of time. I encountered that while trying to make Sorry for forgetting to mention this beforehand. |
No description provided.