<div dir="ltr">I also /really/ wish we could get this info from register allocation or something rather than recomputing it. CC'ing lhames here in case he's got ideas. Will pester him via other channels as well.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 30, 2014 at 8:28 AM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Quick gut-reaction: that class is big enough that it deserves its own file. (I realize you're just looking for a sanity check of the approach - and I'll try to express some opinion on that too, though I don't think I'm fully qualified to be the arbiter for this increased (though probably justified) complexity - I don't have nearly enough knowledge about the backend, location information, and other ways to approach this problem - I'll try to ask around)<span class="HOEnZb"><font color="#888888"><br>

<br>- David</font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 30, 2014 at 8:21 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">  clang-formatize the new code<br>
<br>
Hi dblaikie,<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2658" target="_blank">http://llvm-reviews.chandlerc.com/D2658</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D2658?vs=6773&id=6774#toc" target="_blank">http://llvm-reviews.chandlerc.com/D2658?vs=6773&id=6774#toc</a><br>
<br>
Files:<br>
  lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>