[PATCH] D118948: [MTE] Add -fsanitize=memtag* and friends.
Evgenii Stepanov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 8 10:58:13 PDT 2022
eugenis accepted this revision.
eugenis added a comment.
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:475
def err_stack_tagging_requires_hardware_feature : Error<
- "'-fsanitize=memtag' requires hardware support (+memtag)">;
+ "'-fsanitize=memtag-stack' requires hardware support (+memtag). For Armv8, "
+ "try compiling with -march=armv8a+memtag.">;
> hctim wrote:
> > eugenis wrote:
> > > Split out renaming of memtag to memtag-stack first? This will remove a lot of diff from this patch.
> > splitting into elf -> lld -> clang as per Ray's suggestion, should reduce the diff enough.
> The convention is to omit period for the last sentence.
Mention armv9a, too?
Something like "Try compiling with -march=armv9a, or -march=armv8a+memtag"
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits