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

[Philippe Mathieu-Daudé]

On 16/1/25 22:09, 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,
> let the common "qemu/compiler.h" deal with that by having it
> include "qemu/clang-tsa.h" (renamed as qemu/clang-tsa.h.inc).
> Add a check to not include "qemu/clang-tsa.h.inc" directly,
> remove previous "qemu/clang-tsa.h" inclusions.
>
> Suggested-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> 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/exec-all.h                       | 516 ------------------

Oops, weird thing happened while rebasing!

>   include/exec/page-protection.h                |   2 -
>   include/qemu/compiler.h                       |   2 +
>   include/qemu/thread.h                         |   1 -
>   include/qemu/{clang-tsa.h => clang-tsa.h.inc} |   8 +-
>   block/create.c                                |   1 -
>   tests/unit/test-bdrv-drain.c                  |   1 -
>   tests/unit/test-block-iothread.c              |   1 -
>   util/qemu-thread-posix.c                      |   1 -
>   12 files changed, 8 insertions(+), 529 deletions(-)
>   delete mode 100644 include/exec/exec-all.h
>   rename include/qemu/{clang-tsa.h => clang-tsa.h.inc} (97%)