-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
ARM64 - Add GT_CHK_DIV_BY_ZERO #69170
Conversation
Tagging subscribers to this area: @JulieLeeMSFT Issue DetailsThis is currently a draft, but the hope is to add a specific node that represents the divide-by-zero check.
|
…handling the GT_CHK_DIV_BY_ZERO case in morph.
@@ -10795,6 +10795,11 @@ void Compiler::fgValueNumberAddExceptionSet(GenTree* tree) | |||
fgValueNumberAddExceptionSetForBoundsCheck(tree); | |||
break; | |||
|
|||
#ifdef TARGET_ARM64 | |||
case GT_CHK_DIV_BY_ZERO: | |||
break; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will need to add an impl for the case here.
@@ -5099,6 +5099,13 @@ unsigned Compiler::gtSetEvalOrder(GenTree* tree) | |||
costSz = 7; // jump to cold section | |||
break; | |||
|
|||
#ifdef TARGET_ARM64 | |||
case GT_CHK_DIV_BY_ZERO: | |||
costEx = 4; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what the costs should be. I just copied from GT_BOUNDS_CHECK from above.
Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it. |
This is currently a draft, but the hope is to add a specific node that represents the divide-by-zero check.