Page MenuHomePhabricator

aprantl (Adrian Prantl)
User

Projects

User Details

User Since
Mar 2 2013, 8:12 AM (487 w, 3 d)

Recent Activity

Mon, Jun 27

aprantl added inline comments to D128694: [lldb/Dataformatters] Adapt C++ std::string dataformatter for D128285.
Mon, Jun 27, 5:50 PM · Restricted Project, Restricted Project

Wed, Jun 22

aprantl added inline comments to D128321: [lldb] Add OSLog log handler.
Wed, Jun 22, 4:14 PM · Restricted Project, Restricted Project
aprantl accepted D128378: [lldb] Return the correct utility function in AppleObjCRuntimeV2.

Oh boy, these are scary. Should LLDB fail harder when the utility expression fails, so we can detect these earlier?

Wed, Jun 22, 1:54 PM · Restricted Project, Restricted Project
aprantl accepted D128377: [lldb] Fix off-by-one error in the AppleObjCRuntimeV2 utility function.

I'm curious now why there is both count and max_class_infos and if the second is workaround for this bug? Anyway, this *looks* plausible!

Wed, Jun 22, 1:47 PM · Restricted Project, Restricted Project

Tue, Jun 21

aprantl accepted D128312: [lldb] Add a setting to specify the preferred dynamic class info extractor function.

Patch looks fine. Wording suggestion for the user-visible parts inside.

Tue, Jun 21, 5:01 PM · Restricted Project, Restricted Project
aprantl added inline comments to D128312: [lldb] Add a setting to specify the preferred dynamic class info extractor function.
Tue, Jun 21, 5:01 PM · Restricted Project, Restricted Project
aprantl accepted D128306: [lldb] Instantiate lazily named classes on macOS Ventura..

Thanks!

Tue, Jun 21, 4:36 PM · Restricted Project, Restricted Project

Fri, Jun 17

aprantl added a comment to D123319: Change how we handle auto return types for lambda operator() to be consistent with gcc.

What does the linkage name do for your use case? Which cases are missing linkage names/where do they go missing?

I am happy to consider other approaches as well to solving lookup for local lambdas for D105564. Let me know what you think.

Why does the return type help perform lookup? What kind of lookup?

(again, my take is that "auto" return types probably shouldn't be described at all - we should just not describe functions where we don't know their return types because they're not very useful to know about (overload resolution, yes, but then you can't call them anyway) - but that's a broader argument to make/change to make)

Fri, Jun 17, 5:13 PM · Restricted Project, Restricted Project
aprantl accepted D127102: [CloneFunction][DebugInfo] Avoid cloning DILexicalBlocks of inlined subprograms.

Seems reasonable!

Fri, Jun 17, 2:20 PM · debug-info, Restricted Project, Restricted Project
aprantl accepted D126254: Add support for decoding base64..
Fri, Jun 17, 2:18 PM · Restricted Project, Restricted Project
aprantl committed rGf000de8760c1: [LLDB][ExpressionParser] Fix indices inside format-strings passed to LLDB_LOG (authored by aprantl).
[LLDB][ExpressionParser] Fix indices inside format-strings passed to LLDB_LOG
Fri, Jun 17, 9:27 AM · Restricted Project
aprantl closed D128063: [LLDB][ExpressionParser] Fix indices inside format-strings passed to LLDB_LOG.
Fri, Jun 17, 9:27 AM · Restricted Project, Restricted Project
aprantl accepted D128063: [LLDB][ExpressionParser] Fix indices inside format-strings passed to LLDB_LOG.
Fri, Jun 17, 9:14 AM · Restricted Project, Restricted Project

Mon, Jun 13

aprantl added inline comments to D127626: [docs] Add document "Debugging C++ Coroutines".
Mon, Jun 13, 5:09 PM · debug-info, Restricted Project
aprantl added a comment to D125691: [DebugInfo] Add localDecls field to DISubprogram and DILexicalBlock (1/5).

Do you need to add any special handling to CloneFunction.cpp or will it work correctly?

Mon, Jun 13, 5:07 PM · debug-info, Restricted Project, Restricted Project
aprantl accepted D125469: [DWARFLinker] mark odr candidates inside the same object file..

I think this looks reasonable. @JDevlieghere ?

Mon, Jun 13, 5:04 PM · Restricted Project, Restricted Project
aprantl accepted D127441: [dsymutil] Automatically generate a reproducer when dsymutil crashes.
Mon, Jun 13, 5:00 PM · Restricted Project, Restricted Project
aprantl added a comment to D127605: [lldb] Support non-pointer implicit this/self in GetValueForVariableExpressionPath.

More elegant would be to just add an API to TypeSystem to get the operator to access ivars.

Mon, Jun 13, 4:24 PM · Restricted Project, Restricted Project
aprantl accepted D127605: [lldb] Support non-pointer implicit this/self in GetValueForVariableExpressionPath.

This is good, but it also illustrates how the strings "->" and ".'" should actually come from the typesystem and not be hardcoded. We're just lucky that all languages have a "." operator.

Mon, Jun 13, 4:24 PM · Restricted Project, Restricted Project
aprantl committed rGea9ff9fac3a6: [LLDB][formatters] Add formatter for libc++'s std::span (authored by aprantl).
[LLDB][formatters] Add formatter for libc++'s std::span
Mon, Jun 13, 1:00 PM · Restricted Project
aprantl closed D127481: [LLDB][formatters] Add formatter for libc++'s std::span.
Mon, Jun 13, 12:59 PM · Restricted Project, Restricted Project

Fri, Jun 10

aprantl added inline comments to D127481: [LLDB][formatters] Add formatter for libc++'s std::span.
Fri, Jun 10, 12:40 PM · Restricted Project, Restricted Project
aprantl accepted D127481: [LLDB][formatters] Add formatter for libc++'s std::span.
Fri, Jun 10, 12:26 PM · Restricted Project, Restricted Project
aprantl added a comment to D127481: [LLDB][formatters] Add formatter for libc++'s std::span.

Thanks, this looks really good! I have a couple of small comments inline.

Fri, Jun 10, 8:38 AM · Restricted Project, Restricted Project
aprantl committed rGd36757b511ea: [LLDB][Docs] Fix formatting of example code-block (authored by aprantl).
[LLDB][Docs] Fix formatting of example code-block
Fri, Jun 10, 8:17 AM · Restricted Project
aprantl closed D127437: [LLDB][Docs] Fix formatting of example code-block.
Fri, Jun 10, 8:17 AM · Restricted Project, Restricted Project

Thu, Jun 9

aprantl added inline comments to D127441: [dsymutil] Automatically generate a reproducer when dsymutil crashes.
Thu, Jun 9, 2:27 PM · Restricted Project, Restricted Project
aprantl added inline comments to D127441: [dsymutil] Automatically generate a reproducer when dsymutil crashes.
Thu, Jun 9, 2:27 PM · Restricted Project, Restricted Project
aprantl accepted D127437: [LLDB][Docs] Fix formatting of example code-block.
Thu, Jun 9, 12:44 PM · Restricted Project, Restricted Project

Tue, Jun 7

aprantl added a comment to D126883: [Debuginfo][DWARF][NFC] Refactor DwarfStringPoolEntryRef..

Mechanically this looks fine, but I think the documentation could be improved. Could you add something to the comment of DwarfStringPoolEntryRef that explains what the two cases in the PointerUnion and their use-cases are?

Tue, Jun 7, 2:28 PM · Restricted Project, Restricted Project
aprantl accepted D76878: Implement DW_{OP,AT}_LLVM_* for Heterogeneous Debugging.
Tue, Jun 7, 2:13 PM · Restricted Project, debug-info, Restricted Project
aprantl added inline comments to D126254: Add support for decoding base64..
Tue, Jun 7, 1:56 PM · Restricted Project, Restricted Project
aprantl accepted D115351: [Debugify] Port verify-debuginfo-preserve to NewPM.
Tue, Jun 7, 1:47 PM · Restricted Project, Restricted Project, debug-info
aprantl accepted D105564: Fix for DWARF parsing to better handle auto return type for member functions.
Tue, Jun 7, 1:28 PM · Restricted Project

Jun 2 2022

aprantl added a comment to D126689: [IR] Enable opaque pointers by default.

I was able to update LLDB https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/44252/. We can leave this in.

Jun 2 2022, 2:37 PM · Restricted Project, Restricted Project, Restricted Project, Restricted Project
aprantl committed rGe7b929d75650: Adapt IRForTarget::RewriteObjCConstStrings() for D126689. (authored by aprantl).
Adapt IRForTarget::RewriteObjCConstStrings() for D126689.
Jun 2 2022, 1:07 PM · Restricted Project
aprantl committed rG8eed95c83ecd: Adapt IRForTarget::RewriteObjCSelector() for D126689. (authored by aprantl).
Adapt IRForTarget::RewriteObjCSelector() for D126689.
Jun 2 2022, 11:43 AM · Restricted Project
aprantl added a comment to D126689: [IR] Enable opaque pointers by default.

It looks like this patch has broken 168 tests in LLDB: https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/44239/changes#41d5033eb162cb92b684855166cabfa3983b74c6
I'm going to dig a little deeper, but I might ask you to revert this until we can figure out a solution.

Jun 2 2022, 9:29 AM · Restricted Project, Restricted Project, Restricted Project, Restricted Project

Jun 1 2022

aprantl added a reverting change for rGa544710cd43b: [Driver] Enable to use C++20 standalne by -fcxx-modules: rGd951ca5439bb: Revert "[Driver] Enable to use C++20 standalne by -fcxx-modules".
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project, Restricted Project
aprantl committed rGd951ca5439bb: Revert "[Driver] Enable to use C++20 standalne by -fcxx-modules" (authored by aprantl).
Revert "[Driver] Enable to use C++20 standalne by -fcxx-modules"
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project, Restricted Project
aprantl added a reverting change for rG738c20e6df01: [NFC] Use %clang instead of %clang++ in tests: rGc84b9bbac1ec: Revert "[NFC] Use %clang instead of %clang++ in tests".
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project
aprantl committed rGc84b9bbac1ec: Revert "[NFC] Use %clang instead of %clang++ in tests" (authored by aprantl).
Revert "[NFC] Use %clang instead of %clang++ in tests"
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project
aprantl added a reverting change for D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules: rGd951ca5439bb: Revert "[Driver] Enable to use C++20 standalne by -fcxx-modules".
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project
aprantl added a reverting change for rG35b1cfc76f08: [Driver][Modules] Remove dependence on linking support from…: rG128ffb332bca: Revert "[Driver][Modules] Remove dependence on linking support from….
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project
aprantl committed rG128ffb332bca: Revert "[Driver][Modules] Remove dependence on linking support from… (authored by aprantl).
Revert "[Driver][Modules] Remove dependence on linking support from…
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project
aprantl added a reverting change for D126669: [Driver][Modules] Remove dependence on linking support from clang/test/Driver/modules.cpp: rG128ffb332bca: Revert "[Driver][Modules] Remove dependence on linking support from….
Jun 1 2022, 12:12 PM · Restricted Project, Restricted Project
aprantl added a comment to D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules.

I'm going to revert the patch now. This is not just breaking LLDb, but also clang itself on Darwin platforms. I think we need to be more careful to separate out the enabling of Clang C++ modules and C++20 modules. Either by having -fmodules-ts control the HaveModules flag, or by adding a way to explicitly turn them off. I'm happy to help with testing out any patches!

Jun 1 2022, 12:08 PM · Restricted Project, Restricted Project
aprantl added a reverting change for rGca73de437445: Adapt LLDB for D120540.: rG62b448217595: Revert "Adapt LLDB for D120540.".
Jun 1 2022, 10:34 AM · Restricted Project
aprantl committed rG62b448217595: Revert "Adapt LLDB for D120540." (authored by aprantl).
Revert "Adapt LLDB for D120540."
Jun 1 2022, 10:34 AM · Restricted Project
aprantl added a comment to D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules.

In any case, I would appreciate it if we could revert this patch until we found a solution!

Jun 1 2022, 10:32 AM · Restricted Project, Restricted Project
aprantl added a comment to D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules.

Should we introduce a -fno-modules-ts option on top? Any better suggestions?

Jun 1 2022, 10:31 AM · Restricted Project, Restricted Project
aprantl added a comment to D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules.

I think the problem might be that previously on Darwin -fcxx-modules was used to turn on C++ Clang modules (which was not implied by -fmodules) without turning of C++20 (ts) modules, and after this patch -fcxx-modules implies -fmodules-ts. Does that sound plausible?

Jun 1 2022, 10:30 AM · Restricted Project, Restricted Project
aprantl added a comment to D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules.

https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/44167/

Jun 1 2022, 10:08 AM · Restricted Project, Restricted Project
aprantl added a comment to D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules.

This patch broke a whole bunch tests in the LLDB testsuite. I'm trying to figure out what exactly the semantics of the change are, particularly on Darwin, where -fmodules doesn't imply -fcxx-modules.

Jun 1 2022, 10:07 AM · Restricted Project, Restricted Project

May 31 2022

aprantl committed rGca73de437445: Adapt LLDB for D120540. (authored by aprantl).
Adapt LLDB for D120540.
May 31 2022, 4:57 PM · Restricted Project
aprantl accepted D125839: [gmodules] Skip CXXDeductionGuideDecls when visiting FunctionDecls in DebugTypeVisitor.

Adding the debug-info group. I don't really know enough about deduction guides to comment on the implications, but if it fixes the crash it seems to be a strict improvement. Maybe wait another day to see if someone else has an opinion on this.

May 31 2022, 1:33 PM · debug-info, Restricted Project, Restricted Project
aprantl added a project to D125839: [gmodules] Skip CXXDeductionGuideDecls when visiting FunctionDecls in DebugTypeVisitor: debug-info.
May 31 2022, 1:32 PM · debug-info, Restricted Project, Restricted Project

May 26 2022

aprantl added a comment to D121100: [clang][DebugInfo] clang should not generate DW_TAG_subprogram entry without DW_AT_name.

GNU gdb is now modified to accept functions with linkage name.

commit 6f9b09edaee43ea34d34b1998fe7b844834f251a
Author: Alok Kumar Sharma <AlokKumar.Sharma@amd.com>
Date:   Sun May 22 21:46:06 2022 +0530
May 26 2022, 6:24 PM · Restricted Project, Restricted Project, debug-info
aprantl added a comment to D88661: llvm-dva - Debug Information Visual Analizer.

This is exciting!
I understand where it comes from, but I wanted to at least ask if it would make sense to give the tool a more descriptive and discoverable name than llvm-dva such as llvm-debuginfo-analyzer. I understand there's a tradeoff between having a short command and a descriptive name.

May 26 2022, 6:21 PM · Restricted Project, Restricted Project, debug-info
aprantl accepted D125695: [clang][DebugInfo] Allow function-local type-like entities to be scoped within a lexical block (5/5).

Sgtm.

May 26 2022, 6:10 PM · debug-info, Restricted Project, Restricted Project
aprantl accepted D125694: [clang][DebugInfo] Allow function-local statics to be scoped within a lexical block (4/5).

This looks very good, thanks!

May 26 2022, 6:06 PM · debug-info, Restricted Project, Restricted Project

May 24 2022

aprantl added a comment to D123995: [MachineSink] replace MachineLoop with MachineCycle.

@shchenz Would you consider reverting your patch? Let me know if you have trouble reproducing the issue.

May 24 2022, 4:05 PM · Restricted Project, Restricted Project
aprantl added a comment to D123995: [MachineSink] replace MachineLoop with MachineCycle.

Another bot failure.
https://green.lab.llvm.org/green/view/Clang/job/clang-stage2-Rthinlto/5501/consoleFull#-125847874549ba4694-19c4-4d7e-bec5-911270d8a58c

May 24 2022, 1:48 PM · Restricted Project, Restricted Project
aprantl added a comment to D123995: [MachineSink] replace MachineLoop with MachineCycle.

It looks like this commit broke the -DLLVM_ENABLES_MODULES=1 build: https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/43994/changes

May 24 2022, 12:51 PM · Restricted Project, Restricted Project

May 20 2022

aprantl added a comment to D126080: Adapt C++ std::string dataformatter for D125496.

How do we usually add tests for STL data structures?

May 20 2022, 12:09 PM · Restricted Project, Restricted Project
aprantl added a comment to D126080: Adapt C++ std::string dataformatter for D125496.

That seemed to have done the trick.

May 20 2022, 12:07 PM · Restricted Project, Restricted Project
aprantl committed rGf4570ce442b4: Adapt C++ std::string dataformatter for D125496 (authored by aprantl).
Adapt C++ std::string dataformatter for D125496
May 20 2022, 11:26 AM · Restricted Project
aprantl closed D126080: Adapt C++ std::string dataformatter for D125496.
May 20 2022, 11:25 AM · Restricted Project, Restricted Project
aprantl added a comment to D126080: Adapt C++ std::string dataformatter for D125496.

I'm going to land this quickly to get the bots going again, but a thorough review would still be very much appreciated!

May 20 2022, 11:25 AM · Restricted Project, Restricted Project
aprantl added a comment to D125496: [libc++] Simplify the string structures a bit more.

Note that this broke the LLDB data formatters.

May 20 2022, 11:23 AM · Restricted Project, Restricted Project
aprantl requested review of D126080: Adapt C++ std::string dataformatter for D125496.
May 20 2022, 11:22 AM · Restricted Project, Restricted Project

May 16 2022

aprantl accepted D125474: [DWARFLinker][NFC] Add None value to the DwarfLinkerAccelTableKind enum..
May 16 2022, 3:31 PM · Restricted Project, Restricted Project
aprantl accepted D123534: [dwarf] Emit a DIGlobalVariable for constant strings..
May 16 2022, 3:29 PM · Restricted Project, Restricted Project, Restricted Project
aprantl committed rG9c7c8be4a316: Remove stale file from modulemap (authored by aprantl).
Remove stale file from modulemap
May 16 2022, 10:06 AM · Restricted Project, Restricted Project

May 13 2022

aprantl added a comment to D123534: [dwarf] Emit a DIGlobalVariable for constant strings..

I think this mostly looks good. I left comment about the test inline.

May 13 2022, 10:50 AM · Restricted Project, Restricted Project, Restricted Project
aprantl added inline comments to D123534: [dwarf] Emit a DIGlobalVariable for constant strings..
May 13 2022, 10:50 AM · Restricted Project, Restricted Project, Restricted Project

May 4 2022

aprantl added a comment to D124917: BitcodeReader: Avoid bad debug info warnings when stripping.

The debug info "upgrade" mechanism is unintuitive that I'm worried that this could subtly break intended behavior if it isn't tested.

May 4 2022, 10:24 AM · Restricted Project, Restricted Project
aprantl added a comment to D124917: BitcodeReader: Avoid bad debug info warnings when stripping.

I think agree with the idea of the patch. Can you please check that (1) there is an existing test that ensures warning is printed when not explicitly stripping debug info and (2) add a test for this case?

May 4 2022, 10:23 AM · Restricted Project, Restricted Project

May 3 2022

aprantl added inline comments to D124872: [lldb] Add a function to check if lldb is running in an interactive session.
May 3 2022, 3:08 PM · Restricted Project, Restricted Project
aprantl accepted D124872: [lldb] Add a function to check if lldb is running in an interactive session.

One last request: Could you add a Doxygen comment?

May 3 2022, 1:54 PM · Restricted Project, Restricted Project
aprantl requested changes to D124872: [lldb] Add a function to check if lldb is running in an interactive session.
May 3 2022, 1:35 PM · Restricted Project, Restricted Project
aprantl added inline comments to D124872: [lldb] Add a function to check if lldb is running in an interactive session.
May 3 2022, 1:35 PM · Restricted Project, Restricted Project
aprantl accepted D124872: [lldb] Add a function to check if lldb is running in an interactive session.

Thank you! This is equally useful and untestable :-)

May 3 2022, 1:33 PM · Restricted Project, Restricted Project

Apr 28 2022

aprantl accepted D123954: [lldb] Add setting for max depth of value object printing (NFC).

Few nitpicks inside otherwise I'm happy.

Apr 28 2022, 4:57 PM · Restricted Project, Restricted Project

Apr 27 2022

aprantl accepted D124568: [lldb] Fix escaping when launching in terminal with AppleScript.
Apr 27 2022, 4:36 PM · Restricted Project, Restricted Project
aprantl added a comment to D124568: [lldb] Fix escaping when launching in terminal with AppleScript.

I have some trivial suggestions inside, but otherwise, thanks!

Apr 27 2022, 3:59 PM · Restricted Project, Restricted Project
aprantl added inline comments to D124568: [lldb] Fix escaping when launching in terminal with AppleScript.
Apr 27 2022, 3:58 PM · Restricted Project, Restricted Project

Apr 22 2022

aprantl added inline comments to D123954: [lldb] Add setting for max depth of value object printing (NFC).
Apr 22 2022, 12:19 PM · Restricted Project, Restricted Project
aprantl added inline comments to D123954: [lldb] Add setting for max depth of value object printing (NFC).
Apr 22 2022, 10:49 AM · Restricted Project, Restricted Project
aprantl added inline comments to D124184: [DebugInfo][InstrRef] Describe the size of value referred to by a DBG_PHI.
Apr 22 2022, 10:39 AM · Restricted Project, Restricted Project
aprantl accepted D123469: [DebugInfo][llvm-dwarfutil] Combine overlapped address ranges..
Apr 22 2022, 10:36 AM · Restricted Project, Restricted Project
aprantl accepted D123787: [clang][OpenMP][DebugInfo] Debug support for TLS variables when present in OpenMP consructs.

Oh, it's a global variable that supposed to shadow the function parameter! Thanks for clarifying.

Apr 22 2022, 10:14 AM · Restricted Project, Restricted Project, debug-info

Apr 21 2022

aprantl accepted D123319: Change how we handle auto return types for lambda operator() to be consistent with gcc.

The new test looks good (I would replace the CHECK-NEXT with CHECK though).
It sounds like there was no objections for doing this for lambdas.

Apr 21 2022, 12:04 PM · Restricted Project, Restricted Project

Apr 15 2022

aprantl accepted D123697: [DebugInfo] Add a TargetFuncName field in DISubprogram..
Apr 15 2022, 1:24 PM · debug-info, Restricted Project, Restricted Project
aprantl added a comment to D123697: [DebugInfo] Add a TargetFuncName field in DISubprogram..

I would put it for example into the Fortran function information section and just add a source code example (if that makes sense) to explain under what cirumstances a frontend would want to put in a target function name.

Apr 15 2022, 9:56 AM · debug-info, Restricted Project, Restricted Project

Apr 14 2022

aprantl added a project to D123697: [DebugInfo] Add a TargetFuncName field in DISubprogram.: debug-info.
Apr 14 2022, 3:39 PM · debug-info, Restricted Project, Restricted Project
aprantl added a comment to D123697: [DebugInfo] Add a TargetFuncName field in DISubprogram..

Mechanically I think this looks fine, could you also update SourceLevelDebugging.rst?

Apr 14 2022, 3:37 PM · debug-info, Restricted Project, Restricted Project
aprantl added a comment to D123787: [clang][OpenMP][DebugInfo] Debug support for TLS variables when present in OpenMP consructs.

For an example, if thread local variable is present in copyin clause (testcase attached with the

patch), parameter with same name is generated as parameter to artificial function. When user
inquires the thread Local variable, its debug info is hidden by the parameter. The debug info
for parameters (for thread local) must be suppressed.

Apr 14 2022, 3:33 PM · Restricted Project, Restricted Project, debug-info
aprantl added inline comments to D123319: Change how we handle auto return types for lambda operator() to be consistent with gcc.
Apr 14 2022, 3:28 PM · Restricted Project, Restricted Project
aprantl added a comment to D123469: [DebugInfo][llvm-dwarfutil] Combine overlapped address ranges..

At first glance this looks fine to me. @JDevlieghere ?

Apr 14 2022, 3:21 PM · Restricted Project, Restricted Project