xur marked an inline comment as done. xur added a comment. I'll sent the revised patch based on the comments shortly.
================ Comment at: lib/CodeGen/BackendUtil.cpp:435 @@ +434,3 @@ + if (CodeGenOpts.ProfileIRInstr) { + assert (!CodeGenOpts.ProfileInstrGenerate); + if (!CodeGenOpts.InstrProfileOutput.empty()) ---------------- davidxl wrote: > What is this asserting about? Option ProfileInstrGenerate is to be used in Clang instrumentation only. If I reuse this options, I would need to change all the references in clang. I update the comments in the coming patch. http://reviews.llvm.org/D15829 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits