Test case only passes on non-asserting build

[Richard Smith]

On Thu, Jun 13, 2013 at 10:52 AM, Richard Barton <richard.barton at arm.com>wrote:

> Hi Richard
>
> Your changes to clang test
> CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp in r183872 mean that
> the
> test now only passes for a build of clang with asserts on.
>
> The failure output with a non-asserting build is:
>
>
> /home/ricbar01/llvm/tools/clang/test/CodeGenCXX/cxx0x-initializer-stdinitial
> izerlist.cpp:308:12: error: expected string not found in input
>  // CHECK: call void @_ZN5dtors1SC1Ev(%"struct.dtors::S"* %arrayinit.begin)
>            ^
> <stdin>:704:28: note: scanning from here
> define void @_ZN5dtors1fEv() #2 {
>                            ^
> <stdin>:708:2: note: possible intended match here
>  call void @_ZN5dtors1SC1Ev(%"struct.dtors::S"* %3)
>
> The label arrayinit.begin is not created for non-asserting build, it has
> just a numeric label.
>
> Could you make your changes also pass on a non-asserting build please?
>

Sorry, insufficiently cleaned up testcase. Fixed in r183924.

I'm rather surprised that no buildbots complained here. Do we really have
no buildbots building with NDEBUG?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130613/d8a989bd/attachment.html>