[PATCH] D143769: [lld] [MTE] Add DT_AARCH64_MEMTAG_* dynamic entries, and small cleanup

Mon Feb 13 10:01:55 PST 2023

hctim marked 2 inline comments as done.
hctim added a comment.

In D143769#4120139 <https://reviews.llvm.org/D143769#4120139>, @MaskRay wrote:

> Considering adding some documentation to `lld/docs/ld.lld.1`. It can contain more information than `lld/ELF/Options.td`

I think it might be better to leave it off the manpage for now, given:

1. MTE binaries should generally be linked via. an invocation of clang, much like other sanitizers.
2. Right now, this is only an android-specific feature, so normal developers shouldn't be touching these flags, as it should all be done via. the build system. Not even NDK developers should be using them for now. Maybe once we genericize the API to be `--aarch64` rather than `--android`, we can consider.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143769