eugenis added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/Linux.cpp:767
+ if (IsAArch64) {
+ Res |= SanitizerKind::MemtagGlobals;
+ }
----------------
Hmm why are all the other memtag* not here?
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:7892
+static bool isSanitizerThatUsesGlobals(StringRef Sanitizer) {
+ return Sanitizer == "address" || Sanitizer == "hwaddress" ||
----------------
Maybe "applies to globals"? On the other hand, MSan "applies" to globals but
does not need this logic.
isSanitizerAttributeAllowedOnGlobals?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127163/new/
https://reviews.llvm.org/D127163
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
