Skip to content
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

Enable control flow guard for mono on windows #101293

Merged
merged 4 commits into from
Apr 22, 2024

Conversation

steveisok
Copy link
Member

Resolves binskim warnings for the runtime and aot compilers on windows.

Resolves binskim warnings for the runtime and aot compilers on windows.
src/mono/CMakeLists.txt Outdated Show resolved Hide resolved
Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Build breaks:

2024-04-19T19:57:55.6709922Z   MASM : fatal error A1000:cannot open file : $<0:/GUARD:cf
2024-04-19T19:57:55.6712646Z   [24/359] Building C object mono\eglib\CMakeFiles\eglib_objects.dir\D_\a\_work\1\s\src\native\minipal\utf8.c.obj
2024-04-19T19:57:55.8142203Z   cl : Command line warning D9024 : unrecognized source file type '$<1:/GUARD:cf', object file assumed
2024-04-19T19:57:55.8145990Z   cl : Command line warning D9027 : source file '$<1:/GUARD:cf' ignored

Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
@steveisok
Copy link
Member Author

Thanks, guys. Figures I missed a > 😂

@steveisok
Copy link
Member Author

/ba-g The failure is similar to #98578 except it's not appearing in the console log and so BA can't capture and relate the failure.

@steveisok steveisok merged commit 9ff6bb8 into dotnet:main Apr 22, 2024
65 of 70 checks passed
@steveisok steveisok deleted the add-cfg-mono branch April 22, 2024 13:56
matouskozak pushed a commit to matouskozak/runtime that referenced this pull request Apr 30, 2024
Resolves binskim warnings for the runtime and aot compilers on windows.

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
@github-actions github-actions bot locked and limited conversation to collaborators May 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants