[PATCH] D99355: Implementation of intrinsic and SDNode definitions for VP load, store, gather, scatter.

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

LGTM other than the last few nits.



================
Comment at: llvm/lib/IR/IntrinsicInst.cpp:300
+  assert(PtrParamOpt.hasValue() && "no pointer argument!");
+  return this->getParamAlign(PtrParamOpt.getValue());
+}
----------------
Is the explicit `this->` required?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99355