[PATCH] D99623: [WebAssembly] Implement i64x2 comparisons

Thomas Lively via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 31 10:19:02 PDT 2021


tlively added inline comments.


================
Comment at: clang/include/clang/Basic/BuiltinsWebAssembly.def:218
 
-TARGET_BUILTIN(__builtin_wasm_eq_i64x2, "V2LLiV2LLiV2LLi", "nc", "simd128")
-
----------------
dschuff wrote:
> Is the builtin/intrinsic wrong now? Or just not necessary because we can use builtin operators?
Just not necessary any more. That was only there to make sure no one could use the prototype instruction by accident.


================
Comment at: llvm/test/CodeGen/WebAssembly/simd-select.ll:313
 ; CHECK-NEXT:    # fallthrough-return
                                     <2 x i64> %x, <2 x i64> %y) {
   %c = icmp slt <2 x i64> %a, %b
----------------
dschuff wrote:
> pre-existing, but is there a reason why the CHECKs here are in the middle of the IR function signature?
Wow, I hadn't noticed that. Will investigate.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99623/new/

https://reviews.llvm.org/D99623



More information about the cfe-commits mailing list