[PATCH] D118538: [SLP] Schedule only sub-graph of vectorizable instructions

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added a comment.

Hi,

Heads up that I've seen the following assert (added in 6253b77da <https://reviews.llvm.org/rG6253b77da9f3fd7ee3286a2b4d39e87b0f0f55f5>) trigger in one of our downstream tests:

  opt: ../lib/Transforms/Vectorize/SLPVectorizer.cpp:8066: void llvm::slpvectorizer::BoUpSLP::BlockScheduling::calculateDependencies(llvm::slpvectorizer::BoUpSLP::ScheduleData *, bool, llvm::slpvectorizer::BoUpSLP *): Assertion `DepDest && "must be in schedule window"' failed.

I'll try to extract a reproducer and come back.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118538