[PATCH] D96007: [AArch64] Enable stack clash protection for AArch64 linux in clang

2023-02-21 Thread Ed Maste via Phabricator via cfe-commits
emaste added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:3090 if (!EffectiveTriple.isOSLinux()) return; Why is this limited to `isOSLinux()` only? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96007/new/

[PATCH] D96007: [AArch64] Enable stack clash protection for AArch64 linux in clang

2022-11-03 Thread Sam Tebbs via Phabricator via cfe-commits
samtebbs added inline comments. Herald added a subscriber: MaskRay. Herald added a project: All. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:1669-1672 +if (CodeGenOpts.StackProbeSize != 4096) + B.addAttribute("stack-probe-size", +

[PATCH] D96007: [AArch64] Enable stack clash protection for AArch64 linux in clang

2021-02-05 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard updated this revision to Diff 321689. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96007/new/ https://reviews.llvm.org/D96007 Files: clang/lib/CodeGen/CodeGenModule.cpp clang/lib/Driver/ToolChains/Clang.cpp clang/test/CodeGen/stack-clash-protection.c

[PATCH] D96007: [AArch64] Enable stack clash protection for AArch64 linux in clang

2021-02-04 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard created this revision. ostannard added reviewers: serge-sans-paille, jnspaulsson, bzEq, tnfchris. Herald added subscribers: danielkiss, kristof.beyls. ostannard requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This allows the