mtrofin wrote:

> I'm not an expert on branch-probability facilities, but the test changes look 
> highly valuable,
> 
> It seems awkward to add an extra flag to the block-printer, as it introduces 
> a number of conditionals -- would it be possible to refactor as a dedicated 
> `printBlockName` method rather than an extra option for block printing?

I think @MatzeB's suggestion tackles this, ptal.

https://github.com/llvm/llvm-project/pull/73593
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to