[PATCH] D71082: Allow system header to provide their own implementation of some builtin

Manoj Gupta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 2 18:27:26 PDT 2020


manojgupta added a comment.

Yes, it'd be nice if all of the FORTIFY handling can be improved. For a simple call like memcpy of 8 bytes in the example, there is no reason to emit all these stack/range checks since they'd degrade memcpy performance.

I still think this change should be reverted if it can't handle Linux kernel's FORTIFY implementation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71082





More information about the cfe-commits mailing list