[PATCH] D122673: Add kcfi_unchecked attribute

Sami Tolvanen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 29 11:21:04 PDT 2022


samitolvanen added a comment.

Note that this was split from D119296 <https://reviews.llvm.org/D119296>.

In the previous discussion, @joaomoreira  pointed out that this is very similar to `nocf_check` and proposed reusing that attribute. In an offline discussion, @pcc  was concerned that an attribute may not be the right approach here and suggested  a `__builtin_kcfi_unchecked(function(args))` built-in function to avoid changing the type system.

I would appreciate hearing your thoughts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122673



More information about the cfe-commits mailing list