-Wunused-function diagnostic firing for amxcomplexintrin.h

[ZacharyHenkel]

Best guess is that these two functions should be static __inline__ to match the other functions in the same file.

<path>\lib\clang\17\include\amxcomplexintrin.h(139,13) : error: unused function '__tile_cmmimfp16ps' [-Werror,-Wunused-function]
  139 | static void __tile_cmmimfp16ps(__tile1024i *dst, __tile1024i src0,
      |             ^~~~~~~~~~~~~~~~~~
<path>\lib\clang\17\include\amxcomplexintrin.h(162,13) : error: unused function '__tile_cmmrlfp16ps' [-Werror,-Wunused-function]
  162 | static void __tile_cmmrlfp16ps(__tile1024i *dst, __tile1024i src0,
      |             ^~~~~~~~~~~~~~~~~~
2 errors generated.

[llvmbot]

@llvm/issue-subscribers-backend-x86

Author: Zachary Henkel (ZacharyHenkel)

Best guess is that these two functions should be `static __inline__` to match the other functions in the same file.

<path>\lib\clang\17\include\amxcomplexintrin.h(139,13) : error: unused function '__tile_cmmimfp16ps' [-Werror,-Wunused-function]
  139 | static void __tile_cmmimfp16ps(__tile1024i *dst, __tile1024i src0,
      |             ^~~~~~~~~~~~~~~~~~
<path>\lib\clang\17\include\amxcomplexintrin.h(162,13) : error: unused function '__tile_cmmrlfp16ps' [-Werror,-Wunused-function]
  162 | static void __tile_cmmrlfp16ps(__tile1024i *dst, __tile1024i src0,
      |             ^~~~~~~~~~~~~~~~~~
2 errors generated.

[RKSimon]

CC @phoebewang @KanRobert

[phoebewang]

@ZacharyHenkel Thanks for reporting the issue. Can you please comment the command you used? I tried with below command but cannot reproduce it.

$ cat tmp.c
#include <immintrin.h>

int main() { return 0; }
$ clang -S tmp.c -Werror -Wunused-function

Although it's not conformity with others, we have __DEFAULT_FN_ATTRS_COMPLEX which sets __always_inline__. I want to make sure in which condition triggers the warning.

[ZacharyHenkel]

@phoebewang ack. I'll take a look at providing additional repro details when we take up the LLVM 20 upgrade

[AaronBallman]

Any update on whether this issue still reproduces?

[ZacharyHenkel]

The upgrade testing is in progress this week. It looks like a tentative "no it doesn't reproduce", but I'll provide a final answer no later than Monday 4/7.

[ZacharyHenkel]

We have green builds. Closing the issue.