[PATCH] D15548: Remove the restriction that known and unknown probabilities cannot coexist when being normalized.

[Cong Hou via llvm-commits]

congh added a comment.

In http://reviews.llvm.org/D15548#312148, @davidxl wrote:

> While it makes sense to do this -- can we start to teach producers not to pass unknown Probabilities in the first place?


If you search addSuccessor in LLVM, you will find about 200 users for different platforms without probability passed explicitly. Some are conditionally used such as:

MBB->addSuccessor(Succ1);
if (...)

  MBB->addSuccessor(Succ2); 

It seems that a default probability is more convenient but I don't object to add probabilities explicitly. There are just some tradeoffs.


http://reviews.llvm.org/D15548