[PATCH] D25435: Add -fdebug-info-for-profiling to emit more debug info for sample pgo profile collection

David Blaikie via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 9 11:59:11 PST 2017


I started the llvm-dev thread for the instruction location tradeoffs
(removing the location from instructions to improve profiling support -
possibly at the cost of other debug info consumers).

The motivation for this flag is different - for added functionality that's
generally (imho/based on conversations so far) pretty well justified for
profile needs (the need for function start lines to make profiles somewhat
resilient to unrelated code changes, etc - as mentioned in the review/patch
description).

On Mon, Jan 9, 2017 at 11:51 AM Eric Christopher via Phabricator <
reviews at reviews.llvm.org> wrote:

> echristo added a comment.
>
> I don't think this ever was hashed out in the llvm-dev thread?
>
>
> https://reviews.llvm.org/D25435
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170109/a5968aed/attachment.html>


More information about the cfe-commits mailing list