[PATCH] D52858: [WebAssembly] saturating arithmetic builtins

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 3 21:06:54 PDT 2018


craig.topper 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:")
----------------
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.


Repository:
  rC Clang

https://reviews.llvm.org/D52858





More information about the cfe-commits mailing list