-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[wasm] Backport build improvements, and fixes from #61581 #62757
Conversation
VS does its own tracking of inputs/outputs too, and needs to be told about `@(NativeFileReference)` items, so a build can get triggered with F5 when a native file changes. Based on https://github.com/dotnet/project-system/blob/main/docs/up-to-date-check.md?rgh-link-date=2021-10-26T13%3A23%3A47Z#up-to-date-check Fixes dotnet#60862
…attributes Inspired by https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1364890?src=WorkItemMention&src-action=artifact_link Works around a NRE in CustomAttributeData.AttributeType because it's ConstructorInfo is null. The mono side issue: mono/mono#15340
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsBackport of #61581 to release/6.0 Fixes #60862 Customer ImpactImproves incremental builds in VS, when using native references (#60862), pinvoke signature generation, and surfaces error details in AOT compilation. TestingUnit tests, and manual testing. RiskLow risk
|
Approved over email. |
Closing and reopening to trigger CI again. |
Backport of #61581 to release/6.0
Fixes #60862
Partially fixes #60800
Customer Impact
Improves incremental builds in VS when using native references (#60862), pinvoke signature generation, and surfaces error details in AOT compilation. The pinvoke signature generation has been expanded to use only valid characters in the function name, which improves on the earlier approach of replacing some known invalid characters.
Testing
Unit tests, and manual testing.
Risk
Low risk.