This is an archive of the discontinued LLVM Phabricator instance.

[clang] allow -fstack-clash-protection on FreeBSD
ClosedPublic

Authored by emaste on Aug 23 2021, 11:54 AM.

Details

Summary

-fstack-clash-protection was added in Clang commit e67cbac81211 but was enabled only on Linux. Allow it on FreeBSD as well, as it works fine.

Diff Detail

Event Timeline

emaste created this revision.Aug 23 2021, 11:54 AM
emaste requested review of this revision.Aug 23 2021, 11:54 AM

We make this change in FreeBSD's in-tree Clang about 9 months ago (https://reviews.freebsd.org/D27366)

You know your platform better than I do, and you already ran the patch without any issue, so LGTM

This revision is now accepted and ready to land.Aug 23 2021, 1:06 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 24 2021, 6:03 PM