[PATCH] D52856: [WebAssembly] __builtin_wasm_replace_lane_* builtins

[Craig Topper via Phabricator via cfe-commits]

craig.topper 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:")
----------------
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.


Repository:
  rC Clang

https://reviews.llvm.org/D52856