<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 20, 2015 at 2:45 PM, Adrian Prantl via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aprantl added inline comments.<br>
<span class=""><br>
================<br>
Comment at: include/clang/Frontend/CodeGenOptions.def:164-165<br>
@@ -163,1 +163,4 @@<br>
<br>
+CODEGENOPT(DebugTypeExtRefs, 1, 0) ///< Whether or not debug info should contain<br>
+                                   ///< external references to a PCH or module.<br>
+<br>
----------------<br>
</span><span class="">rsmith wrote:<br>
> Does this flag end up in the module hash? It presumably should do so.<br>
</span>It does not get hashed:<br>
<br>
  - If the module has no debug info, then this flag is meaningless, because it only affects the debug info output.<br>
  - If the module has debug info, the module format is "obj" (hence a different hash) and this flag must be turned on. I will make sure this is enforced via an assertion.<br>
<br>
[I don't see a point in having "obj" modules without debug info, If we want these, we'll need to add it to the hash.]<br></blockquote><div><br>I imagine this will eventually happen when we start putting bitcode and object code in modules (for inline function definitions, for example).<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="im HOEnZb"><br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D11958" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11958</a><br>
<br>
<br>
<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>