xregs_via_sig: Add avx registers to test

[dchagin]

Fix build on Linux

[kib]

tools/test/xregs_via_sig/c2x2c_amd64.S

20 ↗

(On Diff #122739)

We traditionally spell the directive as '.globl'

33 ↗

(On Diff #122739)

movl $1, %eax
You do cmpl $1, %eax after the function call.

36 ↗

(On Diff #122739)

xorl %eax, %eax
It is same as xorq you used, but avoids prefix. You do not need %rax anyway.

[dchagin]

Done, thank you

[kib]

tools/test/xregs_via_sig/c2x2c_amd64.S

24 ↗

(On Diff #122751)

CPUID instruction corrupts %ebp, which is callee-saved register.

[dchagin]

tools/test/xregs_via_sig/c2x2c_amd64.S

24 ↗

(On Diff #122751)

you mean %ebx?

[jhb]

tools/test/xregs_via_sig/c2x2c_amd64.S

8 ↗

(On Diff #122751)

I think xregs_bank should be in C instead. I would rather that you just export cpu_to_xmm, cpu_to_ymm, etc., invoke cpuid once during startup to save the desired bank in a global in C and have C wrappers to choose which MD routines to invoke.

[kib]

tools/test/xregs_via_sig/c2x2c_amd64.S

24 ↗

(On Diff #122751)

Yes %ebx sorry

[dchagin]

Reworked, based on D40410 now

[dchagin]

fix arm64 xregs_banks_max

[kib]

tools/test/xregs_sig/c2x2c_amd64.S

23

Please use the size instruction suffixes consistently