Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Mirror changes from dotnet/coreclr#29012

Closed
dotnet-bot wants to merge 1 commit into
masterfrom
mirror-merge-9612217
Closed

Mirror changes from dotnet/coreclr#29012
dotnet-bot wants to merge 1 commit into
masterfrom
mirror-merge-9612217

Conversation

@dotnet-bot
Copy link
Copy Markdown

This PR contains mirrored changes from dotnet/coreclr

Please REBASE this PR when merging

* Add CreateFromPinnedArray to MemoryMarshal

* Address PR feedback: Add a warning remark to CreateFromPinnedArray

Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>
@ahsonkhan
Copy link
Copy Markdown

Cherry-picked this change into #28992

@stephentoub
Copy link
Copy Markdown
Member

@ahsonkhan, @Anipik, some other changes are blocked behind this. How can we get the mirror flowing again?

@Anipik
Copy link
Copy Markdown

Anipik commented Apr 11, 2018

@stephentoub are these missing changes necessary for this PR to get merged ? if not I will wait for this pr to get merged, otherwise I will close this one and reopen the PR for missing commits first

@stephentoub
Copy link
Copy Markdown
Member

are these missing changes necessary for this PR to get merged ?

No, but I don't think this PR is going to be merged. I believe @ahsonkhan cherry-picked the changes from this PR into #28992, which hasn't been merged yet. Maybe we need to wait for that one to merge and then close this one? I don't fully understand how the mirror works in that regard.

@ahsonkhan
Copy link
Copy Markdown

Maybe we need to wait for that one to merge and then close this one?

Yes.

@Anipik
Copy link
Copy Markdown

Anipik commented Apr 11, 2018

@ahsonkhan why do we need to wait to close this one ?

@stephentoub stephentoub added this to the 2.1.0 milestone Apr 11, 2018
@ahsonkhan
Copy link
Copy Markdown

why do we need to wait to close this one ?

If you close this before the other PR is merged, won't another one get re-opened with the exact same changes (and block on CI)?

@Anipik
Copy link
Copy Markdown

Anipik commented Apr 11, 2018

@ahsonkhan it depends upon the history actually. I may be able to stop the commit from happening. I will close this one and open the mirror with the missing commits

@Anipik Anipik closed this Apr 11, 2018
@Anipik Anipik deleted the mirror-merge-9612217 branch April 11, 2018 19:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants