Re: [PATCH 07/11] wine/asm.h: Use __ASM_CFI on clang too, despite not defining __GCC_HAVE_DWARF2_CFI_ASM

[Jacek Caban]

On 28.05.2020 23:40, Martin Storsjö wrote:
> On Thu, 28 May 2020, Jacek Caban wrote:
> > Hi Martin,
> >
> > On 28.05.2020 10:14, Martin Storsjo wrote:
> > > It has already been unconditionally enabled for macOS (which uses 
> > > clang).
> >
> > Should we use it when building with llvm-mingw and SEH is enabled?
>
> Right - if targeting a PE, we should definitely be emitting SEH 
> instead. (I have no idea how to express the fake frames used in the 
> following patch in SEH form though - but I guess there's still some 
> time before the core of ntdll can be built as a PE...)
>
> Maybe (defined(__clang__) && defined(__ELF__)) for this then?

Maybe (defined(__clang__) && defined(__GNUC__) && !defined(__SEH__))? It 
would take care of macOS as well.

Jacek