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

Conversation

jkotas
Copy link
Member

@jkotas jkotas commented Feb 8, 2017

Fixes #6427

@jkotas
Copy link
Member Author

jkotas commented Feb 8, 2017

Copy link
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

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

Thanks, Jan and Brian.

@brianrob
Copy link
Member

brianrob commented Feb 8, 2017

Ah, you beat me to it. That's pretty much the exact same fix I'm building now. LGTM. Thanks!

@brianrob brianrob self-requested a review February 8, 2017 20:43

#ifdef FEATURE_READYTORUN
if (pLoadedLayout->HasReadyToRunHeader())
if (pModule->IsReadyToRun())
Copy link
Member

Choose a reason for hiding this comment

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

LGTM

@jkotas jkotas merged commit d330eaf into dotnet:master Feb 8, 2017
jkotas added a commit to jkotas/coreclr that referenced this pull request Feb 8, 2017
* Filter RefEmit modules in SendEventsForNgenMethods

Fixes #6427
@jkotas jkotas deleted the issue-6427 branch February 8, 2017 23:19
jkotas added a commit that referenced this pull request Feb 9, 2017
* Filter RefEmit modules in SendEventsForNgenMethods

Fixes #6427
jorive pushed a commit to guhuro/coreclr that referenced this pull request May 4, 2017
* Filter RefEmit modules in SendEventsForNgenMethods

Fixes #6427
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
* Filter RefEmit modules in SendEventsForNgenMethods

Fixes dotnet/coreclr#6427


Commit migrated from dotnet/coreclr@d330eaf
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.

5 participants