<div dir="ltr">This might be easier to review on Phabricator.<br><br>There's some small things like 'else after return' in a few places. Use of standard containers where some of LLVM's custom containers might be better suited, etc.<br>
<br>Also - I suspect this would be easier to review if it were a little more incremental - would it be possible to, for example, split out the ranges support into a follow-up patch? (and/or any other parts of this change so that it'd be smaller more targeted/reviewable chunks)</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 4, 2014 at 7:10 AM, Oliver Stannard <span dir="ltr"><<a href="mailto:oliver.stannard@arm.com" target="_blank">oliver.stannard@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Currently, when using llvm as an assembler, DWARF debug information is only<br>
generated for the .text section. This patch modifies this so that DWARF info<br>
is emitted for all executable sections.<br>
<span class="HOEnZb"><font color="#888888"><br>
Oliver</font></span><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></div>