Build error on WoA when building with DYNAMIC_ARCH=ON

[dnoan]

Hello,
I am building OpenBLAS 0.3.31 on WoA with DYNAMIC_ARCH=ON DYNAMIC_LIST="NEOVERSEN1;CORTEXX1". I am building with Clang/Flang 21.1.8. When I try to link the resulting library in my program, I am getting the following error:

openblas.lib(sgemm_direct_alpha_beta_ARMV8.c.obj) : fatal error LNK1227: conflicting weak extern definition for '__arm_tpidr2_save'. New default '.weak.__arm_tpidr2_save.default.sgemm_direct_alpha_beta_ARMV8' conflicts with old default '.weak.__arm_tpidr2_save.default.sgemm_direct_alpha_beta_NEOVERSEN1' in openblas.lib(sgemm_direct_alpha_beta_NEOVERSEN1.c.obj)

I have never seen this error in previous versions of OpenBLAS.

[martin-frbg]

That looks to be a bug introduced with the recent addition of SME support (which neither of these two cpu models have) - in sgemm_direct_alpha_beta_arm64_sme1.c, the #include sme.abi.h needs to be commented out (or moved inside the #if defined (HAVE_SME) ).

[martin-frbg]

As far as I can tell, the two dummy functions provided by that header were implemented in or before LLVM 18.1.0, not so sure about gcc but probably 14.1, the first to officially support SME. So it may actually be safe to remove this local header (?)

[dnoan]

I commented out #include sme.abi.h and it worked. Thanks! Another issue on WoA with DYNAMIC_ARCH=ON has to do with VORTEXM4 - I had to comment out and modify some code in dynamic_arm64.c to get it compile.

[martin-frbg]

Oops. That's with a CMake build, I guess ?

[dnoan]

Yes, this happens when building with CMake.