[PATCH] D75296: [SLP][NFC] Assert that tree entry operands completed when scheduler looks for dependencies.

Valeriy Dmitriev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 27 13:30:34 PST 2020


vdmitrie added a comment.

In D75296#1896620 <https://reviews.llvm.org/D75296#1896620>, @ABataev wrote:

> Maybe it is better to implement the two-steps approach? If it may lead to a crash, it will definitely cause a crash somewhere


I'm not sure I got your question. The two steps approach is what is implemented and used at the moment. Example in the summary shows situation leading to firing the assertion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75296





More information about the llvm-commits mailing list