<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 20, 2014 at 3:44 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><br></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

I'm not sure whether adding the value to this enum is the right<br>
option, since we'll never see it in the backend (since the attribute<br>
is added to the CU, which won't be used by LLVM whenever this<br>
enumerator is used), but it does fit with the other values here.<br></blockquote><div><br></div><div>Yeah, the new value in the backend's enum seems a bit contrived. It is used exactly once in the DIBuilder. I had initially opted to make it an extra flag for DIBuilder::createCompileUnit, but this seemed conceptually cleaner. One thing that I had in mind was that we may want to do something with it later. </div>
<div><br></div><div>For instance, it may be that LocTrackingOnly can be used to not do some bookkeeping or other activities that we currently do for the other debug modes. I admit it seems a bit contrived now. In </div><div>
 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Eric - what do you reckon? Use a separate flag just for DIBuilder to<br>
know that it should omit the <a href="http://dbg.cu" target="_blank" class="cremed">dbg.cu</a> node, or add it to this existing<br>
flag even though it'll never end up in an actually useful CU metadata<br>
node (unlike the other two values)?<br></blockquote><div><br></div><div>Well, it does end up in a metadata node, we stash it as an operand to the CUNode we create in DIBuilder::createCompileUnit. That's the only reason why I thought it may be useful later.  If you don't think that will be the case, I can just add a boolean flag to the function.</div>
<div><br></div><div><br></div><div><br></div><div>Diego.</div></div></div></div>