<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)<br>
<br>- David</div><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>