Skip to content

[release/6.0] Big-endian fixes: PDB creation by ilasm - #59452

Closed
github-actions[bot] wants to merge 1 commit into
release/6.0from
backport/pr-59422-to-release/6.0
Closed

[release/6.0] Big-endian fixes: PDB creation by ilasm#59452
github-actions[bot] wants to merge 1 commit into
release/6.0from
backport/pr-59422-to-release/6.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Sep 22, 2021

Copy link
Copy Markdown
Contributor

Backport of #59422 to release/6.0

/cc @BruceForstall@uweigand

Customer Impact

Fixes s390x platform source-build using .NET 6 tools. No impact to supported .NET 6 platforms.

Testing

Regular PR testing + ilasm/ildasm round-trip pri-1 test run.

Risk

Low.

* Fix byte order of debug directory fields
* Fix byte order of PDB stream data (PdbHeap::SetData)
@ghost

Copy link
Copy Markdown

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

Issue Details

Backport of #59422 to release/6.0

/cc @BruceForstall@uweigand

Customer Impact

Testing

Risk

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

area-ILTools-coreclr

Milestone:-

@BruceForstall

Copy link
Copy Markdown
Contributor

@dotnet/jit-contrib PTAL

@jeffschwMSFTjeffschwMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved. We should take this for consideration in .NET 6.

@marek-safarmarek-safar added the Servicing-consider Issue for next servicing release review label Sep 27, 2021
@jeffschwMSFT

Copy link
Copy Markdown
Member

Closing as this was merged into RC2 via #59484

@BruceForstall

Copy link
Copy Markdown
Contributor

@jeffschwMSFT Do rc2 changes automatically flow to the release branch?

@jeffschwMSFT

Copy link
Copy Markdown
Member

Do rc2 changes automatically flow to the release branch?

Yes

@akoeplinger
akoeplinger deleted the backport/pr-59422-to-release/6.0 branch October 7, 2021 18:39
@ghostghost locked as resolved and limited conversation to collaborators Nov 6, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-ILTools-coreclrServicing-considerIssue for next servicing release review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@BruceForstall@jeffschwMSFT@AndyAyersMS@marek-safar@JulieLeeMSFT@uweigand