[PATCH] D119855: [SLP]Fix vectorization of the alternate cmp instruction with swapped predicates.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 17 08:41:24 PST 2022


ABataev added a comment.

In D119855#3329852 <https://reviews.llvm.org/D119855#3329852>, @bgraur wrote:

> Since we had so many issues related to the initial patch (https://reviews.llvm.org/D115955), would you please consider reverting that and work on the fix without the stress of adding new issues?

The problems appear in a single corner case - when the main and alternate opcode has swapped predicates. I'll try to fix it ASAP (this case is pretty rare and tricky, did not see any problems with SPEC 2006/2017), if won't be able to fix it soon, will revert the part for swapped alternate vectorization.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119855



More information about the llvm-commits mailing list