|
From: | Richard Henderson |
Subject: | Re: [PATCH v2] qemu/compiler: Absorb 'clang-tsa.h' |
Date: | Fri, 17 Jan 2025 08:26:25 -0800 |
User-agent: | Mozilla Thunderbird |
On 1/16/25 13:11, 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.
I think merging this into compiler.h itself would be better. r~
[Prev in Thread] | Current Thread | [Next in Thread] |