[PATCH] D36351: [lld][ELF] Add profile guided section layout

[Michael Spencer via Phabricator via llvm-commits]

Bigcheese marked 47 inline comments as done.
Bigcheese added inline comments.


================
Comment at: ELF/CallGraphSort.cpp:32
+///
+/// It does so given a call graph profile by the following:
+/// * Build a weighted call graph from the profile
----------------
espindola wrote:
> Bigcheese wrote:
> > espindola wrote:
> > > The part about profile should be in a followup patch. For now this is given the weighted call graph.
> > This is talking about the for loop in `CallGraphSort()` where it builds the graph.
> Building a graph from a set of edges doesn't sound much, but it is OK to keep the comment. Just don't use the word profile as it makes it sound like lld is reading the direct output of perf or similar.
> 
I've made it clear it's talking about the call graph profile, not whatever profile that was generated from.


https://reviews.llvm.org/D36351