glider added inline comments.
================
Comment at: clang/include/clang/Basic/Attr.td:2945
+ let Spellings = [Clang<"disable_sanitizer_instrumentation">];
+ let Subjects = SubjectList<[Function]>;
+ let Documentation = [DisableSanitizerInstrumentationDocs];
----------------
aaron.ballman wrote:
> Do we want this to also appertain to `GlobalVar` and `ObjCMethod` so it's got
> the same subjects as `no_sanitize`?
Good catch, thank you. When adding the new attribute to ASan, we probably want
it to mimic the behavior of `no_sanitize("address")`
================
Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:526-528
+ if (CurFuncDecl->hasAttr<DisableSanitizerInstrumentationAttr>())
+ return true;
+ return false;
----------------
aaron.ballman wrote:
>
Right :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108029/new/
https://reviews.llvm.org/D108029
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
