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



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

Reply via email to