[PATCH] D104750: [LV] Support Interleaved Store Group With Gaps
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 22 14:49:26 PDT 2021
lebedev.ri added a comment.
In D104750#2834626 <https://reviews.llvm.org/D104750#2834626>, @dorit wrote:
> In D104750#2834571 <https://reviews.llvm.org/D104750#2834571>, @lebedev.ri wrote:
>
>> Costmodel is wrong: if there are gaps, you need to load the vector of original values, and insert the non-gap elements into it.
>
> but we are using a masked-store, the non-gaps will be masked-out, they are undefs...
Yep, didn't see this is only for when masked stores are used/allowed.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104750/new/
https://reviews.llvm.org/D104750
More information about the llvm-commits
mailing list