[PATCH] D105020: [SLP]Improve graph reordering.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D105020#2912776 <https://reviews.llvm.org/D105020#2912776>, @sdesmalen wrote:

> Hi @ABataev, I ran into an issue when running the LLVM test-suite. It seems to be a different issue than the one that @mstorsjo reported.
>
> I got it reduced to:
>
>   target triple = "aarch64-unknown-linux-gnu"
>   
>   define void @foo() local_unnamed_addr {
>   entry:
>     %0 = load volatile double, double* poison, align 8
>     %1 = load volatile double, double* poison, align 8
>     %2 = load volatile double, double* poison, align 8
>     %3 = load volatile double, double* poison, align 8
>     br label %for.body
>   
>   for.body:                                         ; preds = %for.body, %entry
>     %d30.0734 = phi double [ undef, %for.body ], [ %0, %entry ]
>     %d01.0733 = phi double [ undef, %for.body ], [ %1, %entry ]
>     %d11.0732 = phi double [ undef, %for.body ], [ %2, %entry ]
>     %d21.0731 = phi double [ undef, %for.body ], [ %3, %entry ]
>     br label %for.body
>   }
>
> Run with: `opt -slp-vectorizer -S < reduced.ll`.
>
> I had actually expected one of the aarch64-buildbots would have caught this, so not sure if there was something special about the way I ran the test-suite, but I don't believe so.

Investigated it, the crash is not related to this patch, caused by one of the previous patches. Going to publish a fix soon.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105020