-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixes explicit deref block #22093
fixes explicit deref block #22093
Conversation
of nkDerefExpr, nkHiddenDeref: | ||
of nkDerefExpr: | ||
result = transformAddrDeref(c, n, {nkAddr, nkHiddenAddr}) | ||
of nkHiddenDeref: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this change. At this point there should be no difference between nkDerefExpr
and nkHiddenDeref
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This transformation was done for lent
's sake. I think it's safe not to transform nkDerefExpr
. After all, the old change didn't transform it as well. It was me that introduced the extra transformation for nkDerefExpr
.
ref b1fe169
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least, it should be much better than before because the block transformation is not necessary for nkDerefExpr
, but I can also investigate it further.
Merging it because it improves the situation but we need to investigate further. |
Thanks for your hard work on this PR! Hint: mm: orc; opt: speed; options: -d:release |
fixes explicit deref
fixes status-im/nim-stew#198