Re: [PATCH v20 0/7] MR450: vkd3d-shader/ir: Flatten structured control flow instructions.

[Giovanni Mascellani (＠giomasce)]

Giovanni Mascellani (@giomasce) commented about libs/vkd3d-shader/d3d_asm.c:
> switch (ins->handler_idx)
>      {
> +        case VKD3DSIH_BRANCH:
> +            if (vsir_register_is_label(&ins->src[0].reg))
> +                break;
> +            /* fall through */

Does it make sense to allow flags NZ and Z on BRANCH? The instruction specifies both labels anyway, so changing the flag is just equivalent to swapping the labels. It seems simpler to just say that flags NZ and Z do not apply and the first branch is always NZ and the second one is always Z.

-- 
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/450#note_56999