[libcxx-commits] [libcxx] [ASan][libc++] Update string ASan annotations to Zero-overhead (PR #76165)

[via libcxx-commits]

https://github.com/EricWF requested changes to this pull request.

I think this works great. I don't love the way it looks but we can iterate on it.

The changes I would like before landing are:

(1) Rename to `_LIBCPP_IF_ASAN`.
(2) Put it in __config.
(3) Always use `{ ... }` after `_LIBCPP_IF_ASAN()`.

I think (3) is important because it's not clear exactly what `_LIBCPP_IF_ASAN()` expands to. We don't know if it actually captures the next statement as part of an `if` or not.

https://github.com/llvm/llvm-project/pull/76165