<div class="gmail_quote">Cool!</div><div class="gmail_quote"><br></div><div class="gmail_quote">On Tue, Dec 20, 2011 at 9:38 AM, Jason W Kim <span dir="ltr"><<a href="mailto:jason.w.kim.2009@gmail.com">jason.w.kim.2009@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 id=":654">First steps in ARM AsmParser support for .eabi_attribute and .arch<br>
(Both used for Linux gnueabi)<br>
No behavioral change yet (no tests need so far)</div></blockquote></div><br><div>Why no tests needed? We should have tests that just exercise the parsing, no? If not, I think it would be good to wait on commit until enough functionality is there to at least have a sanity test. Essentially, you should have enough tests here to ensure you didn't commit an 'abort()' or a divide by zero.</div>