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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7777
   }
-  assert(NumToSchedule == 0 && "could not schedule all instructions");
 
----------------
reames wrote:
> ABataev wrote:
> > Can we keep this assert here or replace it with another one? It helps in many cases with incorrect scheduling.
> Not easily.  We'd need to track the increments through the calls to calculateDependencies since the set size now depends on the transitive use walk.
> 
> I get why you want this, but I don't see an easy way to preserve it.  
> 
> Do you think it's worth the complexity of plumbing an assert only param through calculateDependencies? 
No, sure not. But can you try to implement something simple here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118538