Re: [PATCH v5 0/1] MR1065: ntdll: Add CFI unwind info to __wine_syscall_dispatcher (x86_64)

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/ntdll/unix/signal_x86_64.c:
> -
>               __ASM_CFI_REG_IS_AT2(r13, rcx, 0xd8, 0x00)
>               "movq %r14,0x60(%rcx)\n\t"
> -
>               __ASM_CFI_REG_IS_AT2(r14, rcx, 0xe0, 0x00)
>               "movq %r15,0x68(%rcx)\n\t"
> -
>               __ASM_CFI_REG_IS_AT2(r15, rcx, 0xe8, 0x00)
>               "movw %cs,0x78(%rcx)\n\t"
>               "movw %ds,0x7a(%rcx)\n\t"
>               "movw %es,0x7c(%rcx)\n\t"
>               "movw %fs,0x7e(%rcx)\n\t"
>               "movq %rsp,0x88(%rcx)\n\t"
> -
>               __ASM_CFI_CFA_IS_AT2(rcx, 0x88, 0x01)
> -
>               __ASM_CFI_REG_IS_AT2(rsp, rcx, 0x88, 0x01)
>               "movw %ss,0x90(%rcx)\n\t"
>               "movw %gs,0x92(%rcx)\n\t"
>               "movq %rbp,0x98(%rcx)\n\t"
> -
>               __ASM_CFI_REG_IS_AT2(rbp, rcx, 0x98, 0x01)

Any reason not to add the CFI for all the registers here?