<div class="gmail_quote">On Tue, Jun 5, 2012 at 11:01 AM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@apple.com" target="_blank">echristo@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On Jun 5, 2012, at 12:28 AM, Eric Christopher <<a href="mailto:echristo@apple.com">echristo@apple.com</a>> wrote:<br>
<br>
><br>
> On Jun 4, 2012, at 11:12 PM, John McCall wrote:<br>
><br>
>> On Jun 4, 2012, at 9:45 PM, John McCall wrote:<br>
>>> On Jun 4, 2012, at 5:15 PM, Eric Christopher wrote:<br>
>>>> Author: echristo<br>
>>>> Date: Mon Jun  4 19:15:06 2012<br>
>>>> New Revision: 157970<br>
>>>><br>
>>>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=157970&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=157970&view=rev</a><br>
>>>> Log:<br>
>>>> Only emit debug information for methods that are user defined, there's<br>
>>>> not much reason to emit for constructors and destructors that aren't<br>
>>>> user defined.<br>
>>>><br>
>>>> rdar://11593099<br>
>>><br>
>>> Eric, it looks like this is breaking one of the clang debug info tests:<br>
>>> Clang :: debuginfo-tests___forward-declare-class.cpp<br>
>>><br>
>>> <a href="http://smooshlab.apple.com:8013/builders/clang-x86_64-darwin10-nobootstrap-RA/builds/633" target="_blank">http://smooshlab.apple.com:8013/builders/clang-x86_64-darwin10-nobootstrap-RA/builds/633</a><br>

>><br>
>> I reverted in r157983.<br>
><br>
> Weird, thanks. I was trying to check the bots this evening but they'd frozen for a while. I'll check why this failed in a bit.<br>
<br>
</div>Awesome gdb bug:<br>
<br>
0x00000032:     TAG_class_type [4] *<br>
                 AT_name( "A" )<br>
                 AT_byte_size( 0x04 )<br>
                 AT_decl_file( "/Volumes/Data/sources/llvm/tools/clang/test/debuginfo/forward-declare-class.cpp" )<br>
                 AT_decl_line( 18 )<br>
<br>
0x0000003a:         TAG_member [5]<br>
                     AT_name( "MyData" )<br>
                     AT_type( {0x0000002b} ( int ) )<br>
                     AT_decl_file( "/Volumes/Data/sources/llvm/tools/clang/test/debuginfo/forward-declare-class.cpp" )<br>
                     AT_decl_line( 20 )<br>
                     AT_data_member_location( +0 )<br>
                     AT_accessibility( DW_ACCESS_public )<br>
<br>
0x00000049:         NULL<br>
<br>
yet gdb doesn't believe the tag:<br>
<br>
(gdb) ptype A<br>
type = struct A {<br>
    int MyData;<br>
}<br>
<br>
because there aren't any methods/constructors in there.<br>
<br>
lldb correctly identifies the type:<br>
<br>
(lldb) image lookup -t A<br>
1 match found in /Volumes/Data/builds/build-llvm/tools/clang/test/debuginfo/Output/forward-declare-class.cpp.tmp.o:<br>
id = {0x00000032}, name = "A", byte-size = 4, decl = forward-declare-class.cpp:18, clang_type = "class A {<br>
    int MyData;<br>
}"<br>
<br>
I'll fix the test to work around this and recommit. Sorry for the noise.<br></blockquote><div><br></div><div>Won't this break gdb for users then?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
-eric<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br>