Page MenuHomePhabricator
Feed Advanced Search

Today

aprantl added a comment to D60831: [DebugInfo@O2][LoopVectorize] pr39024: Vectorized code linenos step through loop even after completion.

Thanks for your help!

Wed, Jun 19, 6:26 AM · debug-info, Restricted Project

Yesterday

aprantl committed rGfc5107cde644: Add debug location verification for !llvm.loop attachments. (authored by aprantl).
Add debug location verification for !llvm.loop attachments.
Tue, Jun 18, 1:06 PM
aprantl committed rG1db8d4a8660f: Fix broken debug info in in an !llvm.loop attachment in this testcase. (authored by aprantl).
Fix broken debug info in in an !llvm.loop attachment in this testcase.
Tue, Jun 18, 1:05 PM
aprantl committed rGacc93d62e048: Revert Add debug location verification for !llvm.loop attachments. (authored by aprantl).
Revert Add debug location verification for !llvm.loop attachments.
Tue, Jun 18, 12:52 PM
aprantl committed rG8ff822d61dac: Add debug location verification for !llvm.loop attachments. (authored by aprantl).
Add debug location verification for !llvm.loop attachments.
Tue, Jun 18, 12:41 PM
aprantl added a comment to D63361: Pretend NRVO variables are references so they can be found by debug info.

Thanks, this addresses my concern!

Tue, Jun 18, 11:03 AM · debug-info, Restricted Project, Restricted Project
aprantl added a comment to D60831: [DebugInfo@O2][LoopVectorize] pr39024: Vectorized code linenos step through loop even after completion.

I put up a Verifier patch at https://reviews.llvm.org/D63499. Once that patch has landed, this patch should be safe to re-apply.

Tue, Jun 18, 9:52 AM · debug-info, Restricted Project
aprantl created D63499: Add debug location verification for !llvm.loop attachments.
Tue, Jun 18, 9:52 AM · Restricted Project, debug-info

Mon, Jun 17

aprantl committed rG2d26cf37d777: llgdb.py: Make sure to clean up the debugger on exit. (authored by aprantl).
llgdb.py: Make sure to clean up the debugger on exit.
Mon, Jun 17, 1:04 PM
aprantl added inline comments to D63429: [DebugInfo] Avoid adding too much indirection to pointer-valued variables.
Mon, Jun 17, 10:11 AM · Restricted Project
aprantl added a project to D63387: clang: Promote -fdebug-compilation-dir from cc1 flag to driver-level flag: debug-info.
Mon, Jun 17, 10:07 AM · debug-info, Restricted Project
aprantl added a comment to D60831: [DebugInfo@O2][LoopVectorize] pr39024: Vectorized code linenos step through loop even after completion.

This seems to be a case of having old, incorrect, metadata baked into the test.
The 'startLoc' (!680) and 'endLoc' (!681) in loop metadta for loop !679 are both
missing an 'inlinedAt' node.

Mon, Jun 17, 10:02 AM · debug-info, Restricted Project
aprantl added a comment to D63083: [DebugInfo][Docs] Document that prologue/epilogue variable location changes are ignored.

I think this is good to go.

Mon, Jun 17, 9:49 AM · Restricted Project

Fri, Jun 14

aprantl requested changes to D63361: Pretend NRVO variables are references so they can be found by debug info.
Fri, Jun 14, 4:25 PM · debug-info, Restricted Project, Restricted Project
aprantl added a project to D63361: Pretend NRVO variables are references so they can be found by debug info: debug-info.
Fri, Jun 14, 4:21 PM · debug-info, Restricted Project, Restricted Project
aprantl accepted D62743: Add color to the default thread and frame format..
Fri, Jun 14, 3:01 PM · Restricted Project, Restricted Project
aprantl added inline comments to D63104: Add GSYM utility files along with unit tests..
Fri, Jun 14, 1:16 PM · Restricted Project
aprantl added inline comments to D63322: DWARF: Avoid storing DIERefs in long-lived containers.
Fri, Jun 14, 1:13 PM · Restricted Project
aprantl accepted D61062: Keep call site info valid through the backend.
Fri, Jun 14, 1:09 PM · debug-info
aprantl committed rG573ffd88a0e3: Python 3: decode string as utf-8 to avoid type mismatch. (authored by aprantl).
Python 3: decode string as utf-8 to avoid type mismatch.
Fri, Jun 14, 8:38 AM
aprantl committed rG38be2c65b629: Make crashlog.py less noisy (authored by aprantl).
Make crashlog.py less noisy
Fri, Jun 14, 8:38 AM
aprantl updated the diff for D63311: Python 3: decode string as utf-8 to avoid type mismatch..

Address review feedback.

Fri, Jun 14, 8:31 AM · Restricted Project, Restricted Project

Thu, Jun 13

aprantl created D63311: Python 3: decode string as utf-8 to avoid type mismatch..
Thu, Jun 13, 6:24 PM · Restricted Project, Restricted Project
aprantl created D63310: Make crashlog.py less noisy.
Thu, Jun 13, 5:31 PM · Restricted Project, Restricted Project
aprantl added inline comments to D63286: [GlobalISel][IRTranslator] Don't add debug info to constants emitted into the entry block.
Thu, Jun 13, 12:49 PM · debug-info, Restricted Project
aprantl accepted D63171: Don't try to parse ObjC method if CU isn't ObjC.

Thanks!

Thu, Jun 13, 12:46 PM · Restricted Project
aprantl accepted D63275: [docs][llvm-dwarfdump] Make the --show-parents and --show-children help text and docs more consistent and correct.
Thu, Jun 13, 11:20 AM · Restricted Project
aprantl added inline comments to D63286: [GlobalISel][IRTranslator] Don't add debug info to constants emitted into the entry block.
Thu, Jun 13, 11:11 AM · debug-info, Restricted Project
aprantl added a reviewer for D63286: [GlobalISel][IRTranslator] Don't add debug info to constants emitted into the entry block: vsk.
Thu, Jun 13, 11:11 AM · debug-info, Restricted Project

Wed, Jun 12

aprantl accepted D63229: [Reproducers] Include lldb version in the reproducer root.
Wed, Jun 12, 3:27 PM · Restricted Project, Restricted Project
aprantl added inline comments to D63229: [Reproducers] Include lldb version in the reproducer root.
Wed, Jun 12, 2:44 PM · Restricted Project, Restricted Project
aprantl added a reviewer for D63228: [llvm] Expose DWARFDebugLine::LineTable::getFileNameEntry: JDevlieghere.
Wed, Jun 12, 2:40 PM · debug-info, Restricted Project
aprantl committed rG87f75ecd720a: Skip failing test on older versions of clang. (authored by aprantl).
Skip failing test on older versions of clang.
Wed, Jun 12, 2:30 PM
aprantl accepted D63213: [DebugInfo] Move Value struct out of DebugLocEntry as DbgValueLoc (NFC).
Wed, Jun 12, 9:41 AM · Restricted Project, debug-info
aprantl accepted D62904: [DebugInfo] Honour variable fragments in LiveDebugValues.

One last nit inline. Thanks!

Wed, Jun 12, 8:33 AM · Restricted Project
aprantl added a comment to D60831: [DebugInfo@O2][LoopVectorize] pr39024: Vectorized code linenos step through loop even after completion.

Thanks for the quick response! Please let me know if I can help debugging this.

Wed, Jun 12, 8:28 AM · debug-info, Restricted Project

Tue, Jun 11

aprantl added a comment to D63171: Don't try to parse ObjC method if CU isn't ObjC.

I suppose one could compile Objective-C code on Linux using GCC.

Will GCC not set the language to ObjC or ObjC++?

Tue, Jun 11, 5:33 PM · Restricted Project
aprantl added a comment to D60831: [DebugInfo@O2][LoopVectorize] pr39024: Vectorized code linenos step through loop even after completion.

This patch broke our internal CI. You can reproduce the issue by downloading https://gist.github.com/adrian-prantl/ba88912878db855ec96534e6510246e6 (this is AArch64 bitcode for a file in the Swift stdlib) and running

Tue, Jun 11, 4:37 PM · debug-info, Restricted Project
aprantl added a comment to D63171: Don't try to parse ObjC method if CU isn't ObjC.

I suppose one could compile Objective-C code on Linux using GCC.

Tue, Jun 11, 4:02 PM · Restricted Project
aprantl added a comment to D63119: hwasan: Add a tag_offset DWARF attribute to instrumented stack variables..

Hmm, I'm not sure what else we need aside from the existing isValid() check in Verifier::visitDIExpression.

Tue, Jun 11, 2:31 PM · Restricted Project
aprantl added a comment to D62904: [DebugInfo] Honour variable fragments in LiveDebugValues.

I think this is starting to look good!

Tue, Jun 11, 2:28 PM · Restricted Project
aprantl committed rGe6130a309094: Fix a crash in option parsing. (authored by aprantl).
Fix a crash in option parsing.
Tue, Jun 11, 2:13 PM
aprantl added a comment to D62904: [DebugInfo] Honour variable fragments in LiveDebugValues.

Stylistic nit: unless this will push a lot of lines > 80 columns and kill readability, I would prefer Fragment over Frag.

Tue, Jun 11, 9:56 AM · Restricted Project
aprantl added a comment to D63119: hwasan: Add a tag_offset DWARF attribute to instrumented stack variables..

Can you please add this to either LangRef.rst or SourceLevelDebugging.rst and also explain how this operation composes with other DIExpressions? I'm thinking particularly about DW_OP_LLVM_fragment.
This also needs an assembler round-trip-test and probably some checks in Verifier.cpp.

Tue, Jun 11, 9:44 AM · Restricted Project
aprantl updated subscribers of D63119: hwasan: Add a tag_offset DWARF attribute to instrumented stack variables..
Tue, Jun 11, 9:39 AM · Restricted Project

Mon, Jun 10

aprantl added a comment to D63013: [llvm-dwarfdump] Add -o to help text and remove --out-file from documentation.

dwarfdump(-classic) on macOS also supported the --out-file option and on macOS we strive for compatibility with the older tool.

Mon, Jun 10, 5:08 PM · Restricted Project
aprantl created D63110: Fix a crash in option parsing..
Mon, Jun 10, 4:59 PM · Restricted Project, Restricted Project
aprantl accepted D63013: [llvm-dwarfdump] Add -o to help text and remove --out-file from documentation.

I'm mildly against this change. IIRC, the idea was that --out-file is a long-form option that nobody uses in practice and we wanted people to prefer -o instead, which is what people are most familiar with working with compilers. So if -o shows up in the --help output, I'd leave it at that.

Could we just delete --out-file, if we don't want people to use it?

Mon, Jun 10, 4:15 PM · Restricted Project
aprantl added a comment to D62904: [DebugInfo] Honour variable fragments in LiveDebugValues.

It's my belief that that's how SROA works, but I don't think it's a documented guarantee. SROA::splitAlloca certainly seems to completely rewrite dbg.declare's into ones that all use fragments. The verifier doesn't attempt to enforce this rule though.

Mon, Jun 10, 4:11 PM · Restricted Project
aprantl added inline comments to rG3027a2999c32: [dsymutil] Support more than 4 architectures.
Mon, Jun 10, 1:44 PM

Sat, Jun 8

aprantl added a comment to D63013: [llvm-dwarfdump] Add -o to help text and remove --out-file from documentation.

That said, if clang also documents -o as --out-file, I think consistency wins.

Sat, Jun 8, 8:59 AM · Restricted Project
aprantl added a comment to D63013: [llvm-dwarfdump] Add -o to help text and remove --out-file from documentation.

I'm mildly against this change. IIRC, the idea was that --out-file is a long-form option that nobody uses in practice and we wanted people to prefer -o instead, which is what people are most familiar with working with compilers. So if -o shows up in the --help output, I'd leave it at that.

Sat, Jun 8, 8:58 AM · Restricted Project

Fri, Jun 7

aprantl added inline comments to D63005: DWARF: Don't create lldb CompileUnits for DWARF type units.
Fri, Jun 7, 2:23 PM · Restricted Project
aprantl added inline comments to D62894: DWARF: Share line tables of type units.
Fri, Jun 7, 2:23 PM · Restricted Project
aprantl requested changes to D61062: Keep call site info valid through the backend.

(marking as "request changes", so I don't forget to reply to the question, otherwise this will get lost in my inbox)

Fri, Jun 7, 2:16 PM · debug-info
aprantl added a comment to D61600: [DebugInfo] More precise variable range for stack locations.

Thanks, that looks like an acceptable performance/accuracy tradeoff.

Fri, Jun 7, 2:06 PM · Restricted Project, debug-info
aprantl accepted D62684: [X86] Fixup LEAs - Fix inconsistent codegen with/without debug info.
Fri, Jun 7, 2:06 PM · Restricted Project

Tue, Jun 4

aprantl added a comment to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..

And thanks for doing the measurements!

Tue, Jun 4, 8:58 PM · debug-info, Restricted Project
aprantl added a comment to D60713: [IR] Add DISuprogram and DIE for func decl of an external.

Thanks!

Tue, Jun 4, 8:58 PM · debug-info
aprantl accepted D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..

I'm kind of curious where the performance is gained back, but generally, if the performance is the same, we can tentatively land this. Please be quick to revert if complaints about the performance resurface.

Tue, Jun 4, 8:57 PM · debug-info, Restricted Project

Mon, Jun 3

aprantl added a comment to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..

Have you looked at the performance of the example posted in https://bugs.llvm.org/show_bug.cgi?id=33730 ?

Mon, Jun 3, 9:25 AM · debug-info, Restricted Project
aprantl added a comment to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..

Have you looked at the performance of the example posted in https://bugs.llvm.org/show_bug.cgi?id=33730 ?

Mon, Jun 3, 9:15 AM · debug-info, Restricted Project
aprantl added a comment to D62743: Add color to the default thread and frame format..

Sorry, I got more comments :-)

Mon, Jun 3, 8:59 AM · Restricted Project, Restricted Project
aprantl accepted D61062: Keep call site info valid through the backend.
Mon, Jun 3, 8:59 AM · debug-info
aprantl accepted D61940: [DebugInfo] Don't always extend variable locations when the reg location is unchanging.

The assumption that stack variable locations run to the end of a function, even over the frame destruction, is seen in GCC here [0].

Mon, Jun 3, 8:53 AM · Restricted Project
aprantl added inline comments to D62684: [X86] Fixup LEAs - Fix inconsistent codegen with/without debug info.
Mon, Jun 3, 8:52 AM · Restricted Project
aprantl added inline comments to D62684: [X86] Fixup LEAs - Fix inconsistent codegen with/without debug info.
Mon, Jun 3, 8:52 AM · Restricted Project
aprantl added a comment to D61600: [DebugInfo] More precise variable range for stack locations.

Have you tested the performance when compiling Clang with ASAN enabled to make sure we're not accidentally regressing because of the more thorough single-location check?

I've built LLVM with '-DLLVM_USE_SANITIZER=Address' using downloaded clang binary to produce private clang binary. I've run tests and compiled latest version of gdb with new private binary and I didn't see any regression there. Is that what you meant?

I think that the idea was to see if this patch increases the compilation time when building such a Clang binary.

(Just make sure to use the RelWithDebInfo build type also.)

Mon, Jun 3, 8:45 AM · Restricted Project, debug-info
aprantl accepted D62805: [DebugInfo] Fix possible invalid dereference of non-engaged Optional..
Mon, Jun 3, 8:41 AM · Restricted Project
aprantl added a comment to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..

I'm reposting your reply with an easier to read format:

log(before this patch) :

512B bb.2.for.body:

520B renamable $x8 = MOVaddr target-flags(aarch64-page) @array, target-flags(aarch64-pageoff, aarch64-nc) @array, debug-location !54; test_debug_val.cpp:16:17
524B renamable $w9 = MOVi32imm 56, debug-location !54; test_debug_val.cpp:16:17

DBG_VALUE $x19, $noreg, !"i", !DIExpression(), debug-location !51; test_debug_val.cpp:0 line no:14
544B STRWroX killed renamable $w9, killed renamable $x8, renamable $x19, 0, 1, debug-location !54 :: (store 4 into %ir.scevgep, !tbaa !38); test_debug_val.cpp:16:17

!51 = !DILocation(line: 0, scope: !35)
!52 = !DILocation(line: 14, column: 4, scope: !35)
!53 = !DILocation(line: 21, column: 4, scope: !14)
!54 = !DILocation(line: 16, column: 8, scope: !55)

short log :

512B bb.2.for.body:

520B $x8 = MOVaddr scope: !55
524B $w9 = MOVi32imm scope: !55

DBG_VALUE $x19, $noreg, !"i" scope: !35
544B STRWroX killed renamable $w9 scope: !55

i.e. DBG_VALUE was inserted into enclosed scope.

The reason for this is that scope identified by MachineInstr :

LiveDebugVariables.cpp:902

const InsnRange &Range : Scope->getRanges()

using InsnRange = std::pair<const MachineInstr *, const MachineInstr *>;

At the moment when scope was created the very first instruction was STRWroX.
So it become start of range. Start slot index(with type of Slot_Block) for
DBG_VALUE was trimmed to this instruction range. But after Register Allocator
there were inserted new MOV instructions before STRWroX with the same scope as STRWroX.
But Slot indices for DBG_VALUE was not updated and scope boundaries was not updated.

Actually Slot indices for DBG_VALUE looks correct without trimming : 512B,848B.
512B points to start of basic block:

512B bb.2.for.body:

which match with definition of slot index of type Slot_Block:

Basic block boundary. Used for live ranges entering and leaving a
/ block without being live in the layout neighbor. Also used as the
/// def slot of PHI-defs.

i.e. 512B,848B actually match with lexical scope and does not require trimming.

Also, trimming this slot index and leaving it of type Slot_Block is a error. Slot_Block should point to start of basic block.

I.setStartUnchecked(RStart); <<< trimmed I.start() is a Slot_Block

This concrete problem could be solved by avoiding trimming for Slot_Block only.

But it looks like trimming also is not required for the original problem test case : live-debug-variables.ll. With and without trimming it generates the same DBG_VALUE instructions :

DBG_VALUE $ecx, $noreg, !"i4", !DIExpression(), debug-location !14; foo.c:3:35 @[ foo.c:8:10 ] line no:3
DBG_VALUE $ebx, $noreg, !"i4", !DIExpression(), debug-location !14; foo.c:3:35 @[ foo.c:8:10 ] line no:3

So in short :

For the problem from D62650: SlotIndex was trimmed incorrectly. It does not require trimming and has incorrect value for Slot_Block after trimming.
Trimming for testcase from D35953 looks unnecessary. With and without trimming it generates the same DBG_VALUE instructions.
Thus it looks like trimming could be removed at all.

Mon, Jun 3, 8:33 AM · debug-info, Restricted Project
aprantl added inline comments to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..
Mon, Jun 3, 8:33 AM · debug-info, Restricted Project

Fri, May 31

aprantl added a comment to D62755: [Target] Remove Process::GetCPPLanguageRuntime.

I disagree that it is "carrying purity a little too far". My goal is to see LLDB's non-plugin libraries be more language agnostic.

I have two motivations for this:

Fri, May 31, 4:53 PM · Restricted Project
aprantl added a comment to D62755: [Target] Remove Process::GetCPPLanguageRuntime.

I tend to agree with Jim's comment.

Fri, May 31, 4:02 PM · Restricted Project
aprantl added inline comments to D62755: [Target] Remove Process::GetCPPLanguageRuntime.
Fri, May 31, 3:38 PM · Restricted Project
aprantl added inline comments to D62756: Be consistent when adding names and mangled names to the index.
Fri, May 31, 3:35 PM
aprantl added a comment to D62743: Add color to the default thread and frame format..

Let the bike-shedding begin!
Feel free to ignore this, but personally, I'd only highlight the filename (or highlight the the line number in a different color).
Feel free to ignore this even more, but we should probably have symbolic color names instead of hardcoding red and such, for users with black-on-white vs. white-on-black (or green-on-black) terminals.

Fri, May 31, 3:32 PM · Restricted Project, Restricted Project
aprantl added a comment to D60713: [IR] Add DISuprogram and DIE for func decl of an external.

Oh.. and could you please document this either in LangRef.rst or SourceLevelDebugging.rst?

Fri, May 31, 9:32 AM · debug-info
aprantl added inline comments to D60713: [IR] Add DISuprogram and DIE for func decl of an external.
Fri, May 31, 9:32 AM · debug-info
aprantl accepted D60713: [IR] Add DISuprogram and DIE for func decl of an external.

Thanks!

Fri, May 31, 9:28 AM · debug-info
aprantl updated subscribers of D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..
Fri, May 31, 9:22 AM · debug-info, Restricted Project
aprantl added inline comments to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..
Fri, May 31, 9:16 AM · debug-info, Restricted Project

Thu, May 30

aprantl committed rG4a585a3edd1c: Make CPlusPlusNameParser robust against nullptr StringRefs. (authored by aprantl).
Make CPlusPlusNameParser robust against nullptr StringRefs.
Thu, May 30, 5:20 PM
aprantl accepted D62622: [CMake] Provide an option to use relative paths in debug info.
Thu, May 30, 4:47 PM · debug-info, Restricted Project, Restricted Project
aprantl added a comment to D60713: [IR] Add DISuprogram and DIE for func decl of an external.

Prior to this patch Function declarations may not have a !dbg attachment, because there was no use-case for it. With call site debug info, I think it would make sense to revisit this rule. When I made my previous comment, I thought that the updated rule could be:

Thu, May 30, 8:56 AM · debug-info
aprantl added a reviewer for D60713: [IR] Add DISuprogram and DIE for func decl of an external: dexonsmith.
Thu, May 30, 8:45 AM · debug-info
aprantl added inline comments to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..
Thu, May 30, 8:42 AM · debug-info, Restricted Project
aprantl added inline comments to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly..
Thu, May 30, 8:41 AM · debug-info, Restricted Project
aprantl added a project to D62650: [DebugInfo] DBG_VALUE instruction generated for loop counter is placed incorrectly.: debug-info.
Thu, May 30, 8:38 AM · debug-info, Restricted Project
aprantl added inline comments to D62684: [X86] Fixup LEAs - Fix inconsistent codegen with/without debug info.
Thu, May 30, 8:36 AM · Restricted Project
aprantl added a project to D62622: [CMake] Provide an option to use relative paths in debug info: debug-info.
Thu, May 30, 8:34 AM · debug-info, Restricted Project, Restricted Project
aprantl added a comment to D62622: [CMake] Provide an option to use relative paths in debug info.

I like LLVM_USE_RELATIVE_PATHS_IN_DEBUG_INFO.

Thu, May 30, 8:34 AM · debug-info, Restricted Project, Restricted Project

Wed, May 29

aprantl accepted D62634: Improve DWARF parsing and accessing by 1% to 2%.

Out of curiosity, what was that change in the mmap call that fixed this?

Wed, May 29, 4:32 PM
aprantl added a comment to D62622: [CMake] Provide an option to use relative paths in debug info.

-DLLVM_USE_RELATIVE_PATHS is very generic sounding and might give a false impression. Is there a name that emphasizes that this affects the debug info of the built compiler?

Wed, May 29, 4:26 PM · debug-info, Restricted Project, Restricted Project
aprantl accepted D62630: Fix a regression in DWARF access speed caused by svn revision 356190.
Wed, May 29, 4:26 PM
aprantl accepted D62408: [X86] Avoid SFB - Fix inconsistent codegen with/without debug info(2).
Wed, May 29, 4:16 PM · Restricted Project
aprantl accepted D62605: [X86] Avoid SFB - Ignore meta instructions.
Wed, May 29, 4:16 PM · Restricted Project
aprantl added inline comments to D62605: [X86] Avoid SFB - Ignore meta instructions.
Wed, May 29, 4:16 PM · Restricted Project
aprantl added inline comments to D61062: Keep call site info valid through the backend.
Wed, May 29, 4:16 PM · debug-info
aprantl added a comment to D62570: [WIP] Use LLVM's debug line parser in LLDB.

Very nice!

Wed, May 29, 2:56 PM · Restricted Project, Restricted Project
aprantl added a comment to D60713: [IR] Add DISuprogram and DIE for func decl of an external.

The patch is needed to describe a call site to an external function. Currently, we do not have all necessary info for the debug call site (particular debug subprogram is missing for the external function). Since we can describe call site parameters for calls like that, we are producing additional subprogram for the declaration of the extern function by marking it with the DISPFlagDeclForCallSite flag.

Wed, May 29, 2:55 PM · debug-info