On 17/1/25 18: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.bouv...@linaro.org>
Reviewed-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
Reviewed-by: Kevin Wolf <kw...@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <phi...@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

Patch queued, thanks.

Reply via email to