mehdi_amini added inline comments.
================ Comment at: lib/CodeGen/CodeGenModule.cpp:452 + if (CodeGenOpts.DebugInfoForProfiling) + getModule().addModuleFlag(llvm::Module::Warning, "DebugInfoForProfiling", 1); ---------------- danielcdh wrote: > mehdi_amini wrote: > > Why should we warn on mismatch? > In theory, we expect this to be the same across all modules. Otherwise when > we use this binary for profiling, we may get inaccurate profiles. Oh I thought about the "unset" case but we wouldn't warn in this case. Indeed we never even should have this flag with a zero value. https://reviews.llvm.org/D29205 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
