[PATCH] Allow 0-weight branches in BranchProbabilityInfo.

Diego Novillo dnovillo at google.com
Wed May 6 10:39:56 PDT 2015



On 05/06/15 13:37, Hans Wennborg wrote:
> On Wed, May 6, 2015 at 10:09 AM, Duncan P. N. Exon Smith
>
> Are you saying you don't want to guarantee that anymore, or that
> forcing each weight <= UINT32_MAX / SI.getNumSuccessors() is
> unnecessarily strict?

Yeah, I've wanted to remove this restriction for some time. The only 
requirement we really need is that the sum of all the weights fits 
within 32 bits.

I'm going to be removing this restriction in http://reviews.llvm.org/D9442


Diego.



More information about the llvm-commits mailing list