[llvm-commits] [llvm] r125794 - in /llvm/trunk: include/llvm/CodeGen/SelectionDAG.h lib/CodeGen/SelectionDAG/LegalizeDAG.cpp lib/CodeGen/SelectionDAG/SDNodeDbgValue.h lib/CodeGen/SelectionDAG/SelectionDAG.cpp lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp test/CodeGen/X86/dbg-value-inlined-parameter.ll
Devang Patel
dpatel at apple.com
Fri Feb 18 10:20:12 PST 2011
On Feb 17, 2011, at 10:05 PM, Cameron Zwarich wrote:
> The build has made it further, so it looks like this was the problem.
>
> Cameron
>
> On Feb 17, 2011, at 9:02 PM, Cameron Zwarich wrote:
>
>> I'm going to roll this out to help diagnose the failure on the llvm-gcc-i386-linux-selfhost bot. It is the only significant change in the blamed revision range. Sorry about that!
OK. no worries :)
Thanks,
-
Devang
>>
>> Cameron
>>
>> On Feb 17, 2011, at 3:33 PM, Devang Patel wrote:
>>
>>> Author: dpatel
>>> Date: Thu Feb 17 17:33:27 2011
>>> New Revision: 125794
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=125794&view=rev
>>> Log:
>>> Do not lose debug info of an inlined function argument even if the argument is only used through GEPs.
>>>
>>> Added:
>>> llvm/trunk/test/CodeGen/X86/dbg-value-inlined-parameter.ll
>>> Modified:
>>> llvm/trunk/include/llvm/CodeGen/SelectionDAG.h
>>> llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
>>> llvm/trunk/lib/CodeGen/SelectionDAG/SDNodeDbgValue.h
>>> llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
>>> llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
>>>
>>> Modified: llvm/trunk/include/llvm/CodeGen/SelectionDAG.h
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/SelectionDAG.h?rev=125794&r1=125793&r2=125794&view=diff
>>> ==============================================================================
>>> --- llvm/trunk/include/llvm/CodeGen/SelectionDAG.h (original)
>>> +++ llvm/trunk/include/llvm/CodeGen/SelectionDAG.h Thu Feb 17 17:33:27 2011
>>> @@ -100,10 +100,6 @@
>>> return DbgValMap[Node];
>>> }
>>>
>>> - void removeSDDbgValues(const SDNode *Node) {
>>> - DbgValMap.erase(Node);
>>> - }
>>> -
>>> typedef SmallVector<SDDbgValue*,32>::iterator DbgIterator;
>>> DbgIterator DbgBegin() { return DbgValues.begin(); }
>>> DbgIterator DbgEnd() { return DbgValues.end(); }
>>>
>>> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp?rev=125794&r1=125793&r2=125794&view=diff
>>> ==============================================================================
>>> --- llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp (original)
>>> +++ llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp Thu Feb 17 17:33:27 2011
>>> @@ -87,6 +87,9 @@
>>> // If someone requests legalization of the new node, return itself.
>>> if (From != To)
>>> LegalizedNodes.insert(std::make_pair(To, To));
>>> +
>>> + // Transfer SDDbgValues.
>>> + DAG.TransferDbgValues(From, To);
>>> }
>>>
>>> public:
>>>
>>> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SDNodeDbgValue.h
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SDNodeDbgValue.h?rev=125794&r1=125793&r2=125794&view=diff
>>> ==============================================================================
>>> --- llvm/trunk/lib/CodeGen/SelectionDAG/SDNodeDbgValue.h (original)
>>> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SDNodeDbgValue.h Thu Feb 17 17:33:27 2011
>>> @@ -83,14 +83,6 @@
>>> // Returns the SDNode* for a register ref
>>> SDNode *getSDNode() { assert (kind==SDNODE); return u.s.Node; }
>>>
>>> - // setSDNode - If underlying SDNode is replaced by another node then
>>> - // SelectionDAG can use this to transfer DbgValue.
>>> - void setSDNode(SDNode *N, unsigned R) {
>>> - assert (kind==SDNODE);
>>> - u.s.Node = N;
>>> - u.s.ResNo = R;
>>> - }
>>> -
>>> // Returns the ResNo for a register ref
>>> unsigned getResNo() { assert (kind==SDNODE); return u.s.ResNo; }
>>>
>>>
>>> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp?rev=125794&r1=125793&r2=125794&view=diff
>>> ==============================================================================
>>> --- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp (original)
>>> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp Thu Feb 17 17:33:27 2011
>>> @@ -5508,12 +5508,14 @@
>>> SDNode *FromNode = From.getNode();
>>> SDNode *ToNode = To.getNode();
>>> SmallVector<SDDbgValue*,2> &DVs = GetDbgValues(FromNode);
>>> - DbgInfo->removeSDDbgValues(FromNode);
>>> for (SmallVector<SDDbgValue *, 2>::iterator I = DVs.begin(), E = DVs.end();
>>> I != E; ++I) {
>>> - if ((*I)->getKind() == SDDbgValue::SDNODE) {
>>> - AddDbgValue(*I, ToNode, false);
>>> - (*I)->setSDNode(ToNode, To.getResNo());
>>> + SDDbgValue *Dbg = *I;
>>> + if (Dbg->getKind() == SDDbgValue::SDNODE) {
>>> + SDDbgValue *Clone = getDbgValue(Dbg->getMDPtr(), ToNode, To.getResNo(),
>>> + Dbg->getOffset(), Dbg->getDebugLoc(),
>>> + Dbg->getOrder());
>>> + AddDbgValue(Clone, ToNode, false);
>>> }
>>> }
>>> }
>>>
>>> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp?rev=125794&r1=125793&r2=125794&view=diff
>>> ==============================================================================
>>> --- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (original)
>>> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp Thu Feb 17 17:33:27 2011
>>> @@ -4287,7 +4287,7 @@
>>> N.getResNo(), Offset, dl, SDNodeOrder);
>>> DAG.AddDbgValue(SDV, N.getNode(), false);
>>> }
>>> - } else if (isa<PHINode>(V) && !V->use_empty() ) {
>>> + } else if (!V->use_empty() ) {
>>> // Do not call getValue(V) yet, as we don't want to generate code.
>>> // Remember it for later.
>>> DanglingDebugInfo DDI(&DI, dl, SDNodeOrder);
>>>
>>> Added: llvm/trunk/test/CodeGen/X86/dbg-value-inlined-parameter.ll
>>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/dbg-value-inlined-parameter.ll?rev=125794&view=auto
>>> ==============================================================================
>>> --- llvm/trunk/test/CodeGen/X86/dbg-value-inlined-parameter.ll (added)
>>> +++ llvm/trunk/test/CodeGen/X86/dbg-value-inlined-parameter.ll Thu Feb 17 17:33:27 2011
>>> @@ -0,0 +1,86 @@
>>> +; RUN: llc -mtriple=x86_64-apple-darwin < %s | FileCheck %s
>>> +
>>> +;CHECK: DW_TAG_inlined_subroutine
>>> +;CHECK-NEXT: DW_AT_abstract_origin
>>> +;CHECK-NEXT: DW_AT_low_pc
>>> +;CHECK-NEXT: DW_AT_high_pc
>>> +;CHECK-NEXT: DW_AT_call_file
>>> +;CHECK-NEXT: DW_AT_call_line
>>> +;CHECK-NEXT: DW_TAG_formal_parameter
>>> +;CHECK-NEXT: .ascii "sp" ## DW_AT_name
>>> +
>>> +%struct.S1 = type { float*, i32 }
>>> +
>>> + at p = common global %struct.S1 zeroinitializer, align 8
>>> +
>>> +define i32 @foo(%struct.S1* nocapture %sp, i32 %nums) nounwind optsize ssp {
>>> +entry:
>>> + tail call void @llvm.dbg.value(metadata !{%struct.S1* %sp}, i64 0, metadata !9), !dbg !20
>>> + tail call void @llvm.dbg.value(metadata !{i32 %nums}, i64 0, metadata !18), !dbg !21
>>> + %tmp2 = getelementptr inbounds %struct.S1* %sp, i64 0, i32 1, !dbg !22
>>> + store i32 %nums, i32* %tmp2, align 4, !dbg !22, !tbaa !24
>>> + %call = tail call float* @bar(i32 %nums) nounwind optsize, !dbg !27
>>> + %tmp5 = getelementptr inbounds %struct.S1* %sp, i64 0, i32 0, !dbg !27
>>> + store float* %call, float** %tmp5, align 8, !dbg !27, !tbaa !28
>>> + %cmp = icmp ne float* %call, null, !dbg !29
>>> + %cond = zext i1 %cmp to i32, !dbg !29
>>> + ret i32 %cond, !dbg !29
>>> +}
>>> +
>>> +declare float* @bar(i32) optsize
>>> +
>>> +define void @foobar() nounwind optsize ssp {
>>> +entry:
>>> + tail call void @llvm.dbg.value(metadata !30, i64 0, metadata !9) nounwind, !dbg !31
>>> + tail call void @llvm.dbg.value(metadata !34, i64 0, metadata !18) nounwind, !dbg !35
>>> + store i32 1, i32* getelementptr inbounds (%struct.S1* @p, i64 0, i32 1), align 8, !dbg !36, !tbaa !24
>>> + %call.i = tail call float* @bar(i32 1) nounwind optsize, !dbg !37
>>> + store float* %call.i, float** getelementptr inbounds (%struct.S1* @p, i64 0, i32 0), align 8, !dbg !37, !tbaa !28
>>> + ret void, !dbg !38
>>> +}
>>> +
>>> +declare void @llvm.dbg.value(metadata, i64, metadata) nounwind readnone
>>> +
>>> +!llvm.dbg.sp = !{!0, !6}
>>> +!llvm.dbg.lv.foo = !{!9, !18}
>>> +!llvm.dbg.gv = !{!19}
>>> +
>>> +!0 = metadata !{i32 589870, i32 0, metadata !1, metadata !"foo", metadata !"foo", metadata !"", metadata !1, i32 8, metadata !3, i1 false, i1 true, i32 0, i32 0, i32 0, i32 256, i1 true, i32 (%struct.S1*, i32)* @foo} ; [ DW_TAG_subprogram ]
>>> +!1 = metadata !{i32 589865, metadata !"nm2.c", metadata !"/private/tmp", metadata !2} ; [ DW_TAG_file_type ]
>>> +!2 = metadata !{i32 589841, i32 0, i32 12, metadata !"nm2.c", metadata !"/private/tmp", metadata !"clang version 2.9 (trunk 125693)", i1 true, i1 true, metadata !"", i32 0} ; [ DW_TAG_compile_unit ]
>>> +!3 = metadata !{i32 589845, metadata !1, metadata !"", metadata !1, i32 0, i64 0, i64 0, i32 0, i32 0, i32 0, metadata !4, i32 0, i32 0} ; [ DW_TAG_subroutine_type ]
>>> +!4 = metadata !{metadata !5}
>>> +!5 = metadata !{i32 589860, metadata !2, metadata !"int", null, i32 0, i64 32, i64 32, i64 0, i32 0, i32 5} ; [ DW_TAG_base_type ]
>>> +!6 = metadata !{i32 589870, i32 0, metadata !1, metadata !"foobar", metadata !"foobar", metadata !"", metadata !1, i32 15, metadata !7, i1 false, i1 true, i32 0, i32 0, i32 0, i32 0, i1 true, void ()* @foobar} ; [ DW_TAG_subprogram ]
>>> +!7 = metadata !{i32 589845, metadata !1, metadata !"", metadata !1, i32 0, i64 0, i64 0, i32 0, i32 0, i32 0, metadata !8, i32 0, i32 0} ; [ DW_TAG_subroutine_type ]
>>> +!8 = metadata !{null}
>>> +!9 = metadata !{i32 590081, metadata !0, metadata !"sp", metadata !1, i32 7, metadata !10, i32 0} ; [ DW_TAG_arg_variable ]
>>> +!10 = metadata !{i32 589839, metadata !2, metadata !"", null, i32 0, i64 64, i64 64, i64 0, i32 0, metadata !11} ; [ DW_TAG_pointer_type ]
>>> +!11 = metadata !{i32 589846, metadata !2, metadata !"S1", metadata !1, i32 4, i64 0, i64 0, i64 0, i32 0, metadata !12} ; [ DW_TAG_typedef ]
>>> +!12 = metadata !{i32 589843, metadata !2, metadata !"S1", metadata !1, i32 1, i64 128, i64 64, i32 0, i32 0, i32 0, metadata !13, i32 0, i32 0} ; [ DW_TAG_structure_type ]
>>> +!13 = metadata !{metadata !14, metadata !17}
>>> +!14 = metadata !{i32 589837, metadata !1, metadata !"m", metadata !1, i32 2, i64 64, i64 64, i64 0, i32 0, metadata !15} ; [ DW_TAG_member ]
>>> +!15 = metadata !{i32 589839, metadata !2, metadata !"", null, i32 0, i64 64, i64 64, i64 0, i32 0, metadata !16} ; [ DW_TAG_pointer_type ]
>>> +!16 = metadata !{i32 589860, metadata !2, metadata !"float", null, i32 0, i64 32, i64 32, i64 0, i32 0, i32 4} ; [ DW_TAG_base_type ]
>>> +!17 = metadata !{i32 589837, metadata !1, metadata !"nums", metadata !1, i32 3, i64 32, i64 32, i64 64, i32 0, metadata !5} ; [ DW_TAG_member ]
>>> +!18 = metadata !{i32 590081, metadata !0, metadata !"nums", metadata !1, i32 7, metadata !5, i32 0} ; [ DW_TAG_arg_variable ]
>>> +!19 = metadata !{i32 589876, i32 0, metadata !2, metadata !"p", metadata !"p", metadata !"", metadata !1, i32 14, metadata !11, i32 0, i32 1, %struct.S1* @p} ; [ DW_TAG_variable ]
>>> +!20 = metadata !{i32 7, i32 13, metadata !0, null}
>>> +!21 = metadata !{i32 7, i32 21, metadata !0, null}
>>> +!22 = metadata !{i32 9, i32 3, metadata !23, null}
>>> +!23 = metadata !{i32 589835, metadata !0, i32 8, i32 1, metadata !1, i32 0} ; [ DW_TAG_lexical_block ]
>>> +!24 = metadata !{metadata !"int", metadata !25}
>>> +!25 = metadata !{metadata !"omnipotent char", metadata !26}
>>> +!26 = metadata !{metadata !"Simple C/C++ TBAA", null}
>>> +!27 = metadata !{i32 10, i32 3, metadata !23, null}
>>> +!28 = metadata !{metadata !"any pointer", metadata !25}
>>> +!29 = metadata !{i32 11, i32 3, metadata !23, null}
>>> +!30 = metadata !{%struct.S1* @p}
>>> +!31 = metadata !{i32 7, i32 13, metadata !0, metadata !32}
>>> +!32 = metadata !{i32 16, i32 3, metadata !33, null}
>>> +!33 = metadata !{i32 589835, metadata !6, i32 15, i32 15, metadata !1, i32 1} ; [ DW_TAG_lexical_block ]
>>> +!34 = metadata !{i32 1}
>>> +!35 = metadata !{i32 7, i32 21, metadata !0, metadata !32}
>>> +!36 = metadata !{i32 9, i32 3, metadata !23, metadata !32}
>>> +!37 = metadata !{i32 10, i32 3, metadata !23, metadata !32}
>>> +!38 = metadata !{i32 17, i32 1, metadata !33, null}
>>>
>>>
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
More information about the llvm-commits
mailing list