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

Simon Moll via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 1 02:30:26 PDT 2021


simoll accepted this revision.
simoll added a comment.
This revision is now accepted and ready to land.

In D99355#2850296 <https://reviews.llvm.org/D99355#2850296>, @hussainjk wrote:

> In D99355#2849287 <https://reviews.llvm.org/D99355#2849287>, @simoll wrote:
>
>> One more thing: title & summary do not reflect what's in the patch anymore. Please update those too.
>
> Ahh, can you clarify?

My bad. I had overlooked that the `VPIntrinsics.def` implicitly instantiates the SDNodes. (I thought for a bit there were no SDNodes in this patch anymore). You're all good :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99355



More information about the llvm-commits mailing list