[PATCH] D54175: [PGO] context sensitive PGO

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: lib/LTO/LTOBackend.cpp:152
 
 static void runNewPMPasses(Config &Conf, Module &Mod, TargetMachine *TM,
                            unsigned OptLevel, bool IsThinLTO,
----------------
Presumably need a corresponding change in runOldPMPasses?

You'll also want to have a way to test via llvm-lto2. E.g. llvm-lto2 has an option for passing a sample profile.


================
Comment at: tools/gold/gold-plugin.cpp:898
+    Conf.CSIRProfile = options::cs_profile_path;
+  Conf.RunCSIRInstr = options::cs_pgo_gen;
+
----------------
Needs tests

Also, support should be added to lld too - is there a separate patch for that?


================
Comment at: tools/llvm-profdata/llvm-profdata.cpp:231
+  // outs() << "Loadinput for " << Input.Filename << " CS=" << HasCSIRProfile <<
+  // " IR=" << IsIRProfile << "\n";
+  if (WC->Writer.setIsIRLevelProfile(IsIRProfile, HasCSIRProfile)) {
----------------
leftover debug output


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

https://reviews.llvm.org/D54175