[cfe-commits] r172144 - in /cfe/trunk: lib/CodeGen/CGExprConstant.cpp test/CodeGenCXX/cxx11-trivial-initializer-struct.cpp

[Nico Weber]

On Thu, Jan 10, 2013 at 5:08 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Fri, Jan 11, 2013 at 1:28 AM, Fariborz Jahanian <fjahanian at apple.com> wrote:
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/cxx11-trivial-initializer-struct.cpp?rev=172144&view=auto
>> ==============================================================================
>> --- cfe/trunk/test/CodeGenCXX/cxx11-trivial-initializer-struct.cpp (added)
>> +++ cfe/trunk/test/CodeGenCXX/cxx11-trivial-initializer-struct.cpp Thu Jan 10 17:28:43 2013
>> @@ -0,0 +1,18 @@
>> +// RUN: %clang_cc1 -S -emit-llvm -o %t.ll %s -triple x86_64-apple-darwin10
>> +// RUN: %clang_cc1 -std=c++11 -S -emit-llvm -o %t-c++11.ll %s -triple x86_64-apple-darwin10
>> +// RUN: diff %t.ll  %t-c++11.ll
>
> I think it would be better to specify -std=c++98 explicitly in the
> first line.  Once c++11 becomes the default, this test will be a
> no-op.  (And, it already is on Windows.)

(I don't disagree, but I believe -std=c++11 is added by the driver on
windows, and this test uses clang_cc1.)

>
> Dmitri
>
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits