Skip to content

Conversation

@LakshanF
Copy link
Contributor

The PInvoke analysis changes cause the trimmer warnings to move to the call sites. Once the change gets merged to the runtime repo, there will be new warnings from building the OOB assemblies. This PR has changes that will fix the warnings and adding as a draft PR to expedite the trimmer integration when that happen

@ghost ghost added the linkable-framework Issues associated with delivering a linker friendly framework label Jun 17, 2021
@ghost
Copy link

ghost commented Jun 17, 2021

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost
Copy link

ghost commented Jun 17, 2021

Tagging subscribers to 'linkable-framework': @eerhardt, @vitek-karas, @LakshanF, @sbomer, @joperezr
See info in area-owners.md if you want to be subscribed.

Issue Details

The PInvoke analysis changes cause the trimmer warnings to move to the call sites. Once the change gets merged to the runtime repo, there will be new warnings from building the OOB assemblies. This PR has changes that will fix the warnings and adding as a draft PR to expedite the trimmer integration when that happen

Author: LakshanF
Assignees: -
Labels:

linkable-framework

Milestone: -

@akoeplinger
Copy link
Member

I cherry-picked your commit to #54218 where the linker update is happening :)

@ghost ghost closed this Jul 18, 2021
@ghost
Copy link

ghost commented Jul 18, 2021

Draft Pull Request was automatically closed for inactivity. Please let us know if you'd like to reopen it.

@ghost ghost locked as resolved and limited conversation to collaborators Aug 17, 2021
@LakshanF LakshanF deleted the PInvokeWrnFix branch May 10, 2022 13:21
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

linkable-framework Issues associated with delivering a linker friendly framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants