[PATCH] D101297: [SLP]Allow masked gathers only if allowed by target.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 27 06:18:06 PDT 2021


spatel added a comment.

In D101297#2719479 <https://reviews.llvm.org/D101297#2719479>, @ABataev wrote:

> In D101297#2719468 <https://reviews.llvm.org/D101297#2719468>, @spatel wrote:
>
>> We seem to go back and forth on legality vs. pure cost model to decide what is allowed for vectorization.
>> This would undo part of D90445 <https://reviews.llvm.org/D90445>, so adding potential reviewers for more feedback.
>
> Pure cost won't work here, we need to check if this is legal just to try to generate masked gather at all. Cost problem is different.

Is there some gather construct that is impossible for the backend to expand correctly or just that it can't be expanded efficiently?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101297



More information about the llvm-commits mailing list