Skip to content
This repository was archived by the owner on Nov 1, 2020. It is now read-only.

Merge changes from TFS#3242

Merged
MichalStrehovsky merged 1 commit intodotnet:nmirrorfrom
dotnet-bot:from-tfs
Apr 7, 2017
Merged

Merge changes from TFS#3242
MichalStrehovsky merged 1 commit intodotnet:nmirrorfrom
dotnet-bot:from-tfs

Conversation

@dotnet-bot
Copy link
Collaborator

No description provided.

- When referencing a method outside of the current module, the ReflectionInvokeMapNode should reference via a method name and signature in the native layout, not a metadata token
- When referencing a field outside of the current module, the ReflectionFieldMapNode should reference via method name not token
- When emitting a reference to a indirection cell in the ExternalReferencesTableNode use the calculate delta instead of throwing away the calculation
- When producing GCStatic descs for canonical types for the purpose of the template type loader, they should not be grouped into a single GC Static Desc region, but instead should be held in separate standalone regions
- Update the PrecomputedMetadataManager to take advantage of the new ReflectionInvokeMapNode functionality
- Disable new functionality for field mapping, as that requires cross module static import which isn't yet completely implemented
- Disable the use of the dynamic invoke tables. The dependency analysis interactions are still not quite right in this scenario
- When producing Thread Static GC descs, use a section relative reloc to the TypeThreadStaticsSymbol, not the start of the tls section
- Remove all plumbing around the start of tls section and the ReadyToRun header
  - The need for the data was incorrect, and the data is not useful

[tfs-changeset: 1653533]
@MichalStrehovsky MichalStrehovsky merged commit d7390b3 into dotnet:nmirror Apr 7, 2017
@dotnet-bot dotnet-bot deleted the from-tfs branch April 7, 2017 02:28
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.

4 participants