-
Notifications
You must be signed in to change notification settings - Fork 158
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
Store initializer exprs for Structs in ASR #2056
Conversation
I think that this is fine. Isn't it equivalent to store an iniitalizer in the Variable (inside a struct) or in the Struct? |
Yeah, that works. But it would require a lot of methods to be moved in CommonVisitor and may create problems of visiting them again in BodyVisitor which may affect the performance. |
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.
Ok. I think this PR is reasonably clean and doesn't require major refactor. So let's merge it. I think this still keeps the door open to refactor things later if we figure out a better design.
No description provided.