<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 6, 2017 at 6:38 PM Davide Italiano via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">davide requested changes to this revision.<br class="gmail_msg">
davide added a reviewer: davide.<br class="gmail_msg">
davide added a comment.<br class="gmail_msg">
This revision now requires changes to proceed.<br class="gmail_msg">
<br class="gmail_msg">
Can you add an unittest to make sure we handle correctly > 10 arguments?<br class="gmail_msg"></blockquote><div><br></div><div>It's sort of an interesting question, isn't it - with the new implementation, 10 arguments isn't special so should we bother testing it? (if we test it with 2, there's no reason to believe it'd be different for 10 or 20)<br><br>I don't really know the right answer to this sort of question - in some cases like this (it comes up semi-often - you refactor some code to address a bug, but the bug/test isn't really relevant to the new implementation) I've included the test, sometimes not.<br><br>Open to opinions/perspectives here, for sure.<br><br>- Dave</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
<br class="gmail_msg">
Repository:<br class="gmail_msg">
  rL LLVM<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D28429" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28429</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>