[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