hctim added inline comments.
================ 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.">; ---------------- eugenis wrote: > MaskRay wrote: > > 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" mentioned it similar to armv8, as mentioned we're not 100% clear whether MTE is mandatory on armv9. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118948/new/ https://reviews.llvm.org/D118948 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits