[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