[PATCH] D54176: [PGO] clang part of change for context-sensitive PGO.

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 3 11:50:23 PST 2018


tejohnson added inline comments.


================
Comment at: lib/CodeGen/BackendUtil.cpp:1244
+  // Context sensitive profile.
+  if (CGOpts.hasProfileCSIRInstr()) {
+    Conf.RunCSIRInstr = true;
----------------
davidxl wrote:
> please add +tejohnson for reviewing thinlto related pieces.
This part looks fine (haven't examined the rest in detail). Please add a test. There are some existing ones for the ThinLTO backend via clang in test/CodeGen/thinlto-* that you can look at for examples.


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

https://reviews.llvm.org/D54176





More information about the llvm-commits mailing list