[PATCH] D64169: ARM MTE stack sanitizer.

[Evgenii Stepanov via Phabricator via cfe-commits]

eugenis added inline comments.


================
Comment at: clang/lib/CodeGen/SanitizerMetadata.cpp:28
+  if (!CGM.getLangOpts().Sanitize.hasOneOf(
+          SanitizerKind::Address | SanitizerKind::KernelAddress |
+          SanitizerKind::HWAddress | SanitizerKind::KernelHWAddress |
----------------
vitalybuka wrote:
> maybe shared constant or function for :
> SanitizerKind::Address | SanitizerKind::KernelAddress |
>           SanitizerKind::HWAddress | SanitizerKind::KernelHWAddress |
>           SanitizerKind::MemTag
added a helper function


================
Comment at: llvm/docs/BitCodeFormat.rst:1060
 * code 58: ``shadowcallstack``
+* code 62: ``sanitize_memtag``
 
----------------
vitalybuka wrote:
> why it's 62 and not e.g. 59?
Because people forget to update this file.
And it's 64 already.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64169