aheejin added a comment. Do loads need the vector argument? The proposal has that but I'm wondering why... Can that possibly be an error from the author's side?
================ Comment at: clang/include/clang/Basic/BuiltinsWebAssembly.def:180 +TARGET_BUILTIN(__builtin_wasm_load32_lane, "V4iIii*", "nU", "simd128") +TARGET_BUILTIN(__builtin_wasm_load64_lane, "V2LLiIiLLi*", "nU", "simd128") +TARGET_BUILTIN(__builtin_wasm_store8_lane, "vV16ScIiSc*", "n", "simd128") ---------------- tlively wrote: > aheejin wrote: > > `U` in the third argument [[ > > https://github.com/llvm/llvm-project/blob/72732acade77d5ee55a818e2da77a2c5b7033ccb/clang/include/clang/Basic/Builtins.def#L75 > > | means ]] pure. Can we say loads are pure? (The same for existing > > `__builtin_wasm_load32_zero` and `__builtin_wasm_load64_zero`) > Yes, this is the difference between "pure" and "const." > > https://github.com/llvm/llvm-project/blob/master/clang/include/clang/Basic/Builtins.h#L97-L106 Can't SIMD loads trap? (Normal loads can AFAIK, right?) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89366/new/ https://reviews.llvm.org/D89366 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
