Skip to content

Conversation

@vargaz
Copy link
Contributor

@vargaz vargaz commented Jul 18, 2022

… methods.

These methods store their data in a MONO_PATCH_INFO_GSHARED_METHOD_INFO which was not handled properly.

@ghost ghost assigned vargaz Jul 18, 2022
@vargaz vargaz requested review from ivanpovazan and removed request for SamMonoRT July 18, 2022 16:03
@vargaz
Copy link
Contributor Author

vargaz commented Jul 18, 2022

/azp run runtime-wasm

@vargaz
Copy link
Contributor Author

vargaz commented Jul 18, 2022

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@ivanpovazan ivanpovazan left a comment

Choose a reason for hiding this comment

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

Tested by intentionally skipping the generation of GSHAREDVT variant for a particular generic method and running it from an inflated caller.
lgtm

@lewing
Copy link
Member

lewing commented Jul 18, 2022

looks like #72386 is blocking the wasm build

… methods.

These methods store their data in a MONO_PATCH_INFO_GSHARED_METHOD_INFO which was not handled properly.
@vargaz vargaz merged commit f48bc15 into dotnet:main Jul 19, 2022
@vargaz vargaz deleted the aot-patches branch July 19, 2022 05:12
@ghost ghost locked as resolved and limited conversation to collaborators Aug 18, 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.

3 participants