<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 1 May 2014 17:12, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On 1 May 2014 16:58, Lubos Lunak <span dir="ltr"><<a href="mailto:l.lunak@centrum.cz" target="_blank">l.lunak@centrum.cz</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: llunak<br>
Date: Thu May  1 18:58:20 2014<br>
New Revision: 207814<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=207814&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=207814&view=rev</a><br>
Log:<br>
-fsyntax-only for a test<br>
<br>
<br>
Modified:<br>
    cfe/trunk/test/Frontend/rewrite-includes-messages.c<br>
<br>
Modified: cfe/trunk/test/Frontend/rewrite-includes-messages.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Frontend/rewrite-includes-messages.c?rev=207814&r1=207813&r2=207814&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Frontend/rewrite-includes-messages.c?rev=207814&r1=207813&r2=207814&view=diff</a><br>



==============================================================================<br>
--- cfe/trunk/test/Frontend/rewrite-includes-messages.c (original)<br>
+++ cfe/trunk/test/Frontend/rewrite-includes-messages.c Thu May  1 18:58:20 2014<br>
@@ -1,5 +1,5 @@<br>
 // RUN: %clang_cc1 -E -frewrite-includes %s -I%S/Inputs/ | %clang_cc1 -Wall -fsyntax-only -Wunused-macros -x c - 2>&1 > %t.1<br>
-// RUN: %clang_cc1 -I%S/Inputs/ -Wall -Wunused-macros %s 2>&1 > %t.2<br>
+// RUN: %clang_cc1 -I%S/Inputs/ -Wall -Wunused-macros -fsyntax-only %s 2>&1 > %t.2<br></blockquote><div><br></div></div><div>Huh? This test is running clang -cc1 which defaults to only syntax checking unless something else is specified. -fsyntax-only isn't a -cc1 flag, it's a clang flag.</div>


<div><br></div><div>Please revert.</div></div></div></div></blockquote><div><br></div><div>Heh, I just saw the discussion on r207808. I understand what happened.</div><div><br></div><div>%clang has its own set of flags and reads them, then fork+exec's %clang_cc1 with a different set of flags. The suggestion of using -fsyntax-only and updating to %clang_cc1 were mutually incompatible, just using %clang_cc1 was the right thing here.</div>

<div><br></div><div>Nick</div><div><br></div></div></div></div>