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

[JIT] Support x % {pow2_cns} transformation for ulong types #79676

Closed
wants to merge 4 commits into from

Conversation

TIHan
Copy link
Contributor

@TIHan TIHan commented Dec 14, 2022

Description

This PR allows fgMorphUModToAndSub to get hit if the type is TYP_LONG.

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Dec 14, 2022
@ghost ghost assigned TIHan Dec 14, 2022
@ghost
Copy link

ghost commented Dec 14, 2022

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch, @kunalspathak
See info in area-owners.md if you want to be subscribed.

Issue Details

Description

This PR moves fgMorphUModToAndSub to have a little earlier so GT_UMOD of type TYP_LONG can be taken advantage of.

Author: TIHan
Assignees: -
Labels:

area-CodeGen-coreclr

Milestone: -

@TIHan TIHan changed the title [JIT] Moving 'fgMorphUModToAndSub' to happen a little earlier [JIT] Support x % {pow2_cns} transformation for ulong types Dec 16, 2022
@ghost ghost closed this Jan 16, 2023
@ghost
Copy link

ghost commented Jan 16, 2023

Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it.

@TIHan
Copy link
Contributor Author

TIHan commented Jan 25, 2023

This isn't a critical task, so I'm closing this for now - made an issue: #81189

@TIHan TIHan closed this Jan 25, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Feb 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant