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

Dehao Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 11 21:47:16 PST 2016


danielcdh added a comment.

In https://reviews.llvm.org/D25435#619169, @hfinkel wrote:

> In https://reviews.llvm.org/D25435#609320, @danielcdh wrote:
>
> > change the flag name to -fprofile-debug
>
>
> I don't really like this name because it sounds like it might be enabling some kind of debugging of the profiling. How about -fdebug-info-for-profiling. Another options is to make it a mode for -g such as -gprofiling.


Thanks for the comment. I'm OK with -fdebug-info-for-profiling. If no one objects, I'll update the patch tomorrow morning.

About -gprofiling, as discussed in http://lists.llvm.org/pipermail/llvm-dev/2016-November/107454.html, this should be a -f option instead of -g option.


https://reviews.llvm.org/D25435





More information about the cfe-commits mailing list