[PATCH] D110864: [llvm-profgen] Update test case to check branch count against outline function

[Wenlei He via Phabricator via llvm-commits]

wenlei added a comment.

> For llvm-profgen side, If it gets an outlined function with suffixes, currently we only generate a zero total count and non-zero head count FunctionSample like "foo.cold:0:100".

This seems like an inconsistent state, and it would also confuse profile similarity check depending on whether we have split/outline or not.

> For compiler side, If the outlined function is generated before the SampleLoader, the compiler will consume it, otherwise it will ignore it.

Where do we attribute the body samples for the funclet? If that's with the main body (instead of function), we won't be able to consume the funclet profile for splitting before sample loader.

Though I think most split/outline happens after sample loader, and it's cleaner if we attribute all counts to the original function, then left to the split/outline to decide how to split the counts.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110864/new/

https://reviews.llvm.org/D110864