[PATCH] D141309: [SLP] Do not ignore ordering for root node when it has in-tree uses.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 11:50:33 PST 2023


ABataev added inline comments.


================
Comment at: llvm/test/Transforms/SLPVectorizer/X86/reorder-phi-operand.ll:87
+; Here PHIs have mutual uses of each other. Reordering one requires reordering the other.
+define void @test2(ptr %p1, ptr %p2) {
+; CHECK-LABEL: @test2(
----------------
vdmitrie wrote:
> ABataev wrote:
> > Precommit the test, please.
> It fails with assertion without the fix. It represents a stability issue.
Ah, I see


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141309



More information about the llvm-commits mailing list