[PATCH] D15829: [PGO] Clang Option that enables IR level PGO instrumentation

David Li via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 30 19:51:50 PST 2015


davidxl added a comment.

Should add a test case in test/Driver/instrprof-ld.c.


================
Comment at: lib/CodeGen/BackendUtil.cpp:435
@@ +434,3 @@
+  if (CodeGenOpts.ProfileIRInstr) {
+    assert (!CodeGenOpts.ProfileInstrGenerate);
+    if (!CodeGenOpts.InstrProfileOutput.empty())
----------------
What is this asserting about?

================
Comment at: lib/Frontend/CompilerInvocation.cpp:453
@@ +452,3 @@
+  Opts.ProfileIRInstr = Args.hasArg(OPT_fprofile_ir_instr);
+  if (!Opts.ProfileIRInstr)
+    Opts.ProfileInstrGenerate = Args.hasArg(OPT_fprofile_instr_generate) ||
----------------
Add a comment here.


http://reviews.llvm.org/D15829





More information about the cfe-commits mailing list