[PATCH] D88154: Initial support for vectorization using Libmvec (GLIBC vector math library).

[Renato Golin via Phabricator via llvm-commits]

rengolin added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:1564
 def fveclib : Joined<["-"], "fveclib=">, Group<f_Group>, Flags<[CC1Option]>,
-    HelpText<"Use the given vector functions library">, Values<"Accelerate,MASSV,SVML,none">;
+    HelpText<"Use the given vector functions library">, Values<"Accelerate,LIBMVEC,MASSV,SVML,none">;
 def fno_lax_vector_conversions : Flag<["-"], "fno-lax-vector-conversions">, Group<f_Group>,
----------------
abique wrote:
> I think glibc always refer to the library as "libmvec" in lower case,  should we do so here?
Yes, so clang can be a drop in replacement. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88154/new/

https://reviews.llvm.org/D88154