On Wed, Dec 5, 2012 at 2:23 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Okay, lit just needs to reject this if it doesn't work. It's too easy to write it by accident.</blockquote><div><br></div><div>I've gone ahead and fixed lit.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">
On Dec 5, 2012, at 14:18 , Richard Smith <<a href="mailto:richard-llvm@metafoo.co.uk">richard-llvm@metafoo.co.uk</a>> wrote:<br>
<br>
> Author: rsmith<br>
> Date: Wed Dec  5 16:18:04 2012<br>
> New Revision: 169434<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=169434&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=169434&view=rev</a><br>
> Log:<br>
> Fix an egregiously broken test. This pattern doesn't work:<br>
><br>
> RUN: a<br>
> RUN: b || true<br>
><br>
> lit expands it to a && b || true, and the || true applies to both commands (thus ignoring failures in 'a')! This is PR10867 again.<br>
><br>
> Modified:<br>
>    cfe/trunk/test/Modules/build-fail-notes.m<br>
><br>
> Modified: cfe/trunk/test/Modules/build-fail-notes.m<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/build-fail-notes.m?rev=169434&r1=169433&r2=169434&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/build-fail-notes.m?rev=169434&r1=169433&r2=169434&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/test/Modules/build-fail-notes.m (original)<br>
> +++ cfe/trunk/test/Modules/build-fail-notes.m Wed Dec  5 16:18:04 2012<br>
> @@ -6,7 +6,7 @@<br>
> // CHECK: While building module 'DependsOnModule' imported from<br>
> // CHECK: While building module 'Module' imported from<br>
> // CHECK: error: expected ';' after top level declarator<br>
> -// CHECK: note: expanded from {{macro 'getModuleVersion'|here}}<br>
> +// CHECK: note: expanded from here<br>
> // CHECK: fatal error: could not build module 'Module'<br>
> // CHECK: fatal error: could not build module 'DependsOnModule'<br>
> // CHECK-NOT: error:<br>
> @@ -18,13 +18,13 @@<br>
> // CHECK-REDEF: In module 'Module' imported from<br>
> // CHECK-REDEF: Module.h:15:12: note: previous definition is here<br>
><br>
> -// RUN: %clang_cc1 -fmodule-cache-path %t -fmodules -F %S/Inputs -DgetModuleVersion="epic fail" -serialize-diagnostic-file %t.diag %s 2>&1 || true<br>
> -// RUN: c-index-test -read-diagnostics %t.diag 2>&1 | FileCheck -check-prefix=CHECK-SDIAG %s<br>
> +// RUN: not %clang_cc1 -fmodule-cache-path %t -fmodules -F %S/Inputs -DgetModuleVersion="epic fail" -serialize-diagnostic-file %t/tmp.diag %s 2>&1<br>
> +// RUN: c-index-test -read-diagnostics %t/tmp.diag 2>&1 | FileCheck -check-prefix=CHECK-SDIAG %s<br>
><br>
> // CHECK-SDIAG: Module.h:9:13: error: expected ';' after top level declarator<br>
> // CHECK-SDIAG: build-fail-notes.m:4:32: note: while building module 'DependsOnModule' imported from<br>
> // CHECK-SDIAG: DependsOnModule.h:1:10: note: while building module 'Module' imported from<br>
> -// CHECK-SDIAG: note: expanded from {{macro 'getModuleVersion'|here}}<br>
> +// CHECK-SDIAG: note: expanded from here<br>
> // CHECK-SDIAG: warning: umbrella header does not include header 'NotInModule.h' [-Wincomplete-umbrella]<br>
> // CHECK-SDIAG: DependsOnModule.h:1:10: fatal: could not build module 'Module'<br>
> // CHECK-SDIAG: build-fail-notes.m:4:32: note: while building module 'DependsOnModule' imported from<br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br>
</div></div></blockquote></div><br>