dexonsmith added a comment.

In https://reviews.llvm.org/D51240#1213358, @davidxl wrote:

> Re  "Why not":
>
> The common use model of instrumentation based PGO and SamplePGO are quite 
> different. The former usually uses 'fresh' profile that matches the source. 
> If there are massive code refactoring or ABI changes, the user can simply 
> regenerate the profile.  For the latter, the profile data is usually 
> collected from production binaries, so the source is almost always newer than 
> the profile.


Frontend-based instrumentation is designed to degrade gracefully as the source 
changes.  I don't follow your conclusion that the profile is necessarily fresh. 
 (I'm fine with the patch being split though.)


https://reviews.llvm.org/D51240



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D51240: A... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D512... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D512... David Li via Phabricator via cfe-commits
    • [PATCH] D512... Duncan P. N. Exon Smith via Phabricator via cfe-commits
    • [PATCH] D512... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D512... David Li via Phabricator via cfe-commits
    • [PATCH] D512... Duncan P. N. Exon Smith via Phabricator via cfe-commits

Reply via email to