Skip to content

Conversation

@jkotas
Copy link
Member

@jkotas jkotas commented Oct 12, 2025

Contributes to #119706

@jkotas jkotas requested review from bartonjs, Copilot and vcsjones and removed request for bartonjs and Copilot October 12, 2025 06:16
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-security, @bartonjs, @vcsjones
See info in area-owners.md if you want to be subscribed.

Copy link
Member

@vcsjones vcsjones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The crypto changes look good.

Do we know when it is likely we are going to get a CI pipeline with clang 21? This seems likely to introduce again unless CI blocks it.

@jkotas
Copy link
Member Author

jkotas commented Oct 12, 2025

Do we know when it is likely we are going to get a CI pipeline with clang 21?

Opened #120645 .

@jkotas jkotas merged commit 8a3b850 into dotnet:main Oct 12, 2025
145 of 151 checks passed
@jkotas jkotas deleted the clang21 branch October 12, 2025 16:15
@tmds
Copy link
Member

tmds commented Oct 14, 2025

@jkotas thanks for addressing this! Can these changes be backported to .NET 10, and perhaps also earlier .NET versions?

@jkotas
Copy link
Member Author

jkotas commented Oct 14, 2025

We do not backport fixes for C/C++ compiler warnings. We disable C/C++ warnings in release branches instead. I see that we forgot to do that for .NET 10. #120693 is fixing this.

@tmds
Copy link
Member

tmds commented Oct 21, 2025

@jkotas The changes for enum CorDebugUserState -1 value may have to be backported. I set PRERELEASE 0 and they still show up as an error (while the other ones become regular warnings).

jkotas added a commit to jkotas/runtime that referenced this pull request Oct 21, 2025
@jkotas
Copy link
Member Author

jkotas commented Oct 21, 2025

The changes for enum CorDebugUserState -1 value may have to be backported.

We can propose backport of all fixes required to make clang 21 work in your setup once we understand the full set. cc @AaronRobinsonMSFT .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants