Skip to content

fix: 'else if' is now lowered into an else > if > else#1620

Open
Angus-Bethke-Bachmann wants to merge 1 commit intomasterfrom
antb/PRG-3920
Open

fix: 'else if' is now lowered into an else > if > else#1620
Angus-Bethke-Bachmann wants to merge 1 commit intomasterfrom
antb/PRG-3920

Conversation

@Angus-Bethke-Bachmann
Copy link
Contributor

Changed

  • 'else if' is now lowered into an else > if > else to ensure functions that return aggregate are not always evaluated when a previous control statement returns true

Testing

  • Added a lit test to reproduce the error and confirmed that the error is resolved
  • Added a unit test to verify the AST changes

@Angus-Bethke-Bachmann Angus-Bethke-Bachmann linked an issue Mar 5, 2026 that may be closed by this pull request
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.

ELSIF conditional regression

1 participant