[PATCH] D28746: Mention ThinLTO with PGO in ReleaseNotes 4.0

Teresa Johnson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 15 14:04:35 PST 2017


tejohnson added a comment.

In https://reviews.llvm.org/D28746#646793, @Prazek wrote:

> Maybe I should change ThinLTO for '-flto=thin' and also mention this improvement in LLVM release notes?
>
> BTW '-flto=thin' is not documented in UsersManual


It's documented here:

  http://clang.llvm.org/docs/index.html

and here:

  http://clang.llvm.org/docs/CommandGuide/clang.html

but not (yet) here:

  http://clang.llvm.org/docs/UsersManual.html

I can add that.

Thanks!



================
Comment at: docs/ReleaseNotes.rst:69
+  called from cold callsite.
+
 
----------------
mehdi_amini wrote:
> We need to mention ThinLTO in the Release Notes, so thanks for initiating this. 
> We likely should go with a paragraph summarizing the improvements, I don't know though if there is anything else than 1) PGO and 2) Significant build-time and binary-size improvements with debug-info.
> 
Yes this is a good start. One of the big improvements from PGO is the integration with indirect call promotion. I would suggest something like:

Improvements to ThinLTO (-flto=thin):
- Integration with profile data. When available, profile data enables more accurate function importing decisions, as well as cross-module indirect call promotion.
- Significant build-time and binary-size improvements when compiling with debug info (-g).


https://reviews.llvm.org/D28746





More information about the cfe-commits mailing list