<div dir="ltr"><div>I don't test IRGen in these files - not sure why i had -emit-llvm to begin with (might be a hold over from a testing strategy i had used on windows that was checking printed output which I was asked to abandon). <br>
<br></div><div>Not sure if it's all right to not include any IRGen tests as part of this patch.  If someone has a strong opinion on this I can get around to IRGen tests eventually.<br><br><br></div><br></div><div class="gmail_extra">
<br clear="all"><div>Faisal Vali<br><br></div>
<br><br><div class="gmail_quote">On Fri, Sep 27, 2013 at 1:50 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
That does mean we're not testing IRGen for these tests, though. Is that all right?<br>
<br>
(The portable equivalent of "-o /dev/null" might also work. We have one somewhere in lit.)<br>
<br>
<br>
On Sep 27, 2013, at 9:45 , Faisal Vali <<a href="mailto:faisalv@yahoo.com">faisalv@yahoo.com</a>> wrote:<br>
<br>
> Author: faisalv<br>
> Date: Fri Sep 27 11:45:48 2013<br>
> New Revision: 191531<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=191531&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=191531&view=rev</a><br>
> Log:<br>
> Fix the test files by removing the unnecessary -emit-llvm flag (should address Matt Beaumont-Gay's concern regarding failure on a read-only filesystem)<br>
><br>
> Modified:<br>
>    cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/generic-lambda-unimplemented-1y.cpp<br>
>    cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/p5-generic-lambda-1y.cpp<br>
><br>
> Modified: cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/generic-lambda-unimplemented-1y.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/generic-lambda-unimplemented-1y.cpp?rev=191531&r1=191530&r2=191531&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/generic-lambda-unimplemented-1y.cpp?rev=191531&r1=191530&r2=191531&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/generic-lambda-unimplemented-1y.cpp (original)<br>
> +++ cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/generic-lambda-unimplemented-1y.cpp Fri Sep 27 11:45:48 2013<br>
> @@ -1,4 +1,4 @@<br>
> -// RUN: %clang_cc1 -fsyntax-only -std=c++1y %s -verify -emit-llvm<br>
> +// RUN: %clang_cc1 -fsyntax-only -std=c++1y %s -verify<br>
> namespace return_type_deduction_ok {<br>
>  auto l = [](auto a) ->auto { return a; }(2);<br>
>  auto l2 = [](auto a) ->decltype(auto) { return a; }(2);<br>
><br>
> Modified: cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/p5-generic-lambda-1y.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/p5-generic-lambda-1y.cpp?rev=191531&r1=191530&r2=191531&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/p5-generic-lambda-1y.cpp?rev=191531&r1=191530&r2=191531&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/p5-generic-lambda-1y.cpp (original)<br>
> +++ cfe/trunk/test/CXX/expr/expr.prim/expr.prim.lambda/p5-generic-lambda-1y.cpp Fri Sep 27 11:45:48 2013<br>
> @@ -1,4 +1,4 @@<br>
> -// RUN: %clang_cc1 -fsyntax-only -verify %s -std=c++1y -DCXX1Y -emit-llvm<br>
> +// RUN: %clang_cc1 -fsyntax-only -verify %s -std=c++1y -DCXX1Y<br>
><br>
> namespace test_factorial {<br>
><br>
> @@ -70,7 +70,6 @@ int test2() {<br>
>   {<br>
>     auto L = [](auto *a) {<br>
>                 return (*a)(a); }; //expected-error{{called object type 'double' is not a function}}<br>
> -    //l(&l);<br>
>     double d;<br>
>     L(&d); //expected-note{{in instantiation of}}<br>
>     auto M = [](auto b) { return b; };<br>
> @@ -79,7 +78,6 @@ int test2() {<br>
>   {<br>
>     auto L = [](auto *a) ->decltype (a->foo()) { //expected-note2{{candidate template ignored:}}<br>
>                 return (*a)(a); };<br>
> -    //l(&l);<br>
>     double d;<br>
>     L(&d); //expected-error{{no matching function for call}}<br>
>     auto M = [](auto b) { return b; };<br>
> @@ -118,8 +116,6 @@ namespace nested_lambdas {<br>
>                            return a + b;<br>
>                         };<br>
>               };<br>
> -   // auto M = L(3.14);<br>
> -   // return M('4');<br>
>   }<br>
>   auto get_lambda() {<br>
>     return [](auto a) {<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>
</blockquote></div><br></div>