<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 3, 2013 at 10:06 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The member hack was this:<br>
<br>
// In compatibility mode with older gdbs we put the linkage name on both<br>
// the TAG_variable DIE and on the TAG_member DIE.<br>
<br>
and without it there was a bug in that gdb to where it couldn't<br>
find/resolve the member.<br>
<br>
I'd like to keep the test and only xfail it on darwin since it should<br>
pass with a modern gdb. Ideally, I think, for darwin the test harness<br>
should be modified to run lldb and the tests updated to deal with any<br>
differences.<br></blockquote><div><br></div><div>Thanks for the details - makes sense.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-eric<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Tue, Sep 3, 2013 at 10:01 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
> Eric - mind adding some more detail on what this 'member hack' is?<br>
><br>
> and/or should we just delete these tests?<br>
><br>
><br>
> On Tue, Sep 3, 2013 at 9:44 PM, Michael Gottesman <<a href="mailto:mgottesman@apple.com">mgottesman@apple.com</a>><br>
> wrote:<br>
>><br>
>> Author: mgottesman<br>
>> Date: Tue Sep  3 23:44:05 2013<br>
>> New Revision: 189915<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=189915&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=189915&view=rev</a><br>
>> Log:<br>
>> XFAILed the static-member.cpp test since we no longer support emitting<br>
>> debug info that support what Eric calls "the member hack".<br>
>><br>
>> Modified:<br>
>>     debuginfo-tests/trunk/static-member.cpp<br>
>><br>
>> Modified: debuginfo-tests/trunk/static-member.cpp<br>
>> URL:<br>
>> <a href="http://llvm.org/viewvc/llvm-project/debuginfo-tests/trunk/static-member.cpp?rev=189915&r1=189914&r2=189915&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/debuginfo-tests/trunk/static-member.cpp?rev=189915&r1=189914&r2=189915&view=diff</a><br>

>><br>
>> ==============================================================================<br>
>> --- debuginfo-tests/trunk/static-member.cpp (original)<br>
>> +++ debuginfo-tests/trunk/static-member.cpp Tue Sep  3 23:44:05 2013<br>
>> @@ -1,6 +1,7 @@<br>
>>  // RUN: %clangxx -O0 -g %s -o %t -c<br>
>>  // RUN: %clangxx %t -o %t.out<br>
>>  // RUN: %test_debuginfo %s %t.out<br>
>> +// XFAIL: *<br>
>><br>
>>  // DEBUGGER: delete breakpoints<br>
>>  // DEBUGGER: break static-member.cpp:33<br>
>><br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
</div></div></blockquote></div><br></div></div>