Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Sep 4, 2025

Backport of #119334 to release/10.0

/cc @AndyAyersMS

Customer Impact

  • Customer reported
  • Found internally

coredistools was flagged as out of compliance (was not built /guard:cf).

It is not part of the shipping product but is used in testing.

Regression

  • Yes
  • No

Testing

Verfied the 1.6 binary has /guard:cf enabled via dumpbin.

Risk

Low

AndyAyersMS and others added 2 commits September 4, 2025 17:45
Windows builds now enable /guard:cf. Also pick up a risc-v enhancement.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@AndyAyersMS
Copy link
Member

@dotnet/jit-contrib PTAL
cc @JulieLeeMSFT @jeffschwMSFT

Needed for compliance

Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

approved. we can merge when ready

@jeffschwMSFT jeffschwMSFT added Servicing-approved Approved for servicing release area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI labels Sep 4, 2025
@jeffschwMSFT jeffschwMSFT added this to the 10.0.0 milestone Sep 4, 2025
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

@AndyAyersMS
Copy link
Member

Build analysis seems to know about the failures but is still red... not sure why.

@jeffschwMSFT jeffschwMSFT merged commit 8c45b3d into release/10.0 Sep 5, 2025
158 of 163 checks passed
@akoeplinger akoeplinger deleted the backport/pr-119334-to-release/10.0 branch September 8, 2025 13:41
@github-actions github-actions bot locked and limited conversation to collaborators Oct 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants