[PATCH] D64911: [AMDGPU] Extend the SI Load/Store optimizer

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 19 07:25:52 PDT 2019


arsenm added a comment.

In D64911#1593515 <https://reviews.llvm.org/D64911#1593515>, @nhaehnle wrote:

> In D64911#1591502 <https://reviews.llvm.org/D64911#1591502>, @arsenm wrote:
>
> > I still think we should be handling these on the IR level
>
>
> Matt, where do you think would be the right place to do this at the IR level? Presumably you have an existing pass in mind?


Either a new pass, or teach the LoadStoreVectorizer about intrinsics. Things are generally easier in the IR, and I don't think this needs any information only available after selection


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64911





More information about the llvm-commits mailing list