Skip to content

Conversation

@AndyAyersMS
Copy link
Member

OSR methods must do special setups for register parameters, loading them
from the original method's frame. Update the computation we do here so it
works for TYP_STRUCT register params.

Closes #59875.

OSR methods must do special setups for register parameters, loading them
from the original method's frame. Update the computation we do here so it
works for TYP_STRUCT register params.

Closes dotnet#59875.
@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Sep 30, 2021
@ghost
Copy link

ghost commented Sep 30, 2021

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

Issue Details

OSR methods must do special setups for register parameters, loading them
from the original method's frame. Update the computation we do here so it
works for TYP_STRUCT register params.

Closes #59875.

Author: AndyAyersMS
Assignees: -
Labels:

area-CodeGen-coreclr

Milestone: -

@AndyAyersMS
Copy link
Member Author

cc @dotnet/jit-contrib

Copy link
Contributor

@kunalspathak kunalspathak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AndyAyersMS
Copy link
Member Author

/azp run runtime-jit-experimental

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@AndyAyersMS
Copy link
Member Author

OSR stress R2R tests are all failing with various internal errors.

Same failures are there in last weekend's runs so it must be some other problem

@AndyAyersMS
Copy link
Member Author

/azp run runtime-jit-experimental

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@AndyAyersMS
Copy link
Member Author

The crossgen2 test failure is the same issue, it's just cropping up in the dotnet used to run crossgen2, which is currently 6.0.100-rc.1.21430.12.

That is, we're running crossgen2 under OSR stress, using an older version of the jit, that still has this bug. Running the test where the crossgen2 is an updated corerun passes.

So this will fix itself eventually, once we update that set of bits.

@AndyAyersMS AndyAyersMS merged commit 646e195 into dotnet:main Oct 1, 2021
@AndyAyersMS AndyAyersMS deleted the OSREmitTypeFix branch October 1, 2021 01:00
@ghost ghost locked as resolved and limited conversation to collaborators Nov 3, 2021
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.

Cache compaction will never start

3 participants