On Mon, Apr 27, 2009 at 9:45 AM, Jay Foad <span dir="ltr"><<a href="mailto:jay.foad@gmail.com">jay.foad@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">> The patch looks fine to me, does it pass regression tests etc?<br>
<br>
</div>Yes, I've just run a successful "make" in the test-suite module. It<br>
took hours! Is there a smaller set of regression tests I could run for<br>
changes like this in future?</blockquote><div><br>"make unittests" is a smaller set of tests, but given that it doesn't have particularly high coverage, it shouldn't be the only tests you run.<br><br>There are unittests for APInt, but they wouldn't cover a refactoring change such as this one.  Please feel free to add a unittest with your change.<br>
</div></div>