Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jan 24, 2024

Backport of #96817 to release/9.0-preview1

/cc @carlossanlop @filipnavara

Customer Impact

  • Customer reported
  • Found internally

We noticed when switching from the old to the new ObjWriter that the debug symbols were larger than before.

The important piece of this change is that it groups all the ObjWriter replacement changes into Preview 1, rather than splitting across multiple releases.

Regression

  • Yes
  • No

This regression was introduced by the ObjWriter change, which happened a few weeks ago.

Testing

Manual testing

Risk

Medium, because we have very limited unit testing of the symbol quality coming out of Native AOT. This is the primary reason to take this change in Preview 1 -- if we put these changes in ASAP, we can get feedback from users if they broke anything and have plenty of time to change it before 9.0 ships.

@ghost
Copy link

ghost commented Jan 24, 2024

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #96817 to release/9.0-preview1

/cc @carlossanlop @filipnavara

Customer Impact

  • Customer reported
  • Found internally

[Select one or both of the boxes. Describe how this issue impacts customers, citing the expected and actual behaviors and scope of the issue. If customer-reported, provide the issue number.]

Regression

  • Yes
  • No

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Testing

[How was the fix verified? How was the issue missed previously? What tests were added?]

Risk

[High/Medium/Low. Justify the indication by mentioning how risks were measured and addressed.]

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

  • If the change touches code that ships in a NuGet package, you have added the necessary package authoring and gotten it explicitly reviewed.

Author: github-actions[bot]
Assignees: -
Labels:

area-NativeAOT-coreclr

Milestone: -

@carlossanlop carlossanlop added the Servicing-approved Approved for servicing release label Jan 24, 2024
@carlossanlop carlossanlop added this to the 9.0.0 milestone Jan 24, 2024
@carlossanlop
Copy link
Contributor

Approved by Tactics via email.

@carlossanlop carlossanlop merged commit b4aa81d into release/9.0-preview1 Jan 24, 2024
@carlossanlop carlossanlop deleted the backport/pr-96817-to-release/9.0-preview1 branch January 24, 2024 20:04
@github-actions github-actions bot locked and limited conversation to collaborators Feb 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants