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

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 27 07:19:04 PDT 2021


anton-afanasyev added a comment.

>> I believe it will work correct after lowering fixes. I'm still investigating this case https://bugs.llvm.org/show_bug.cgi?id=50015, this is definitly lowering issue since llvm code looks correct.
>> Ok, I think this patch is good temp solution until miscompile issue isn't solved.
>
> Maybe better to rework it and land it in X86TargetTransfromInfo.cpp for better cost modelling if it is not legal directly?

Yes, I believe this will fix PR50015, though it doesn't fix the whole problem related to lowering.


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