Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 25, 2021

Backport of #58070 to release/6.0-rc1

/cc @davidwrighton

Customer Impact

Locals in ready to run compiled binaries targeting X86 will not be handled correctly on an X86 debugger. This was discovered by the VS debugger team, and @gregg-miskelly filed #57951. The impact noticed by the VS debugger team was that debugging Wait relationships between Tasks could not be understood on a Windows X86 build of the runtime.

Testing

Manual inspection of generated local information in R2R images was verified to be correct.

Risk

Low, Debugging information for locals in X86 R2R images may continue to be incorrect.

Regression

Yes. This is a bug that was not present in Crossgen 1.

@lewing
Copy link
Member

lewing commented Aug 25, 2021

runtime (CoreCLR Pri0 Runtime Tests Run windows x86 checked) hit #11063

Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

Approved. Please get a code review and we can consider for RC1.

@jeffschwMSFT jeffschwMSFT added the Servicing-consider Issue for next servicing release review label Aug 25, 2021
@jeffschwMSFT jeffschwMSFT added this to the 6.0.0 milestone Aug 25, 2021
@jeffschwMSFT jeffschwMSFT added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Aug 25, 2021
@jeffschwMSFT
Copy link
Member

@Anipik can you help with merging this change when appropriate?

@Anipik Anipik merged commit 0669a53 into release/6.0-rc1 Aug 26, 2021
@akoeplinger akoeplinger deleted the backport/pr-58070-to-release/6.0-rc1 branch August 28, 2021 22:03
@ghost ghost locked as resolved and limited conversation to collaborators Sep 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-crossgen2-coreclr Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants