Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[cdac] Basic ISOSDacInterface::GetMethodDescData #106413
[cdac] Basic ISOSDacInterface::GetMethodDescData #106413
Changes from 64 commits
601938d
5f3599c
2cf8fd4
7ef9016
9cf7a7a
2d1e34e
2d4252a
c83ad25
c6dae6f
43e1748
72369e3
93f0a3d
58aa614
4c211d4
ce772d9
891f07f
5717664
1eeb513
2a95bda
ebf76a3
7defaef
9181f3f
86954df
0d0b592
63e0334
c177e4d
3956b06
0b91b85
d3a8bb0
5fcdc60
efad4ea
6febf72
dd86095
5b33737
de58ab5
0b6f6e4
502cd31
312f2ce
c813a78
f225adb
2cb8e86
f4852d2
da7065e
818c21c
1f6a909
04f41f6
abf9df0
71dee34
2fbc89e
2e3b938
1776abe
099b5fa
50046b8
e2d637b
7724fa8
a53b3ae
124ee6a
fb1676b
f1d757e
f1d7be8
ebb4a67
aac3faf
ef51f0e
92764a2
4e1a620
037c349
25d8343
ac0165a
8f5e98b
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we'll want to have a LoaderAllocator contract. There is a bunch of ... stuff... on there, and I feel like we'll likely need something there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For this particular API (checking if the module is collectible), it seems like it should cut out the
LoaderAllocator
concept? Since once you're at theModule
(instead of a MethodDesc), the module's IsCollectible should match the module's loader allocator's IsCollectible.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're going to need the ability to walk the list of IL versions, and Native code versions, and query various details about these things. This doesn't feel like the right api. Could you get it onto the CodeVersions contract?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This just extracts the address from the MethodDescCodeData. All the logic for working with the versioning state is in CodeVersions.
It's not that different from
RTS gives you a pointer and Loader turns it into a handle that you can use for queries. Similarly RTS will give a pointer to the versioning state and CodeVersions will consume it in some manner.
Currently the versioning state is consumed entirely internally in the CodeVersions contract, but in the future it could be exposed via a handle or through some data API (see
GetSpecificNativeCodeVersion
andIsActiveNativeCodeVersion
implementations)