[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