Skip to content
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.

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

Fix linker ref param warnings #70082

Merged
merged 12 commits into from
Jun 7, 2022
Merged

Conversation

jtschuster
Copy link
Member

The linker added support for tracking out and ref params after a method call. This caused some error codes to change and some new ones to appear. This PR changes / adds the necessary suppressions to accommodate the linker change

dotnet-maestro bot and others added 7 commits May 24, 2022 12:27
…523.1

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22270.1 -> To Version 7.0.100-1.22273.1
…524.1

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22270.1 -> To Version 7.0.100-1.22274.1
…525.1

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22270.1 -> To Version 7.0.100-1.22275.1
…530.1

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22270.1 -> To Version 7.0.100-1.22280.1
…531.1

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22270.1 -> To Version 7.0.100-1.22281.1
@ghost
Copy link

ghost commented Jun 1, 2022

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

Issue Details

The linker added support for tracking out and ref params after a method call. This caused some error codes to change and some new ones to appear. This PR changes / adds the necessary suppressions to accommodate the linker change

Author: jtschuster
Assignees: jtschuster
Labels:

area-Meta

Milestone: -

@jtschuster jtschuster requested a review from sbomer June 1, 2022 14:02
@jtschuster jtschuster marked this pull request as ready for review June 3, 2022 21:19
@sbomer sbomer merged commit 50d289c into dotnet:main Jun 7, 2022
maraf added a commit to maraf/runtime that referenced this pull request Jun 8, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jul 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants