> -----Original Message-----
> From: Jakub Jelinek <[email protected]>
> Sent: Wednesday, December 17, 2025 2:12 PM
> To: Uros Bizjak <[email protected]>; Liu, Hongtao <[email protected]>
> Cc: [email protected]
> Subject: [PATCH] i386: Obfuscate _mm_maskmove_si64 local variable
> [PR123155]
>
> Hi!
>
> Even automatic variables in inline functions in installed headers need to be
> obfuscated, as e.g.
> #define offset +++
> #include <x86intrin.h>
> is a valid program. Most of the automatic variables are obfuscated, but one
> has leaked in in the xmmintrin.h header.
> gcc -S -O2 -O2 -Werror-implicit-function-declaration -march=novalake -
> msse4a -m3dnow -mavx -mavx2 -mfma4 -mxop -maes -mpclmul -mpopcnt -
> mabm -mlzcnt -mbmi -mbmi2 -mtbm -mlwp -mfsgsbase -mrdrnd -mf16c -
> mfma -mrtm -mrdseed -mprfchw -madx -mfxsr -mxsaveopt -msha -mxsavec -
> mxsaves -mclflushopt -mavx512vp2intersect -mclwb -mmwaitx -mclzero -
> mpku -msgx -mrdpid -mgfni -mpconfig -mwbnoinvd -menqcmd -mserialize -
> mtsxldtrk -mamx-tile -mamx-int8 -mamx-bf16 -mkl -mwidekl -mavxvnni -
> mavxifma -mavxvnniint8 -mavxneconvert -mcmpccxadd -mamx-fp16 -
> mprefetchi -mraoint -mamx-complex -mavxvnniint16 -msm3 -msha512 -
> msm4 -mavx10.2 -mamx-avx512 -mamx-tf32 -mamx-fp8 -mmovrs -mamx-
> movrs -g -dA gcc/testsuite/gcc.target/i386/sse-13.c -fno-eliminate-unused-
> debug-{symbols,types} -o sse-13.s grep -A1 DW_TAG_variable sse-13.s | grep
> DW_AT_name | grep -v 'scii "__' | grep -v 'DW_AT_name: "__'
> shows variables not prefixed with __ and the following patch fixes it.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk and later
> on backport to 15/14/13 branch)?
Ok.
>
> 2025-12-17 Jakub Jelinek <[email protected]>
>
> PR target/123155
> * config/i386/xmmintrin.h (_mm_maskmove_si64): Rename offset
> automatic
> variable to __offset.
>
> --- gcc/config/i386/xmmintrin.h.jj 2025-02-13 19:59:55.293577243
> +0100
> +++ gcc/config/i386/xmmintrin.h 2025-12-16 14:25:15.222850385
> +0100
> @@ -1223,17 +1223,17 @@ _mm_maskmove_si64 (__m64 __A, __m64
> __N,
> __v2di __N128 = __extension__ (__v2di) { ((__v1di) __N)[0], 0 };
>
> /* Check the alignment of __P. */
> - __SIZE_TYPE__ offset = ((__SIZE_TYPE__) __P) & 0xf;
> - if (offset)
> + __SIZE_TYPE__ __offset = ((__SIZE_TYPE__) __P) & 0xf; if (__offset)
> {
> /* If the misalignment of __P > 8, subtract __P by 8 bytes.
> Otherwise, subtract __P by the misalignment. */
> - if (offset > 8)
> - offset = 8;
> - __P = (char *) (((__SIZE_TYPE__) __P) - offset);
> + if (__offset > 8)
> + __offset = 8;
> + __P = (char *) (((__SIZE_TYPE__) __P) - __offset);
>
> /* Shift __A128 and __N128 to the left by the adjustment. */
> - switch (offset)
> + switch (__offset)
> {
> case 1:
> __A128 = __builtin_ia32_pslldqi128 (__A128, 8);
>
> Jakub
