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