Skip to content

fix issue 490#492

Closed
ayyyabean-alt wants to merge 1 commit intodadhi:masterfrom
ayyyabean-alt:issue490
Closed

fix issue 490#492
ayyyabean-alt wants to merge 1 commit intodadhi:masterfrom
ayyyabean-alt:issue490

Conversation

@ayyyabean-alt
Copy link

@ayyyabean-alt ayyyabean-alt commented Jan 23, 2026 •
edited
Loading

  • add unittest for issue490
  • fix issue490, Remove unnecessary and erroneous calls to the EmitStoreAndLoadLocalVariableAddress method.

@ayyyabean-alt
fix issue 490 (#1)
db36173 
* add unittest for issue490

* fix issue490
@dadhi
Copy link
Owner

dadhi commented Feb 28, 2026

Hi @ayyyabean-alt
Sorry, honestly I've missed your PR.
Let keep it open, I will use what possible from the fix and test.

@dadhi dadhi reopened this Feb 28, 2026
dadhi pushed a commit that referenced this pull request Mar 2, 2026
added tests (passing) from the #492 PR by @ayyyabean-alt
9b1e8ec
@dadhi
Copy link
Owner

dadhi commented Mar 2, 2026

I have added the tests, they passing. I will close the PR.
Thank you @ayyyabean-alt

@dadhi dadhi closed this Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ayyyabean-alt @dadhi