[PATCH] D19948: Propagate branch metadata when some branch probability is missing.

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Sat May 7 12:28:53 PDT 2016


spatel added a comment.

I changed http://reviews.llvm.org/D19674 in order to avoid the msan bot failure; it seems unnecessarily ugly to me, but it's passing on that bot now. There's still an Apple internal bot issue that seems to be triggered by http://reviews.llvm.org/rL268767 , so it may need some other change.

For this patch, I'm wondering if we should be doing a one-off correction for a missing weight. Wouldn't it be better to fill in all of the missing info before we reach SimplifyCFG?


http://reviews.llvm.org/D19948





More information about the llvm-commits mailing list