Re: [PATCH v3 01/13] qemu/atomic: Drop special case for unsupported compiler

[Stefan Hajnoczi <stefanha@xxxxxxxxxx>]

On Thu, Dec 10, 2020 at 05:47:40PM +0400, marcandre.lureau@xxxxxxxxxx wrote:
> From: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
> 
> Since commit efc6c070aca ("configure: Add a test for the
> minimum compiler version") the minimum compiler version
> required for GCC is 4.8, which has the GCC BZ#36793 bug fixed.
> 
> We can safely remove the special case introduced in commit
> a281ebc11a6 ("virtio: add missing mb() on notification").
> 
> With clang 3.4, __ATOMIC_RELAXED is defined, so the chunk to
> remove (which is x86-specific), isn't reached either.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
> Reviewed-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
>  include/qemu/atomic.h | 17 -----------------
>  1 file changed, 17 deletions(-)

Reviewed-by: Stefan Hajnoczi <stefanha@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature