<div dir="ltr"><div dir="ltr">Hello Vedant,<br><br>This commit added broken test to the builder
llvm-clang-x86_64-expensive-checks-win
.<br><br><a href="http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/19367/steps/test-check-all/logs/stdio">http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/19367/steps/test-check-all/logs/stdio</a>:<br>. . . <br>Failing Tests (8):<br> . . .<br> LLVM :: DebugInfo/MIR/X86/DW_OP_entry_value.mir<br> . . .<br><br>The builder was already red and did not send notifications on this.<br><div>For now it the only broken test on this builder.</div><div> Please have a look?</div><br>Thanks<br><br>Galina</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 26, 2019 at 1:51 PM Vedant Kumar via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Author: vedantk<br>
Date: Mon Aug 26 13:53:12 2019<br>
New Revision: 369966<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=369966&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=369966&view=rev</a><br>
Log:<br>
[DWARF] Pick the DWARF5 OP_entry_value opcode on Darwin<br>
<br>
Use the GNU extension for OP_entry_value consistently (i.e. whenever GNU<br>
extensions are used for TAG_call_site).<br>
<br>
Added:<br>
llvm/trunk/test/DebugInfo/MIR/X86/DW_OP_entry_value.mir<br>
Modified:<br>
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp<br>
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h<br>
llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpp<br>
<br>
Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp?rev=369966&r1=369965&r2=369966&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp?rev=369966&r1=369965&r2=369966&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp Mon Aug 26 13:53:12 2019<br>
@@ -892,9 +892,13 @@ void DwarfCompileUnit::constructAbstract<br>
ContextCU->addDIEEntry(*AbsDef, dwarf::DW_AT_object_pointer, *ObjectPointer);<br>
}<br>
<br>
+/// Whether to use the GNU analog for a DWARF5 tag, attribute, or location atom.<br>
+static bool useGNUAnalogForDwarf5Feature(DwarfDebug *DD) {<br>
+ return DD->getDwarfVersion() == 4 && DD->tuneForGDB();<br>
+}<br>
+<br>
dwarf::Tag DwarfCompileUnit::getDwarf5OrGNUCallSiteTag(dwarf::Tag Tag) const {<br>
- bool ApplyGNUExtensions = DD->getDwarfVersion() == 4 && DD->tuneForGDB();<br>
- if (!ApplyGNUExtensions)<br>
+ if (!useGNUAnalogForDwarf5Feature(DD))<br>
return Tag;<br>
switch (Tag) {<br>
case dwarf::DW_TAG_call_site:<br>
@@ -908,8 +912,7 @@ dwarf::Tag DwarfCompileUnit::getDwarf5Or<br>
<br>
dwarf::Attribute<br>
DwarfCompileUnit::getDwarf5OrGNUCallSiteAttr(dwarf::Attribute Attr) const {<br>
- bool ApplyGNUExtensions = DD->getDwarfVersion() == 4 && DD->tuneForGDB();<br>
- if (!ApplyGNUExtensions)<br>
+ if (!useGNUAnalogForDwarf5Feature(DD))<br>
return Attr;<br>
switch (Attr) {<br>
case dwarf::DW_AT_call_all_calls:<br>
@@ -929,6 +932,18 @@ DwarfCompileUnit::getDwarf5OrGNUCallSite<br>
}<br>
}<br>
<br>
+dwarf::LocationAtom<br>
+DwarfCompileUnit::getDwarf5OrGNULocationAtom(dwarf::LocationAtom Loc) const {<br>
+ if (!useGNUAnalogForDwarf5Feature(DD))<br>
+ return Loc;<br>
+ switch (Loc) {<br>
+ case dwarf::DW_OP_entry_value:<br>
+ return dwarf::DW_OP_GNU_entry_value;<br>
+ default:<br>
+ llvm_unreachable("DWARF5 location atom with no GNU analog");<br>
+ }<br>
+}<br>
+<br>
DIE &DwarfCompileUnit::constructCallSiteEntryDIE(<br>
DIE &ScopeDIE, const DISubprogram *CalleeSP, bool IsTail,<br>
const MCSymbol *PCAddr, const MCExpr *PCOffset, unsigned CallReg) {<br>
<br>
Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h?rev=369966&r1=369965&r2=369966&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h?rev=369966&r1=369965&r2=369966&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h (original)<br>
+++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfCompileUnit.h Mon Aug 26 13:53:12 2019<br>
@@ -234,6 +234,9 @@ public:<br>
/// GNU attribute if needed.<br>
dwarf::Attribute getDwarf5OrGNUCallSiteAttr(dwarf::Attribute Attr) const;<br>
<br>
+ /// This takes a DWARF 5 location atom and either returns it or a GNU analog.<br>
+ dwarf::LocationAtom getDwarf5OrGNULocationAtom(dwarf::LocationAtom Loc) const;<br>
+<br>
/// Construct a call site entry DIE describing a call within \p Scope to a<br>
/// callee described by \p CalleeSP.<br>
/// \p IsTail specifies whether the call is a tail call.<br>
<br>
Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpp?rev=369966&r1=369965&r2=369966&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpp?rev=369966&r1=369965&r2=369966&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfExpression.cpp Mon Aug 26 13:53:12 2019<br>
@@ -311,10 +311,7 @@ void DwarfExpression::addEntryValueExpre<br>
assert(!isMemoryLocation() &&<br>
"We don't support entry values of memory locations yet");<br>
<br>
- if (DwarfVersion >= 5)<br>
- emitOp(dwarf::DW_OP_entry_value);<br>
- else<br>
- emitOp(dwarf::DW_OP_GNU_entry_value);<br>
+ emitOp(CU.getDwarf5OrGNULocationAtom(dwarf::DW_OP_entry_value));<br>
emitUnsigned(Op->getArg(0));<br>
}<br>
<br>
<br>
Added: llvm/trunk/test/DebugInfo/MIR/X86/DW_OP_entry_value.mir<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/MIR/X86/DW_OP_entry_value.mir?rev=369966&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/MIR/X86/DW_OP_entry_value.mir?rev=369966&view=auto</a><br>
==============================================================================<br>
--- llvm/trunk/test/DebugInfo/MIR/X86/DW_OP_entry_value.mir (added)<br>
+++ llvm/trunk/test/DebugInfo/MIR/X86/DW_OP_entry_value.mir Mon Aug 26 13:53:12 2019<br>
@@ -0,0 +1,81 @@<br>
+# RUN: llc -debug-entry-values -mtriple=x86_64-apple-darwin -o %t %s -filetype=obj<br>
+# RUN: llvm-dwarfdump %t | FileCheck %s<br>
+#<br>
+# int global;<br>
+# int foo(int p, int q, int r) {<br>
+# global = p + 1;<br>
+# asm __volatile("" : : : "edi", "esi", "edx");<br>
+# return 123;<br>
+# }<br>
+<br>
+# CHECK: DW_TAG_formal_parameter<br>
+# CHECK: DW_OP_entry_value<br>
+<br>
+--- |<br>
+ ; ModuleID = 'multiple-param-dbg-value-entry.ll'<br>
+ source_filename = "multiple-param-dbg-value-entry.c"<br>
+ target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"<br>
+<br>
+ @global = common dso_local local_unnamed_addr global i32 0, align 4, !dbg !0<br>
+<br>
+ define dso_local i32 @foo(i32 %p, i32 %q, i32 %r) local_unnamed_addr !dbg !11 {<br>
+ entry:<br>
+ call void @llvm.dbg.value(metadata i32 %p, metadata !15, metadata !DIExpression()), !dbg !18<br>
+ call void @llvm.dbg.value(metadata i32 %q, metadata !16, metadata !DIExpression()), !dbg !18<br>
+ call void @llvm.dbg.value(metadata i32 %r, metadata !17, metadata !DIExpression()), !dbg !18<br>
+ %add = add nsw i32 %p, 1, !dbg !18<br>
+ store i32 %add, i32* @global, align 4, !dbg !18<br>
+ tail call void asm sideeffect "", "~{edi},~{esi},~{edx},~{dirflag},~{fpsr},~{flags}"(), !dbg !18, !srcloc !19<br>
+ ret i32 123, !dbg !18<br>
+ }<br>
+<br>
+ ; Function Attrs: nounwind readnone speculatable<br>
+ declare void @llvm.dbg.value(metadata, metadata, metadata)<br>
+<br>
+ !<a href="http://llvm.dbg.cu" rel="noreferrer" target="_blank">llvm.dbg.cu</a> = !{!2}<br>
+ !llvm.module.flags = !{!7, !8, !9}<br>
+ !llvm.ident = !{!10}<br>
+<br>
+ !0 = !DIGlobalVariableExpression(var: !1, expr: !DIExpression())<br>
+ !1 = distinct !DIGlobalVariable(name: "global", scope: !2, file: !3, line: 8, type: !6, isLocal: false, isDefinition: true)<br>
+ !2 = distinct !DICompileUnit(language: DW_LANG_C99, file: !3, producer: "clang version 9.0.0 ", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !4, globals: !5, nameTableKind: None)<br>
+ !3 = !DIFile(filename: "multiple-param-dbg-value-entry.c", directory: "/")<br>
+ !4 = !{}<br>
+ !5 = !{!0}<br>
+ !6 = !DIBasicType(name: "int", size: 32, encoding: DW_ATE_signed)<br>
+ !7 = !{i32 2, !"Dwarf Version", i32 4}<br>
+ !8 = !{i32 2, !"Debug Info Version", i32 3}<br>
+ !9 = !{i32 1, !"wchar_size", i32 4}<br>
+ !10 = !{!"clang version 9.0.0 "}<br>
+ !11 = distinct !DISubprogram(name: "foo", scope: !3, file: !3, line: 9, type: !12, scopeLine: 9, flags: DIFlagPrototyped | DIFlagAllCallsDescribed, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !2, retainedNodes: !14)<br>
+ !12 = !DISubroutineType(types: !13)<br>
+ !13 = !{!6, !6, !6, !6}<br>
+ !14 = !{!15, !16, !17}<br>
+ !15 = !DILocalVariable(name: "p", arg: 1, scope: !11, file: !3, line: 9, type: !6, flags: DIFlagArgumentNotModified)<br>
+ !16 = !DILocalVariable(name: "q", arg: 2, scope: !11, file: !3, line: 9, type: !6, flags: DIFlagArgumentNotModified)<br>
+ !17 = !DILocalVariable(name: "r", arg: 3, scope: !11, file: !3, line: 9, type: !6, flags: DIFlagArgumentNotModified)<br>
+ !18 = !DILocation(line: 9, column: 13, scope: !11)<br>
+ !19 = !{i32 213}<br>
+<br>
+...<br>
+---<br>
+name: foo<br>
+alignment: 4<br>
+tracksRegLiveness: true<br>
+liveins:<br>
+ - { reg: '$edi' }<br>
+body: |<br>
+ bb.0.entry:<br>
+ liveins: $edi<br>
+<br>
+ DBG_VALUE $edi, $noreg, !15, !DIExpression(), debug-location !18<br>
+ DBG_VALUE $edi, $noreg, !15, !DIExpression(), debug-location !18<br>
+ DBG_VALUE $esi, $noreg, !16, !DIExpression(), debug-location !18<br>
+ DBG_VALUE $edx, $noreg, !17, !DIExpression(), debug-location !18<br>
+ renamable $edi = nsw INC32r killed renamable $edi, implicit-def dead $eflags, debug-location !18<br>
+ MOV32mr $rip, 1, $noreg, @global, $noreg, killed renamable $edi, debug-location !18 :: (store 4 into @global)<br>
+ INLINEASM &"", 1, 12, implicit-def dead early-clobber $edi, 12, implicit-def dead early-clobber $esi, 12, implicit-def dead early-clobber $edx, 12, implicit-def dead early-clobber $df, 12, implicit-def dead early-clobber $fpsw, 12, implicit-def dead early-clobber $eflags, !19, debug-location !18<br>
+ $eax = MOV32ri 123, debug-location !18<br>
+ RETQ killed $eax, debug-location !18<br>
+<br>
+...<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>