[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

Thomas Lively via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 4 14:34:25 PDT 2018


tlively added inline comments.


================
Comment at: include/clang/Basic/BuiltinsWebAssembly.def:59
 
+BUILTIN(__builtin_wasm_add_saturate_s_i8x16, "V16cV16cV16c", "ncV:128:")
+BUILTIN(__builtin_wasm_add_saturate_u_i8x16, "V16cV16cV16c", "ncV:128:")
----------------
craig.topper wrote:
> Don't copy the "V:128:" part from X86. You just need the "nc".  The V:128: is a special thing I'm working on for our AVX512 frequency issues.
ok, sounds good


Repository:
  rC Clang

https://reviews.llvm.org/D52858





More information about the cfe-commits mailing list