aheejin added inline comments.

================
Comment at: lib/CodeGen/CGBuiltin.cpp:12528
+  case WebAssembly::BI__builtin_wasm_trunc_saturate_s_f32x4:
+  case WebAssembly::BI__builtin_wasm_trunc_saturate_s_f64x2: {
     Value *Src = EmitScalarExpr(E->getArg(0));
----------------
But unlike others, isn't the input type different from the output type for this 
builtin? It seems other non-SIMD `saturate_s` intrinsics follow the same 
convention, `trunc_saturate_s_output_input`. 


Repository:
  rC Clang

https://reviews.llvm.org/D53724



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

Reply via email to