<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Thanks for the review.  <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Looks good with one little nit, below. Do you need me to commit this<br>
for you?<br></blockquote><div><br></div><div>I should be fine to commit myself, thanks! </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span><br>
> -// %clang -E -dependency-file bla -MT %t -MP -o %t -x c /dev/null<br>
> +// RUN: not %clang_cc1 -E -dependency-file bla -MT %t -MP -o %t -x c /dev/null 2>&1 | FileCheck %s<br>
<br>
</span>This could use -verify instead of FileCheck.<span><br>
</span></blockquote><div><br></div><div>I previously tried this approach, but ran into an issue that -verify doesn't seem to work in this case with both -E and -o on the command line.  I could get rid of one or both of them, but then I get no errors at all so I'd need //expected-no-diagnostics.  However, without any visibility of the original bug report I'm assuming that these are both prerequisites for the original crash this was supposed to test.   Unfortunately, my attempts to go and build a revision prior to the bugfix (April 2011) are met with failure so I can't try and reconstruct a test from that.</div><div><br></div><div>Bearing this in mind, it seemed prudent to just use FileCheck instead and try to keep the test as unchanged as possible.  Sorry, I should have explained this in my original review summary!<br></div><div><br></div><div>Can you think of anything else I can try, or should I commit as-is?</div><div><br></div><div>Thanks!</div><div><br></div><div>-Greg</div></div></div></div>