yetingk wrote:

> The patch basically changes the ShadowCallStack back-end to emit an 
> sspush/sspopchk instead of the usual SCS push/pop, which seems like a 
> reasonable approach to me. However, it would be helpful to mention the 
> dependency on `-fsanitize=shadow-call-stack` in the commit message, and you 
> should also update [the 
> documentation](https://clang.llvm.org/docs/ShadowCallStack.html).

Thank you for the reminder. I have updated the document by 
[e32fc6c](https://github.com/llvm/llvm-project/pull/68075/commits/e32fc6cfb624d2ee90dc5b1da2c2e505f9920ac0).

https://github.com/llvm/llvm-project/pull/68075
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to