<div dir="ltr">In the patch comments I suggested adding a separate named metadata node to root compile units that you want in the IR, but not emitted into the binary. I assume that would work for you?<div><br></div><div>Duncan, do you like that approach?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 18, 2016 at 7:54 PM, Diego Novillo <span dir="ltr"><<a href="mailto:dnovillo@google.com" target="_blank">dnovillo@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Jan 15, 2016 at 11:24 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Diego: To include line/col info for backend diagnostics, we produced debug info but omitted the <a href="http://llvm.dbg.cu" target="_blank">llvm.dbg.cu</a> entry, right? So there are subprogram debug info descriptions that are not referenced from a CU in <a href="http://llvm.dbg.cu" target="_blank">llvm.dbg.cu</a>, yes?</div></div></div></div></blockquote><div><br></div></span><div>Yes, that and for sample PGO.  Omitting <a href="http://llvm.dbg.cu" target="_blank">llvm.dbg.cu</a> prevents codegen from emitting all that debug info to the final binary.</div></div></div></div>
</blockquote></div><br></div>