Skip to content

Enable AOT and platform compatibility analyzers. #340

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Enable AOT and platform compatibility analyzers. #340

wants to merge 2 commits into from

Conversation

teo-tsirpanis
Copy link
Contributor

  • The SymUnmanaged***Factory APIs are marked as supported on Windows only.
  • The AOT compatibility analyzer was enabled. There was a warning in the COM registry activation path, which was rewritten to use ComWrappers.
    • It is not covered by tests at the moment because of known issues, but after tweaking the test I was able to validate it on my machine (activation succeeded but it subsequently failed because it found a very old diasymreader.dll).

@teo-tsirpanis
Mark unmanaged reader/writer APIs as supported only on Windows.
3110407
@teo-tsirpanis
Enable AOT compatibility analyzer.
ff87c9a 
This required to add a trim-friendly version of the COM registry activation path.
@ViktorHofer ViktorHofer requested a review from tmat April 23, 2025 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tmat tmat Awaiting requested review from tmat

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@teo-tsirpanis