[PATCH] D148665: Change -fsanitize=function to place two words before the function entry

Peter Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 3 11:11:07 PDT 2023


peter.smith added a comment.

My apologies for not responding. If I've got this right there are 4 related patches:
D148573 <https://reviews.llvm.org/D148573> (approved)
D148785 <https://reviews.llvm.org/D148785> Use type hashes rather than RTTI 
D148827 <https://reviews.llvm.org/D148827> support C
D148665 <https://reviews.llvm.org/D148665> (this one)

My initial impressions is that this makes -fsanitize=function look more like -fsanitize=kcfi which makes it accessible from C and available to more targets. Is there anything that we lose in the process of making these changes? For example I would expect RTTI to have more information available than a type hash, although this might not make any functional difference.

I'll try and look over the next few days and leave some comments, apologies a bit busy at work at the moment so I can't promise anything speedy.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148665



More information about the cfe-commits mailing list