[PATCH] D94435: [SampleFDO] Add the support to split the function profiles with context into separate sections.

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 13 10:01:27 PST 2021


wmi added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:462
+  /// postlink phase, only profiles with context will be read.
+  bool IsThinLTOPostLink;
 
----------------
wmi wrote:
> hoy wrote:
> > I'm wondering if we can just use one field for the phase it's currently are at. We may also want to check it for fullLTO in the future.
> Good point. Will change it.
Sent out a NFC change in https://reviews.llvm.org/D94613 as a preparation for the change here.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94435/new/

https://reviews.llvm.org/D94435



More information about the llvm-commits mailing list