[PATCH] D124211: Add __builtin_call_kcfi_unchecked
Nick Desaulniers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 28 11:47:04 PDT 2022
nickdesaulniers added a comment.
In D124211#3480709 <https://reviews.llvm.org/D124211#3480709>, @samitolvanen wrote:
> In D124211#3480652 <https://reviews.llvm.org/D124211#3480652>, @nickdesaulniers wrote:
>> Can you link to the lore thread on discussions around the builtin?
> There's no LKML discussion about the built-in, but there were earlier proposals of using an attribute to accomplish the same thing. @pcc suggested using a built-in instead, and the reviewers in D122673 <https://reviews.llvm.org/D122673> agreed that it's a more appropriate solution. In the kernel, the indirect `static_call` trampoline calls that are patched into direct calls at boot are the primary reason we need a finer-grained method of disabling CFI checks.
Perhaps it's time to start a discussion on LKML then, before landing this?
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits