[PATCH] D107228: [TTI] Add basic SK_InsertSubvector shuffle mask recognition

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

In D107228#2919231 <https://reviews.llvm.org/D107228#2919231>, @foad wrote:

> In D107228#2918570 <https://reviews.llvm.org/D107228#2918570>, @RKSimon wrote:
>
>> Treat a 'subvector widening' shuffle as free - this introduces what I think are benign changes to a amdgpu test
>
> Adding @dfukalov for awareness.

BTW - I'm intending to commit the widening cost = 0 patch separately so if it does cause any problems we can address them without affecting the SK_InsertSubvector recogniser. If it proves necessary we can add a SK_WidenSubvector class to handle those cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107228