Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[cdac] Implement ISOSDacInterface::GetModule and IXCLRDataModule::GetFileName - #109133

#109133
Merged
elinor-fung merged 15 commits into
dotnet:maindotnet/runtime:mainfrom
elinor-fung:cdac-xclrmodule-filenameelinor-fung/runtime:cdac-xclrmodule-filenameCopy head branch name to clipboard
Oct 24, 2024
Merged

[cdac] Implement ISOSDacInterface::GetModule and IXCLRDataModule::GetFileName#109133
elinor-fung merged 15 commits into
dotnet:maindotnet/runtime:mainfrom
elinor-fung:cdac-xclrmodule-filenameelinor-fung/runtime:cdac-xclrmodule-filenameCopy head branch name to clipboard

Conversation

@elinor-fung

Copy link
Copy Markdown
Member
  • Add ClrDataModule to cDAC as its implementation of IXCLRDataModule and IXCLRDataModule2
    • Delegates to legacy implementation for everything except for GetFileName
    • Implements ICustomQueryInterface to handle QI-ing the legacy DAC for IMetaDataImport
  • Implement ISOSDacInterface::GetModule
    • Gets a module from the legacy DAC and passes it to the cDAC ClrDataModule
  • Cache the module file name on Module
    • Update comments around how it is expected to be set
  • Add GetFileName to Loader contract
  • Implement IXClrDataModule::GetFileName

Contributes to #99302, #108553

cc @AaronRobinsonMSFT @davidwrighton

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

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

Comment thread src/native/managed/cdacreader/src/Legacy/SOSDacImpl.cs
Comment thread src/native/managed/cdacreader/tests/LoaderTests.cs Outdated
Comment thread src/native/managed/cdacreader/tests/LoaderTests.cs Outdated
@elinor-fung
elinor-fung merged commit d450d9c into dotnet:main Oct 24, 2024
@elinor-fung
elinor-fung deleted the cdac-xclrmodule-filename branch October 24, 2024 20:04
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 24, 2024
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.