[PATCH] D105871: implementation of sdag support for VP memory intrinsics

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/SelectionDAGNodes.h:2354
+  const SDValue &getOffset() const {
+    return getOperand(getOpcode() == ISD::MLOAD ? 2 : 3);
+  }
----------------
rogfer01 wrote:
> Sorry for reviving, this (I was working on a patch for strided load store).
> 
> Shouldn't this have been `ISD::VP_LOAD` instead?
Oh, good spot! I don't think anyone's using the shared base class (everyone uses the concrete subclasses which have their own `getOffset`.

Do you want to submit a patch or shall I?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105871