On Feb 7, 2013, at 10:39 AM, David Blaikie <[email protected]> wrote:

> On Thu, Feb 7, 2013 at 10:32 AM, Chad Rosier <[email protected]> wrote:
>> Author: mcrosier
>> Date: Thu Feb  7 12:32:25 2013
>> New Revision: 174640
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=174640&view=rev
>> Log:
>> Testcase for r174477.
>> 
>> Added:
>>    cfe/trunk/test/Sema/invalid-cast.cpp
>> 
>> Added: cfe/trunk/test/Sema/invalid-cast.cpp
>> URL: 
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/invalid-cast.cpp?rev=174640&view=auto
>> ==============================================================================
>> --- cfe/trunk/test/Sema/invalid-cast.cpp (added)
>> +++ cfe/trunk/test/Sema/invalid-cast.cpp Thu Feb  7 12:32:25 2013
>> @@ -0,0 +1,11 @@
>> +// RUN: %clang_cc1 -verify -fsyntax-only %s
>> +// expected-no-diagnostics
>> +// <rdar://problem/13153516> - This previously triggered an assertion 
>> failure.
> 
> Requires: Asserts ?

I saw similar test cases that were checking things that perviously asserted, 
but didn't require an assert build.  I don't think it is necessary, but I also 
don't object to adding it.

 Chad

> 
>> +template<class T>
>> +struct X {
>> + T array;
>> +};
>> +
>> +int foo(X<int[1]> x0) {
>> + return x0.array[17];
>> +}
>> 
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> [email protected]
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to