<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 13, 2016 at 11:07 AM, Duncan P. N. Exon Smith via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-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"><span class=""><br>
> On 2016-Apr-13, at 10:53, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
> dblaikie accepted this revision.<br>
> dblaikie added a comment.<br>
> This revision is now accepted and ready to land.<br>
><br>
> Generally looks good - bunch of cleanup sort of stuff & a few casual questions, none of which I think would block this being committed.<br>
><br>
><br>
><br>
> ================<br>
</span><span class="">> Comment at: lib/Bitcode/Writer/BitcodeWriter.cpp:1101<br>
> @@ -1100,3 +1100,3 @@<br>
>   Record.push_back(VE.getMetadataOrNullID(N->getRetainedTypes().get()));<br>
> -  Record.push_back(VE.getMetadataOrNullID(N->getSubprograms().get()));<br>
> +  Record.push_back(/* subprograms */ 0);<br>
>   Record.push_back(VE.getMetadataOrNullID(N->getGlobalVariables().get()));<br>
> ----------------<br>
> Not worth changing the record layout to have one fewer records?<br>
<br>
</span>Adrian chatted about this with me before sending out the patch.  I was<br>
skeptical at first, but he convinced me it's okay as is.  Here's my<br>
current thinking:<br>
<br>
  - It's as easy to change later as to change now.<br>
<br>
  - There are relatively few DICompileUnit records, so this has a<br>
    negligible effect on the size of the bitcode on disk.<br>
<br>
  - Changing the record layout complicates the logic on the reader<br>
    side, which is a non-negligible maintenance burden.<br></blockquote><div><br></div><div>Yep, figured something along those lines but didn't hurt to ask/have it written out explicitly. If we end up doing this to other lists (I could imagine doing this to the global variables list too, for the same reasons, right?) then we can do a batch cleanup at some point.<br><br>- Dave</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>