[PATCH] D119296: KCFI sanitizer

Joao Moreira via Phabricator via cfe-commits cfe-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 cfe-commits mailing list