<div dir="ltr"><br><br><div class="gmail_quote">On Thu, Apr 16, 2015 at 9:26 AM Richard <<a href="mailto:legalize@xmission.com">legalize@xmission.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="http://reviews.llvm.org/D8149#157110" target="_blank">http://reviews.llvm.org/D8149#157110</a>, @klimek wrote:<br>
<br>
> Argh, sorry for missing it - please always feel free to ping earlier (especially my reviews you can ping 2x per week).<br>
>  Can you please add a unit test? Apart from that it looks good.<br>
<br>
<br>
I added a `FileCheck` test in the tools-extra repository (see <a href="http://reviews.llvm.org/D8150" target="_blank">http://reviews.llvm.org/D8150</a>).  Is that sufficient, or would you also like to see a unit test?<br></blockquote><div><br></div><div>I'd like to see a unit test - we often make changes to the matcher infrastructure assuming all matchers are at least unit tested.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D8149" target="_blank">http://reviews.llvm.org/D8149</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div></div>