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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
