On Thu, Jun 13, 2013 at 10:52 AM, Richard Barton <span dir="ltr"><<a href="mailto:richard.barton@arm.com" target="_blank">richard.barton@arm.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Richard<br>
<br>
Your changes to clang test<br>
CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp in r183872 mean that the<br>
test now only passes for a build of clang with asserts on.<br>
<br>
The failure output with a non-asserting build is:<br>
<br>
/home/ricbar01/llvm/tools/clang/test/CodeGenCXX/cxx0x-initializer-stdinitial<br>
izerlist.cpp:308:12: error: expected string not found in input<br>
 // CHECK: call void @_ZN5dtors1SC1Ev(%"struct.dtors::S"* %arrayinit.begin)<br>
           ^<br>
<stdin>:704:28: note: scanning from here<br>
define void @_ZN5dtors1fEv() #2 {<br>
                           ^<br>
<stdin>:708:2: note: possible intended match here<br>
 call void @_ZN5dtors1SC1Ev(%"struct.dtors::S"* %3)<br>
<br>
The label arrayinit.begin is not created for non-asserting build, it has<br>
just a numeric label.<br>
<br>
Could you make your changes also pass on a non-asserting build please?<br></blockquote><div><br></div><div>Sorry, insufficiently cleaned up testcase. Fixed in r183924.</div><div><br></div><div>I'm rather surprised that no buildbots complained here. Do we really have no buildbots building with NDEBUG?</div>
</div>