[PATCH] D57779: [SLP] Add support for throttling.

Dinar Temirbulatov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 15:54:37 PDT 2020


dtemirbulatov added a comment.

In D57779#1932289 <https://reviews.llvm.org/D57779#1932289>, @dtemirbulatov wrote:

> Changed getInsetCost() to correct insert cost calculation by allowing getEntryCost to handle entries that proposed to gather similarly as NeedToGather entries.


oh, I forgot to consider multiply use of a single tree entry by two or more different entry users, we need to calculate that our entry gather cost only once.


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

https://reviews.llvm.org/D57779





More information about the llvm-commits mailing list