Skip to content
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

Asr pass refactor #2795

Merged
merged 12 commits into from
Aug 16, 2024
Merged

Conversation

tanay-man
Copy link
Contributor

@tanay-man tanay-man commented Aug 2, 2024

fixes #2777

@Thirumalai-Shaktivel
Copy link
Collaborator

Thirumalai-Shaktivel commented Aug 16, 2024

Remember the following while creating a next PR:

  • After a final review, let's clean up the commits with proper commit message (short description about the changes made in each commits)
  • Proper PR heading which explain what changes were made.

Copy link
Collaborator

@Thirumalai-Shaktivel Thirumalai-Shaktivel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Thirumalai-Shaktivel Thirumalai-Shaktivel marked this pull request as ready for review August 16, 2024 15:02
@Thirumalai-Shaktivel Thirumalai-Shaktivel enabled auto-merge (squash) August 16, 2024 15:04
@Thirumalai-Shaktivel Thirumalai-Shaktivel merged commit 61a27d5 into lcompilers:main Aug 16, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor the StructConstructor
2 participants