Skip to content

Spill low ref count vector that has lot of save/restore #66161

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.

Sign up for GitHub

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

Jump to bottom
Closed
kunalspathak wants to merge 5 commits into from
Closed

Spill low ref count vector that has lot of save/restore #66161

kunalspathak wants to merge 5 commits into from

Conversation

@kunalspathak
Copy link
Contributor

@kunalspathak kunalspathak commented Mar 3, 2022 •
edited
Loading

This is a draft to avoid do save/restore for upper vector if the actual usage is minimal.

Reference: #59315

@ghost ghost assigned kunalspathak Mar 3, 2022
@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Mar 3, 2022
@ghost
Copy link

ghost commented Mar 3, 2022

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

Issue Details

This is a draft to avoid do save/restore for upper vector if the actual usage is minimal.

Author: kunalspathak
Assignees: kunalspathak
Labels:

area-CodeGen-coreclr
Milestone: -

@kunalspathak kunalspathak mentioned this pull request Mar 3, 2022
Merged
@ghost ghost closed this Apr 2, 2022
@ghost
Copy link

ghost commented Apr 2, 2022

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

@kunalspathak
Copy link
Contributor Author

kunalspathak commented Apr 7, 2022

ping to myself.

@kunalspathak kunalspathak reopened this Apr 7, 2022
@ghost ghost added the no-recent-activity label Apr 21, 2022
@ghost
Copy link

ghost commented Apr 21, 2022

This pull request has been automatically marked no-recent-activity because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 more days. Any new comment (by anyone, not necessarily the author) will remove no-recent-activity.

@kunalspathak
Copy link
Contributor Author

kunalspathak commented Apr 22, 2022

ping

@ghost ghost removed the no-recent-activity label Apr 22, 2022
@ghost ghost closed this May 22, 2022
@ghost
Copy link

ghost commented May 22, 2022

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

@ghost ghost locked as resolved and limited conversation to collaborators Jun 22, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@kunalspathak kunalspathak

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kunalspathak