[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