[PATCH] D24144: [CFGPrinter] Display branch weight on the edges

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 16:44:52 PDT 2016


anemet marked an inline comment as done.
anemet added a comment.

In https://reviews.llvm.org/D24144#531757, @davidxl wrote:

> The problem I see is that people may confuse 'weight' with actual profile count, while in-reality it may not be the case (due to scaling). Perhaps add '(W)' after the number?


Makes sense.  I went with 'W:', prepended, e.g.:

https://reviews.llvm.org/F2392296

The W after made me think of electric power ;)


https://reviews.llvm.org/D24144





More information about the llvm-commits mailing list