[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 09/13] audio: remove GNUC & MSVC check
|
From: |
Gerd Hoffmann |
|
Subject: |
Re: [PATCH v2 09/13] audio: remove GNUC & MSVC check |
|
Date: |
Fri, 27 Nov 2020 08:15:14 +0100 |
Hi,
> > #if defined AUDIO_BREAKPOINT_ON_BUG
> > # if defined HOST_I386
> > + __asm__ ("int3");
> > # else
> > abort ();
> > # endif
> > --
> > 2.29.0
>
> I would prefer to just drop this attempt to emit an inline
> breakpoint insn (which won't work on non-x86 hosts and seems
> to me to have no benefit over just calling abort(), which will
> also drop you into the debugger) and simply make it abort() if
> AUDIO_BREAKPOINT_ON_BUG is defined. Gerd, do you have an
> opinion ?
kraxel@sirius ~/projects/qemu# git grep AUDIO_BREAKPOINT_ON_BUG
audio/audio.c:#if defined AUDIO_BREAKPOINT_ON_BUG
Seems this is unused, so just remove it?
take care,
Gerd
- Re: [PATCH v2 05/13] qemu-plugin.h: remove GCC < 4, (continued)
[PATCH v2 06/13] tests: remove GCC < 4 fallbacks, marcandre . lureau, 2020/11/26
[PATCH v2 07/13] virtiofsd: replace _Static_assert with QEMU_BUILD_BUG_ON, marcandre . lureau, 2020/11/26
[PATCH v2 08/13] compiler.h: explicit case for Clang printf attribute, marcandre . lureau, 2020/11/26
[PATCH v2 09/13] audio: remove GNUC & MSVC check, marcandre . lureau, 2020/11/26
[PATCH v2 10/13] poison: remove GNUC check, marcandre . lureau, 2020/11/26
[PATCH v2 11/13] xen: remove GNUC check, marcandre . lureau, 2020/11/26
[PATCH v2 12/13] compiler: remove GNUC check, marcandre . lureau, 2020/11/26
[PATCH v2 13/13] linux-user: remove GNUC check, marcandre . lureau, 2020/11/26