tlively added inline comments.

================
Comment at: clang/lib/Headers/wasm_simd128.h:969
+static __inline__ v128_t __DEFAULT_FN_ATTRS wasm_f32x4_ceil(v128_t __a) {
+  return (v128_t)__builtin_wasm_ceil_f32x4((__f32x4)__a);
+}
----------------
aheejin wrote:
> Sometimes builtin names seem slightly different from intrinsic names like 
> this case: the intrinsic name is `f32x4_ceil` while the builtin name is 
> `ceil_f32x4`. Is that intentional?
Yes, the builtin names use _s and _u rather than communicating the sign in the 
lane interpretation and they always put the instruction prefix at the end. It's 
just a different arbitrary naming convention.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101112

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to