[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