[PATCH] Allow 0-weight branches in BranchProbabilityInfo.

Diego Novillo dnovillo at google.com
Wed May 6 10:19:55 PDT 2015


On Wed, May 6, 2015 at 10:09 AM, Duncan P. N. Exon Smith
<dexonsmith at apple.com> wrote:

>
> This looks like `visitSwitch()`.  I don't see why we can't handle a
> weight of 0 here.
>
> It would be easy to change `SelectionDAGBuilder::splitWorkItem` to be
> resilient to weights of 0 instead:

I was going for mechanical changes in this patch. I want to have
something that's easy to bisect if I happen to break something else.

Perhaps a followup patch?

I just sent an updated patch with the change in visitSwitch(). That's
the only one that's required for the testsuite.


Diego.



More information about the llvm-commits mailing list