[libc-commits] [libc] [libc][mathvec] Initial commit for LIBC vector math component (PR #173058)

[Muhammad Bassiouni via libc-commits]

================
@@ -0,0 +1,16 @@
+//===-- Single-precision SIMD e^x vector function -------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#include "src/mathvec/expf.h"
+#include "src/__support/mathvec/expf.h"
+
+namespace LIBC_NAMESPACE_DECL {
+
+cpp::simd<float> expf(cpp::simd<float> x) { return mathvec::expf(x); }
----------------
bassiounix wrote:

Shouldn't this be wrapped with `LLVM_LIBC_FUNCTION`?
I'm not aware of the dynamics of this new library but from my understanding every public entry/symbol in libc should be wrapped with this macro.

```suggestion
LLVM_LIBC_FUNCTION(cpp::simd<float>, expf, (cpp::simd<float> x)) {
  return mathvec::expf(x);
}
```

So my question becomes: is this public symbol in `libmvec`?
cc @michaelrj-google @lntue @jhuber6 

https://github.com/llvm/llvm-project/pull/173058