[PATCH] D43108: Support for the mno-stack-arg-probe flag

[Amara Emerson via Phabricator via cfe-commits]

aemerson added a comment.

Can we clarify the meaning of this option a bit. The doc you've added here is saying that `-mno-stack-arg-probe` disables stack probes. Then what does `-mstack-arg-probe` mean specifically? Does it mean that only stack probes for ABI required reasons are enabled, or probes are done even in cases where the ABI doesn't require them? Either way, the doc needs to be clearer on the exact purpose.

I'm currently working on enabling stack probes for reasons other than ABI, and so if the answer is that this option is only concerned with ABI, we will need another option like `-fstack-check`.


Repository:
  rC Clang

https://reviews.llvm.org/D43108