r174640 - Testcase for r174477.

Chad Rosier mcrosier at apple.com
Thu Feb 7 10:49:16 PST 2013


On Feb 7, 2013, at 10:39 AM, David Blaikie <dblaikie at gmail.com> wrote:

> On Thu, Feb 7, 2013 at 10:32 AM, Chad Rosier <mcrosier at apple.com> 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
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list