[PATCH] D139254: Enhance stack protector

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added a comment.

Would there be any objections to adding a flag to restore the previous behavior? This is a fairly hefty size increase for us, and we don't think the additional checking is worth the size increase in our case. We also intend to look into the size increases more to determine how much is from guarding calls to e.g. `__cxa_throw`, where you will reuse the stack frame, vs. calls to e.g. `abort` or `_Unwind_Resume`, where the stack frame should never be reused and checking the stack cookie doesn't seem worthwhile (if my logic is sound, which I'm not 100% sure of).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139254/new/

https://reviews.llvm.org/D139254