<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 23, 2014 at 5:33 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" class="cremed">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Jun 23, 2014 at 2:29 PM, Diego Novillo <<a href="mailto:dnovillo@google.com" class="cremed">dnovillo@google.com</a>> wrote:<br>

><br>
><br>
><br>
> On Mon, Jun 23, 2014 at 4:31 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="cremed">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>> ================<br>
>> Comment at: include/llvm/IR/DIBuilder.h:112<br>
>> @@ +111,3 @@<br>
>> +    /// @param Kind     The kind of debug information to generate.<br>
>> +    /// @param EmitDebugInfo   A boolean flag which indicates whether<br>
>> debug<br>
>> +    ///                        information should be written to the final<br>
>> ----------------<br>
>> I wonder if there's a less vague name we could use for this parameter, but<br>
>> I haven't thought of a better one yet. (I imagine inverted would make more<br>
>> sense - true for "use this weird thing where we make metadata nodes but<br>
>> don't add the CUs to the list" - but still don't know what we'd name it)<br>
><br>
> Not sure what you consider vague about it. In my mind, this distinguishes<br>
> the generation of the debug info from its emission.  I'll gladly rename it<br>
> to something else if you can think of another way of naming this flow.<br>
<br>
</div></div>Sorry, to clarify - what I consider vague about it, or what goes<br>
through my head when I see it is "all this code is about generating<br>
debug info (or, alternatively: this metadata /is/ the debug info),<br>
what does it mean to not generate debug info when we're clearly in<br>
code generating debug info".<br></blockquote><div><br></div><div>Right. That's why it's named 'Emit'. To distinguish it from 'Generate'. We still generate it, we just throw it into the bit bucket at the end.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
It'll probably be more obvious if/when we make the changes that are a<br>
bit more explicit about this - just producing scopes/file names, then<br>
we won't be building unused compile_units and other such things</blockquote><div><br></div><div>Yeah. Could be. </div></div></div></div>