-
-
Notifications
You must be signed in to change notification settings - Fork 964
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
Clean up unsupported native AOT flags #2616
Conversation
* `llcOptimizationPreference` was renamed to `OptimizationPreference` and became supported in .NET 8 (or maybe .NET 7, it doesn't matter) * `IlcGenerateStackTraceData` is now `StackTraceSupport` and supported. (Also got renamed in .NET 8 or earlier.) I'm leaving the old names of properties for backcompat. We could also delete them. I don't know how much BDN cares about .NET versions that are out of support. IlcGenerateCompleteTypeMetadata is unsupported, we don't test it, it's a mode for troubleshooting trimming issues for people who ignore trimming warnings. I don't think BDN should be setting this. You'll only find bugs (dotnet/runtime#106439). So deleting that. Not deleting the API because it looks to be a public API.
@MichalStrehovsky, there is another warning due to line 167:
This patch should fix it: - : "<TrimMode>link</TrimMode><TrimmerDefaultAction>link</TrimmerDefaultAction>";
+ : @"<TrimMode>link</TrimMode>
+ <TrimmerDefaultAction Condition=""$([MSBuild]::VersionLessThan('$(NETCoreSdkVersion)', '7.0'))"">link</TrimmerDefaultAction>"; |
We'd also need to condition TrimMode - valid values of TrimMode after .NET 7 are These are however all orthogonal to what this PR is doing. |
Co-authored-by: Adeel Mujahid <3840695+am11@users.noreply.github.com>
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.
LGTM, thank you @MichalStrehovsky and @am11 !
llcOptimizationPreference
was renamed toOptimizationPreference
and became supported in .NET 8 (or maybe .NET 7, it doesn't matter)IlcGenerateStackTraceData
is nowStackTraceSupport
and supported. (Also got renamed in .NET 8 or maybe in 7 already.)I'm leaving the old names of properties for backcompat. We could also delete them. I don't know how much BDN cares about .NET versions that are out of support.
The last one,
IlcGenerateCompleteTypeMetadata
is unsupported, we don't test it, it's a mode for troubleshooting trimming issues for people who ignore trimming warnings. I don't think BDN should be setting this. You'll only find bugs (dotnet/runtime#106439). So deleting that. Not deleting the API because it looks to be a public API.