<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 15, 2016 at 3:25 PM, Keno Fischer 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">loladiro added a comment.<br>
<br>
The only cases I've found in the test cases that intentionally omit compile units from the .cu node are the AddDiscriminator tests. Are those the ones you are thinking about.</blockquote><div><br></div><div>It's possibel that that's where it's showing up, but discriminators aren't the things I was thinking of - we produce discriminators because we want them in the debug info (not sure why these tests are omitting <a href="http://llvm.dbg.cu">llvm.dbg.cu</a> at all).<br><br>There's some kind of backend diagnostic system - that's it "remarks" I think they were called in the end. For the optimizers to report things like "this failed to be inlined/vectorized/whatever". We use debug info to identify the location for the diagnostics, if I remember correctly, but we don't want just enabling remarks to turn on debug info emission, so we skip putting the debug info in <a href="http://llvm.dbg.cu">llvm.dbg.cu</a>.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> For now, I made them all `isDefinition: false`.</blockquote><div><br>That's a bit more hackish than the original hack - since these are clearly subprogram definitions.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> However, if this is something we want to support (creating debug info, but not codegening it), how about we just create a different global named metadata `llvm.dbg.cu.noemit` (bad name, please suggest something better), into which we pull all compile units we don't want to emit, but the Verifier could still look at it.<br></blockquote><div><br>Plausible - don't mind too much. Trying to rope Diego in so he can speak more specifically about the functionality and how it's tested (or not, as it would seem).<br><br>- Dave</div></div></div></div>