<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Done in r210699.<div><br></div><div>I did not see any similar tests, and my hope is that we will expand this one to include more checking, so I’d prefer to keep it here.</div><div><br><div><div>On Jun 11, 2014, at 1:17 PM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br>On 11/06/2014 22:49, David Blaikie wrote:<br><blockquote type="cite">I think you can drop the -o /dev/null now that it's -fsyntax-only.<br></blockquote><br>And the REQUIRES: x86-registered-target can be dropped too.<br><br>(PS. I'm pretty sure there's an existing test for basic -target functionality this RUN line could be added to.)<br><br>Alp.<br><br><blockquote type="cite"><br>On Wed, Jun 11, 2014 at 12:34 PM, Bob Wilson <<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>> wrote:<br><blockquote type="cite">Author: bwilson<br>Date: Wed Jun 11 14:34:14 2014<br>New Revision: 210693<br><br>URL: <a href="http://llvm.org/viewvc/llvm-project?rev=210693&view=rev">http://llvm.org/viewvc/llvm-project?rev=210693&view=rev</a><br>Log:<br>Run new test with -fsyntax-only.<br><br>Modified:<br> cfe/trunk/test/Misc/win32-macho.c<br><br>Modified: cfe/trunk/test/Misc/win32-macho.c<br>URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Misc/win32-macho.c?rev=210693&r1=210692&r2=210693&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Misc/win32-macho.c?rev=210693&r1=210692&r2=210693&view=diff</a><br>==============================================================================<br>--- cfe/trunk/test/Misc/win32-macho.c (original)<br>+++ cfe/trunk/test/Misc/win32-macho.c Wed Jun 11 14:34:14 2014<br>@@ -1,3 +1,3 @@<br> // Check that basic use of win32-macho targets works.<br> // REQUIRES: x86-registered-target<br>-// RUN: %clang -c -target x86_64-pc-win32-macho %s -o /dev/null<br>+// RUN: %clang -fsyntax-only -target x86_64-pc-win32-macho %s -o /dev/null<br><br><br>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote>_______________________________________________<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">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br></blockquote><br>--<span class="Apple-converted-space"> </span><br><a href="http://www.nuanti.com/">http://www.nuanti.com</a><br>the browser experts</div></blockquote></div><br></div></body></html>