[PATCH] D52856: [WebAssembly] __builtin_wasm_replace_lane_* builtins

[Thomas Lively via Phabricator via cfe-commits]

tlively added inline comments.


================
Comment at: include/clang/Basic/BuiltinsWebAssembly.def:55
+BUILTIN(__builtin_wasm_replace_lane_i32x4, "V4iV4iIii", "ncV:128:")
+BUILTIN(__builtin_wasm_replace_lane_i64x2, "V2WiV2WiIiWi", "ncV:128:")
+BUILTIN(__builtin_wasm_replace_lane_f32x4, "V4fV4fIif", "ncV:128:")
----------------
craig.topper wrote:
> aheejin wrote:
> > craig.topper wrote:
> > > I think maybe you want LL instead of W, but I'm not sure.
> > Not sure what the differences are either. I've used LL for other builtins in this file though.
> W changes based on what's returned from getInt64Type() for the target. I think it will be either L or LL. It looks to be almost exclusively used by MSVC compatibility intrinsics except for two builtins in NVPTX.
Ok, I'll switch to using LL.


Repository:
  rC Clang

https://reviews.llvm.org/D52856