Skip to content
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

Symbols API #601

Open
wants to merge 62 commits into
base: develop
Choose a base branch
from
Open

Symbols API #601

wants to merge 62 commits into from

Conversation

wasabii
Copy link
Contributor

@wasabii wasabii commented Oct 2, 2024

Replace all usages of System.Reflection and IKVM.Reflection in the Runtime classes with IKVM.CoreLib.Symbols. This removes nearly all of the #IFDEFs. We should be free to start breaking up the Runtime classes into IKVM.CoreLib.Symbols.

Verified

This commit was signed with the committer’s verified signature.
Ana06 Ana María Martínez Gómez

Verified

This commit was signed with the committer’s verified signature.
Ana06 Ana María Martínez Gómez

Verified

This commit was signed with the committer’s verified signature.
Ana06 Ana María Martínez Gómez
Turn on -skiperror. Need to figure out what happened here.
Fix libpath for discovery of corelibname.
Remove a number of ifdefs.
Add a DiagnosticOptions to allow NoWarn/WarnAsError. These are quite how I want them, but they serve for now.
Allow overriding levels using those options.
Trap fatal exception.
Fix resolver.
@wasabii wasabii marked this pull request as ready for review October 2, 2024 23:43
…od. Instead, we now support GetMethod properly on the delegate type itself using _incompleteMethods.
wasabii added 30 commits October 6, 2024 16:08
…te causes recursion.
…e emitted as a different underlying type depending on the target. DynamicMethod requires RuntimeTypeHandles, and so cannot accept builders. But, building types on the fly to be used by DynamicMethods is a thing. Think this just worked out on accident previously. But now it's explicit.
,
.
Remove folder.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant