Re: [PATCH v3] qemu/compiler: Absorb 'clang-tsa.h'

[Richard Henderson]

On 1/17/25 09:02, Philippe Mathieu-Daudé wrote:
> We already have "qemu/compiler.h" for compiler-specific arrangements,
> automatically included by "qemu/osdep.h" for each source file. No
> need to explicitly include a header for a Clang particularity.
>
> Suggested-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
> Reviewed-by: Kevin Wolf <kwolf@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   bsd-user/qemu.h                  |   1 -
>   include/block/block_int-common.h |   1 -
>   include/block/graph-lock.h       |   2 -
>   include/exec/page-protection.h   |   2 -
>   include/qemu/clang-tsa.h         | 114 -------------------------------
>   include/qemu/compiler.h          |  87 +++++++++++++++++++++++
>   include/qemu/thread.h            |   1 -
>   block/create.c                   |   1 -
>   tests/unit/test-bdrv-drain.c     |   1 -
>   tests/unit/test-block-iothread.c |   1 -
>   util/qemu-thread-posix.c         |   1 -
>   11 files changed, 87 insertions(+), 125 deletions(-)
>   delete mode 100644 include/qemu/clang-tsa.h

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~