On Wed, Feb 9, 2022, 3:29 AM Zebediah Figura <zfigura@codeweavers.com> wrote:
We had to avoid strd previously due to some processors not supporting it
[1]. Is this instruction portable enough?

[1] https://bugs.winehq.org/show_bug.cgi?id=44365

LDRD and STRD are ARMv7 instructions. There is at least one instance of LDRD in dlls/ntdll/signal_arm.c, in call_consolidate_callback. If we're aiming for pre-ARMv7 CPUs, we might as well have to remove it.

Otherwise I'd just blame the assembler. Maybe use .inst instead?

-- 
Sincerely,
Jinoh Kang