[PATCH] D119296: KCFI sanitizer
Joao Moreira via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 28 23:57:29 PDT 2022
joaomoreira added a comment.
> I looked at your code quickly and I wonder if using operand bundles would be better than adding an attribute. Thoughts?
Perhaps @pcc can provide a better feedback on the hassle (or joys) behind using operand bundles. My insights on the attribute is that it is very simple and cheap to set, get, copy... The bad side is that now you have an additional attribute on a frequently used class.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119296/new/
https://reviews.llvm.org/D119296
More information about the llvm-commits
mailing list