flame / blis

BLAS-like Library Instantiation Software Framework

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

ARMSVE blacklist for Clang 10 and lower

devinamatthews opened this issue · comments

The armsve config should be added to the blacklist for clang 10 and lower, as ARM SVE intrinsics support (used in the packm kernels) is only in clang11+. I am checking on gcc...

@fgvanzee you can probably handle this quite easily.

I'll wait until you finish checking for gcc.

gcc 10+ required

Okay, so disallow armsve for clang10 and older, and gcc 9 and older?

Yes. I don't know if this affects A64fx because it is only the intrinsics. @xrq-phys ?

@devinamatthews Do I need to also add the armsve blacklisting to the "Apple" branch of the clang conditional handling?

Indeed those sve-intrinsic kernels are not called by a64fx. If we block those kernels from being compiled (maybe by adding preprocessor guards?) GCC 10+ requirement might be lift.

For assembler sorry I'm not sure. At least 2.32+ is known to work.

By the way those sve-intrinsic kernels are not named according to the convension (should be bli_*_armsve_int_* instead of bli_*_armsve_*_asm_* I suppose). Maybe we can also correct this. :D

I believe this is fixed nowl