Draft commit for reviews and questions

Author: badumbatish

llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp

@@ -93,6 +93,8 @@ WebAssemblyTargetLowering::WebAssemblyTargetLowering(
       setOperationAction(ISD::LOAD, T, Custom);
       setOperationAction(ISD::STORE, T, Custom);
     }
+
+    setOperationAction(ISD::LOAD, MVT::i128, Custom);
   }
   if (Subtarget->hasFP16()) {
     setOperationAction(ISD::LOAD, MVT::v8f16, Custom);
@@ -1609,6 +1611,19 @@ void WebAssemblyTargetLowering::ReplaceNodeResults(
   case ISD::SUB:
     Results.push_back(Replace128Op(N, DAG));
     break;
+  case ISD::STORE:
+    // TODO: I think i can get away with just LOAD for now
+    break;
+  case ISD::LOAD: {
+    LoadSDNode *LD = cast<LoadSDNode>(N);
+    EVT VT = LD->getValueType(0);
+    assert(VT == MVT::i128);
+    SDValue Pair = DAG.getLoad(MVT::v2i64, SDLoc(LD), LD->getChain(),
+                               LD->getBasePtr(), LD->getMemOperand());
+    Results.push_back(Pair.getValue(0));
+    Results.push_back(Pair.getValue(1));
+    break;
+  }
   default:
     llvm_unreachable(
         "ReplaceNodeResults not implemented for this op for WebAssembly!");

llvm/test/CodeGen/WebAssembly/simd-load-custom-128.ll

@@ -6,10 +6,8 @@ define void @i128_to_v128(ptr %ptr) {
 ; CHECK-LABEL: i128_to_v128:
 ; CHECK:         .functype i128_to_v128 (i32) -> ()
 ; CHECK-NEXT:  # %bb.0:
-; CHECK-NEXT:    i64.load $push0=, 0($0)
-; CHECK-NEXT:    i64.store 0($0), $pop0
-; CHECK-NEXT:    i64.load $push1=, 8($0)
-; CHECK-NEXT:    i64.store 8($0), $pop1
+; CHECK-NEXT:    v128.load $push0=, 0($0)
+; CHECK-NEXT:    v128.store 0($0), $pop0
 ; CHECK-NEXT:    return
     %2 = load i128, ptr %ptr
     store volatile i128 %2, ptr %ptr