[PATCH] D136757: [SLP] Extend reordering data of tree entry to support PHI nodes

[David Spickett via Phabricator via llvm-commits]

DavidSpickett added a comment.

Hi @skc7, this has caused a test suite failure on Linaro's SVE vector length agnostic buildbot.

   #8 0x0000ffff9be1a490 __assert_fail_base /build/glibc-RIFKjK/glibc-2.31/assert/assert.c:89:7
   #9 0x0000ffff9be1a4f4 (/lib/aarch64-linux-gnu/libc.so.6+0x2d4f4)
  #10 0x0000aaaae40da824 (/home/tcwg-buildbot/worker/clang-aarch64-sve-vla/stage1.install/bin/clang+0x7287824)
  #11 0x0000aaaae410ab68 areTwoInsertFromSameBuildVector(llvm::InsertElementInst*, llvm::InsertElementInst*, llvm::function_ref<llvm::Value* (llvm::InsertElementInst*)>) SLPVectorizer.cpp:0:0
  #12 0x0000aaaae41284d8 llvm::slpvectorizer::BoUpSLP::getReorderingData(llvm::slpvectorizer::BoUpSLP::TreeEntry const&, bool)::$_10::operator()(llvm::Value*, llvm::Value*) const SLPVectorizer.cpp:0:0
  #13 0x0000aaaae4129328 void std::__merge_adaptive<llvm::Value**, long, llvm::Value**, __gnu_cxx::__ops::_Iter_comp_iter<llvm::slpvectorizer::BoUpSLP::getReorderingData(llvm::slpvectorizer::BoUpSLP::TreeEntry const&, bool)::$_10>>(llvm::Value**, llvm::Value**, llvm::Value**, long, long, llvm::Value**, long, __gnu_cxx::__ops::_Iter_comp_iter<llvm::slpvectorizer::BoUpSLP::getReorderingData(llvm::slpvectorizer::BoUpSLP::TreeEntry const&, bool)::$_10>) SLPVectorizer.cpp:0:0

When building McCat eks (whatever that is).

https://lab.llvm.org/buildbot/#/builders/197/builds/3237

The logs are pretty jumbled, so we'll dig out the reproducer for you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136757