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(CodeGenOpts.StackProbeSize));
+ }
----------------
Would it be worth always setting the `stack-probe-size` attribute so that it is
always correct and to avoid the risk of it going out-of-sync in case of changes
to the assumed size in the backend?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96007/new/
https://reviews.llvm.org/D96007
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
