On Feb 7, 2013, at 10:57 AM, David Blaikie <[email protected]> wrote:
> On Thu, Feb 7, 2013 at 10:49 AM, Chad Rosier <[email protected]> wrote: >> >> 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. > > Yeah, no actually strong opinion either - not sure if it's cheaper for > the infrastructure to check the requirement & not run the test than to > just run it anyway... probably slightly cheaper to exclude it on > non-assert builds, but *shrug* Haha.. Ok, I'll just leave it as is. Thanks, David. >> >> 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
