[PATCH] D53724: [WebAssembly] Fix type names in truncation builtins

Heejin Ahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 25 15:03:41 PDT 2018


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





More information about the cfe-commits mailing list