[局部变量扩展]同步上游更改 - #310
Open
SimonShiki wants to merge 5 commits into
Open
Conversation
Signed-off-by: SimonShiki <sinangentoo@gmail.com>
Signed-off-by: SimonShiki <sinangentoo@gmail.com>
Signed-off-by: SimonShiki <sinangentoo@gmail.com>
Contributor
|
大概看了一遍 change，看起来应该可以直接上线，没有什么兼容问题？ |
* fix: 遍历列表行为和原版保持一致；range行为和原版保持一致； fix: 更改局部变量名的生成方式 * fix: 遍历列表行为和原版保持一致；range行为和原版保持一致； fix: 更改局部变量名的生成方式 * fix: 移除_scopeVarPool * fix: 修复了非编译模式下由于ccw自制返回值积木没有stackFrame导致的bug
不使用pushStack模拟非编译行为，而是直接用对象存放数据，提高效率
Contributor
|
快1年了，没时间做单元测试吗 |
Contributor
|
距离该PR的最后一个提交的时间已经过去1年了。 |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
概述
同步了原局部变量扩展近期上游更改。
具体内容
增加了对动态变量名的支持
通过扫描静态变量名，当线程内存在动态变量名时自动回退到更慢的变量管理模式。
同步编译/非编译模式局部变量行为
如题，在静态变量名模式保证原先性能的同时尽可能将一部分运行期代码提前至编译期
增加 range 积木
建议
当 range 积木域末存在 在兼容层中运行的积木(如未进行编译器兼容的扩展积木，原版无法复现，亦在原扩展中得到修复looks_say) 会阻塞运行