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/
https://reviews
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",
+ llvm::utostr(Cod
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
clang/test/Driver/
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 -fsta
