danielcdh added inline comments.

================
Comment at: lib/CodeGen/CGDebugInfo.cpp:2743-2745
   if (LinkageName == Name || (!CGM.getCodeGenOpts().EmitGcovArcs &&
                               !CGM.getCodeGenOpts().EmitGcovNotes &&
+                              !CGM.getCodeGenOpts().ProfileDebug &&
----------------
echristo wrote:
> Should we be encapsulating all of these for profile debug info? I.e. I think 
> coverage analysis is going to want the same things.
Do you mean that -fcoverage also implies -fprofile-debug?

I think the reason we introduce -fprofile-debug is that it has different 
requirments for debug info than coverage/sanitizer. E.g. we want to emit 
discriminator for -fprofile-debug, but not coverage/sanitizer.


https://reviews.llvm.org/D25435



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to