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

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105020