<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Mar 26, 2012, at 1:50 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Mon, Mar 26, 2012 at 1:46 AM, NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com">geek4civic@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Rewrite MSYS-incompatible tests not to use grep.</blockquote></div><br><div>Just thought I'd point out that removing the use of 'grep' from the test suite has very significant benefits outside of any one platform.</div>
<div><br></div><div>I regularly find regressions in LLVM where a 'not grep' pattern fails to match because the comments change in the output, leaving bugs and other problems undiagnosed. FileCheck-izing these tests is of significant long term maintenance value IMO.</div>
</blockquote><div><br></div><div>+1</div></div><br></body></html>