Skip to content

Conversation

@phate
Copy link
Owner

@phate phate commented Feb 10, 2026

This PR reimplements the loop unswitching transformation. It performs the following changes compared to the previous implementation:

  1. Fixes a bug where the wrong origins for variables was picked up. In other words, the old transformation was broken.
  2. Avoids the duplication of the exit subregion (the old transformation duplicates it)
  3. Adds documentation to avoid confusion in the future.

Close #895

@phate phate requested a review from haved February 10, 2026 06:14
haved
haved previously approved these changes Feb 11, 2026
@haved
Copy link
Collaborator

haved commented Feb 11, 2026

@phate looks good, but HLS is unhappy

@phate
Copy link
Owner Author

phate commented Feb 11, 2026

@haved I adjusted the golden cycles to appease HLS

@phate phate enabled auto-merge (squash) February 11, 2026 07:14
@phate phate merged commit 14fb501 into master Feb 11, 2026
18 checks passed
@phate phate deleted the LoopUnswitching-Rework branch February 11, 2026 07:34
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.

Fix bug in ThetaGammaInversion

2 participants