refactoring on spreading #236
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tiye
force-pushed
the
call-spread
branch
2 times, most recently
from
6bd688a to
477a3b9
Compare
|
updated custom formatter |
|
wow |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
优化了一下 & 语法的 spreading 实现方案, 本机(计算 fibo34)又节省了 100ms
&判断改成了 Syntax 节点判断, 少了一些字符串对比的开销&?~~@四个语法做了这样的处理,&call-spread语法, 专门用于执行包含 spreading 的参数展开, 不对所有参数做检查但有个极端情况, 因为
&call-spread是在 preprocess 阶段自动插入的,defmacro内部用到时, 可能还未插入, 这个特殊的情况需要手动插入一个&call-spread调用.