[PATCH] D32124: [BPI] Move tail computation out of the loop. NFC

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 17:39:50 PDT 2017


chandlerc added a comment.

In https://reviews.llvm.org/D32124#732170, @skatkov wrote:

> BTW, probably we should not add the tail to any edge at all... It might be that the information that two edges have exactly the same probability is mot important than the requirement that the sum of all probabilities should be 1. it seems that sum == 1 is not a strict requirement. For example, if we have three edges with the same weight then as a result the sum of probabilities will not be equal to one.  What do you think?


I think we already handle this kind of case in a few places in BPI and we should be consistent there. I can go digging for it but you may already know where it is...


https://reviews.llvm.org/D32124





More information about the llvm-commits mailing list