<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">good point! <span style="font-family: Menlo; font-size: 14px;" class="">r225399.</span><div class="">-- adrian<font face="Menlo" class=""><span style="font-size: 14px;" class=""><br class=""></span></font><div class=""><font face="Menlo" class=""><span style="font-size: 14px;" class=""><br class=""></span></font><div style=""><blockquote type="cite" class=""><div class="">On Jan 7, 2015, at 1:28 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" class="">echristo@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">Feel like updating the comment to describe what's happening?<br class=""><br class="">Thanks!<div class=""><br class=""></div><div class="">-eric</div><br class=""><div class="gmail_quote">On Wed Jan 07 2015 at 12:52:25 PM Adrian Prantl <<a href="mailto:aprantl@apple.com" class="">aprantl@apple.com</a>> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: adrian<br class="">
Date: Wed Jan 7 14:48:58 2015<br class="">
New Revision: 225378<br class="">
<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=225378&view=rev" target="_blank" class="">http://llvm.org/viewvc/llvm-<u class=""></u>project?rev=225378&view=rev</a><br class="">
Log:<br class="">
Debug info: Allow aggregate types to be described by constants.<br class="">
<br class="">
Added:<br class="">
llvm/trunk/test/DebugInfo/X86/<u class=""></u>constant-aggregate.ll<br class="">
Modified:<br class="">
llvm/trunk/lib/CodeGen/<u class=""></u>AsmPrinter/DwarfUnit.cpp<br class="">
<br class="">
Modified: llvm/trunk/lib/CodeGen/<u class=""></u>AsmPrinter/DwarfUnit.cpp<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfUnit.cpp?rev=225378&r1=225377&r2=225378&view=diff" target="_blank" class="">http://llvm.org/viewvc/llvm-<u class=""></u>project/llvm/trunk/lib/<u class=""></u>CodeGen/AsmPrinter/DwarfUnit.<u class=""></u>cpp?rev=225378&r1=225377&r2=<u class=""></u>225378&view=diff</a><br class="">
==============================<u class=""></u>==============================<u class=""></u>==================<br class="">
--- llvm/trunk/lib/CodeGen/<u class=""></u>AsmPrinter/DwarfUnit.cpp (original)<br class="">
+++ llvm/trunk/lib/CodeGen/<u class=""></u>AsmPrinter/DwarfUnit.cpp Wed Jan 7 14:48:58 2015<br class="">
@@ -630,10 +630,13 @@ static bool isUnsignedDIType(DwarfDebug<br class="">
// FIXME: reference and rvalue_reference /probably/ shouldn't be allowed<br class="">
// here, but accept them for now due to a bug in SROA producing bogus<br class="">
// dbg.values.<br class="">
- if (T == dwarf::DW_TAG_pointer_type ||<br class="">
+ if (T == dwarf::DW_TAG_array_type ||<br class="">
+ T == dwarf::DW_TAG_class_type ||<br class="">
+ T == dwarf::DW_TAG_pointer_type ||<br class="">
T == dwarf::DW_TAG_ptr_to_member_<u class=""></u>type ||<br class="">
T == dwarf::DW_TAG_reference_type ||<br class="">
- T == dwarf::DW_TAG_rvalue_<u class=""></u>reference_type)<br class="">
+ T == dwarf::DW_TAG_rvalue_<u class=""></u>reference_type ||<br class="">
+ T == dwarf::DW_TAG_structure_type)<br class="">
return true;<br class="">
assert(T == dwarf::DW_TAG_typedef || T == dwarf::DW_TAG_const_type ||<br class="">
T == dwarf::DW_TAG_volatile_type ||<br class="">
<br class="">
Added: llvm/trunk/test/DebugInfo/X86/<u class=""></u>constant-aggregate.ll<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/X86/constant-aggregate.ll?rev=225378&view=auto" target="_blank" class="">http://llvm.org/viewvc/llvm-<u class=""></u>project/llvm/trunk/test/<u class=""></u>DebugInfo/X86/constant-<u class=""></u>aggregate.ll?rev=225378&view=<u class=""></u>auto</a><br class="">
==============================<u class=""></u>==============================<u class=""></u>==================<br class="">
--- llvm/trunk/test/DebugInfo/X86/<u class=""></u>constant-aggregate.ll (added)<br class="">
+++ llvm/trunk/test/DebugInfo/X86/<u class=""></u>constant-aggregate.ll Wed Jan 7 14:48:58 2015<br class="">
@@ -0,0 +1,118 @@<br class="">
+; RUN: llc %s -filetype=obj -o %t.o<br class="">
+; RUN: llvm-dwarfdump -debug-dump=info %t.o | FileCheck %s<br class="">
+; Test emitting a constant for an aggregate type.<br class="">
+;<br class="">
+; clang -S -O1 -emit-llvm<br class="">
+;<br class="">
+; typedef struct { unsigned i; } S;<br class="">
+;<br class="">
+; unsigned foo(S s) {<br class="">
+; s.i = 1;<br class="">
+; return s.i;<br class="">
+; }<br class="">
+;<br class="">
+; class C { public: unsigned i; };<br class="">
+;<br class="">
+; unsigned foo(C c) {<br class="">
+; c.i = 2;<br class="">
+; return c.i;<br class="">
+; }<br class="">
+;<br class="">
+; unsigned bar() {<br class="">
+; int a[1] = { 3 };<br class="">
+; return a[0];<br class="">
+; }<br class="">
+;<br class="">
+; CHECK: DW_TAG_formal_parameter<br class="">
+; CHECK-NEXT: DW_AT_const_value [DW_FORM_udata] (1)<br class="">
+; CHECK-NEXT: DW_AT_name {{.*}} "s"<br class="">
+;<br class="">
+; CHECK: DW_TAG_formal_parameter<br class="">
+; CHECK-NEXT: DW_AT_const_value [DW_FORM_udata] (2)<br class="">
+; CHECK-NEXT: DW_AT_name {{.*}} "c"<br class="">
+;<br class="">
+; CHECK: DW_TAG_variable<br class="">
+; CHECK-NEXT: DW_AT_const_value [DW_FORM_udata] (3)<br class="">
+; CHECK-NEXT: DW_AT_name {{.*}} "a"<br class="">
+<br class="">
+; ModuleID = 'sroasplit-4.cpp'<br class="">
+target datalayout = "e-m:o-i64:64-f80:128-n8:16:<u class=""></u>32:64-S128"<br class="">
+target triple = "x86_64-apple-macosx10.10.0"<br class="">
+<br class="">
+; Function Attrs: nounwind readnone ssp uwtable<br class="">
+define i32 @_Z3foo1S(i32 %s.coerce) #0 {<br class="">
+entry:<br class="">
+ tail call void @llvm.dbg.value(metadata i32 %s.coerce, i64 0, metadata !18, metadata !37), !dbg !38<br class="">
+ tail call void @llvm.dbg.value(metadata i32 1, i64 0, metadata !18, metadata !37), !dbg !38<br class="">
+ ret i32 1, !dbg !39<br class="">
+}<br class="">
+<br class="">
+; Function Attrs: nounwind readnone ssp uwtable<br class="">
+define i32 @_Z3foo1C(i32 %c.coerce) #0 {<br class="">
+entry:<br class="">
+ tail call void @llvm.dbg.value(metadata i32 %c.coerce, i64 0, metadata !23, metadata !37), !dbg !40<br class="">
+ tail call void @llvm.dbg.value(metadata i32 2, i64 0, metadata !23, metadata !37), !dbg !40<br class="">
+ ret i32 2, !dbg !41<br class="">
+}<br class="">
+<br class="">
+; Function Attrs: nounwind readnone ssp uwtable<br class="">
+define i32 @_Z3barv() #0 {<br class="">
+entry:<br class="">
+ tail call void @llvm.dbg.value(metadata i32 3, i64 0, metadata !28, metadata !37), !dbg !42<br class="">
+ ret i32 3, !dbg !43<br class="">
+}<br class="">
+<br class="">
+; Function Attrs: nounwind readnone<br class="">
+declare void @llvm.dbg.value(metadata, i64, metadata, metadata) #1<br class="">
+<br class="">
+attributes #0 = { nounwind readnone ssp uwtable }<br class="">
+attributes #1 = { nounwind readnone }<br class="">
+<br class="">
+!<a href="http://llvm.dbg.cu/" target="_blank" class="">llvm.dbg.cu</a> = !{!0}<br class="">
+!llvm.module.flags = !{!33, !34, !35}<br class="">
+!llvm.ident = !{!36}<br class="">
+<br class="">
+!0 = !{!"0x11\004\00clang version 3.6.0 (trunk 225364) (llvm/trunk 225366)\001\00\000\00\001", !1, !2, !3, !11, !2, !2} ; [ DW_TAG_compile_unit ] [/sroasplit-4.cpp] [DW_LANG_C_plus_plus]<br class="">
+!1 = !{!"sroasplit-4.cpp", !""}<br class="">
+!2 = !{}<br class="">
+!3 = !{!4, !8}<br class="">
+!4 = !{!"0x13\00\001\0032\0032\000\<u class=""></u>000\000", !1, null, null, !5, null, null, !"_ZTS1S"} ; [ DW_TAG_structure_type ] [line 1, size 32, align 32, offset 0] [def] [from ]<br class="">
+!5 = !{!6}<br class="">
+!6 = !{!"0xd\00i\001\0032\0032\000\<u class=""></u>000", !1, !"_ZTS1S", !7} ; [ DW_TAG_member ] [i] [line 1, size 32, align 32, offset 0] [from unsigned int]<br class="">
+!7 = !{!"0x24\00unsigned int\000\0032\0032\000\000\007"<u class=""></u>, null, null} ; [ DW_TAG_base_type ] [unsigned int] [line 0, size 32, align 32, offset 0, enc DW_ATE_unsigned]<br class="">
+!8 = !{!"0x2\00C\008\0032\0032\000\<u class=""></u>000\000", !1, null, null, !9, null, null, !"_ZTS1C"} ; [ DW_TAG_class_type ] [C] [line 8, size 32, align 32, offset 0] [def] [from ]<br class="">
+!9 = !{!10}<br class="">
+!10 = !{!"0xd\00i\008\0032\0032\000\<u class=""></u>003", !1, !"_ZTS1C", !7} ; [ DW_TAG_member ] [i] [line 8, size 32, align 32, offset 0] [public] [from unsigned int]<br class="">
+!11 = !{!12, !19, !24}<br class="">
+!12 = !{!"0x2e\00foo\00foo\00_<u class=""></u>Z3foo1S\003\000\001\000\000\<u class=""></u>00256\001\003", !1, !13, !14, null, i32 (i32)* @_Z3foo1S, null, null, !17} ; [ DW_TAG_subprogram ] [line 3] [def] [foo]<br class="">
+!13 = !{!"0x29", !1} ; [ DW_TAG_file_type ] [/sroasplit-4.cpp]<br class="">
+!14 = !{!"0x15\00\000\000\000\000\<u class=""></u>000\000", null, null, null, !15, null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align 0, offset 0] [from ]<br class="">
+!15 = !{!7, !16}<br class="">
+!16 = !{!"0x16\00S\001\000\000\000\<u class=""></u>000", !1, null, !"_ZTS1S"} ; [ DW_TAG_typedef ] [S] [line 1, size 0, align 0, offset 0] [from _ZTS1S]<br class="">
+!17 = !{!18}<br class="">
+!18 = !{!"0x101\00s\0016777219\000", !12, !13, !16} ; [ DW_TAG_arg_variable ] [s] [line 3]<br class="">
+!19 = !{!"0x2e\00foo\00foo\00_<u class=""></u>Z3foo1C\0010\000\001\000\000\<u class=""></u>00256\001\0010", !1, !13, !20, null, i32 (i32)* @_Z3foo1C, null, null, !22} ; [ DW_TAG_subprogram ] [line 10] [def] [foo]<br class="">
+!20 = !{!"0x15\00\000\000\000\000\<u class=""></u>000\000", null, null, null, !21, null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align 0, offset 0] [from ]<br class="">
+!21 = !{!7, !"_ZTS1C"}<br class="">
+!22 = !{!23}<br class="">
+!23 = !{!"0x101\00c\0016777226\000", !19, !13, !"_ZTS1C"} ; [ DW_TAG_arg_variable ] [c] [line 10]<br class="">
+!24 = !{!"0x2e\00bar\00bar\00_<u class=""></u>Z3barv\0015\000\001\000\000\<u class=""></u>00256\001\0015", !1, !13, !25, null, i32 ()* @_Z3barv, null, null, !27} ; [ DW_TAG_subprogram ] [line 15] [def] [bar]<br class="">
+!25 = !{!"0x15\00\000\000\000\000\<u class=""></u>000\000", null, null, null, !26, null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align 0, offset 0] [from ]<br class="">
+!26 = !{!7}<br class="">
+!27 = !{!28}<br class="">
+!28 = !{!"0x100\00a\0016\000", !24, !13, !29} ; [ DW_TAG_auto_variable ] [a] [line 16]<br class="">
+!29 = !{!"0x1\00\000\0032\0032\000\<u class=""></u>000\000", null, null, !30, !31, null, null, null} ; [ DW_TAG_array_type ] [line 0, size 32, align 32, offset 0] [from int]<br class="">
+!30 = !{!"0x24\00int\000\0032\0032\<u class=""></u>000\000\005", null, null} ; [ DW_TAG_base_type ] [int] [line 0, size 32, align 32, offset 0, enc DW_ATE_signed]<br class="">
+!31 = !{!32}<br class="">
+!32 = !{!"0x21\000\001"} ; [ DW_TAG_subrange_type ] [0, 0]<br class="">
+!33 = !{i32 2, !"Dwarf Version", i32 2}<br class="">
+!34 = !{i32 2, !"Debug Info Version", i32 2}<br class="">
+!35 = !{i32 1, !"PIC Level", i32 2}<br class="">
+!36 = !{!"clang version 3.6.0 (trunk 225364) (llvm/trunk 225366)"}<br class="">
+!37 = !{!"0x102"} ; [ DW_TAG_expression ]<br class="">
+!38 = !{i32 3, i32 16, !12, null}<br class="">
+!39 = !{i32 5, i32 3, !12, null}<br class="">
+!40 = !{i32 10, i32 16, !19, null}<br class="">
+!41 = !{i32 12, i32 3, !19, null}<br class="">
+!42 = !{i32 16, i32 6, !24, null}<br class="">
+!43 = !{i32 17, i32 3, !24, null}<br class="">
<br class="">
<br class="">
______________________________<u class=""></u>_________________<br class="">
llvm-commits mailing list<br class="">
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank" class="">llvm-commits@cs.uiuc.edu</a><br class="">
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="">http://lists.cs.uiuc.edu/<u class=""></u>mailman/listinfo/llvm-commits</a><br class="">
</blockquote></div>
</div></blockquote></div><br class=""></div></div></body></html>