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

Separate overflow checks from arithmetic operations in SSA #6979

Closed
TomAFrench opened this issue Jan 7, 2025 · 0 comments · Fixed by #7011
Closed

Separate overflow checks from arithmetic operations in SSA #6979

TomAFrench opened this issue Jan 7, 2025 · 0 comments · Fixed by #7011
Assignees

Comments

@TomAFrench
Copy link
Member

As a step towards #6623 we need to have a representation of an overflow check in SSA which is separate from the arithmetic operations.

We have range check instructions already so we should be able to insert these along any arithmetic operations to maintain the current checked behaviour. This would however unlock more opportunities for instruction hoisting etc, as we can hoist the arithmetic while keeping the range checks under a conditional, etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants