<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 3, 2014 at 12:04 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">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 dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">
<div class="">On Wed, Sep 3, 2014 at 11:58 AM, Frédéric Riss <span dir="ltr"><<a href="mailto:friss@apple.com" target="_blank">friss@apple.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 style="word-wrap:break-word">No problem, I’ll rebase the series.<div>
<br></div><div>Is the patch OK with that change? Or do you want to see a patch with a user first ?</div>
</div></blockquote><div><br></div></div><div>I think it'll be fine with that change. I can imagine what sort of users will eventually need this.</div></div></div></div></blockquote><div><br></div><div>+1</div><div>I also considered this change as usable at some point in future.</div>
<div>Also, could you please use <a href="http://llvm.org/docs/Phabricator.html">http://llvm.org/docs/Phabricator.html</a> for your patches? Commenting/watching the context is superior there. Thanks.</div><div><br></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"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<div class=""><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">
<div style="word-wrap:break-word"><div><div><div><br><div><blockquote type="cite"><div>On 03 Sep 2014, at 20:49, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div>

<br><div><div dir="ltr">Could we pass that by reference instead of pointer, perhaps? (I know there's lots of pointers in the exsting codebase (here and elsewhere), but there's at least some desire to move towards references where practical/reasonable (essentially when rebinding and nullness are not required))</div>


<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 3, 2014 at 11:45 AM, Frédéric Riss <span dir="ltr"><<a href="mailto:friss@apple.com" target="_blank">friss@apple.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">


First preparatory patch of the dwarfdump work. No feature or functional change, just introduce a DWARFContext member in the DWARFUnit that will be used later to make cross-section accesses or to store global state like dumping options.<br>



<br>
Ok to go in?<br>
<br>
<br> <br></blockquote></div><br></div>
</div></blockquote></div><br></div></div></div></div></blockquote></div></div><br></div></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div>
</div></div>