llvm-project.git/llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp, branch users/nico/python-2

Revert gsymutil changes due to concurrency problems (#142829)

2025-06-04T18:34:12+00:00

We saw occasional segfaults while processing some binaries. The reason
probably is that we may clear the DIE while we are reading it's data
from another thread which happens due to cross-unit references.

---------

Co-authored-by: Arslan Khabutdinov

Reduce llvm-gsymutil memory usage (#140740)

2025-05-21T16:49:12+00:00

Same as https://github.com/llvm/llvm-project/pull/139907/ except there
is now a special dovoidwork helper function.
Previous approach with assert(f();return success;) failed tests for
release builds, so I created a separate helper. Open to suggestions how
to solve this more elegantly.

Co-authored-by: Arslan Khabutdinov

Revert "Reduce llvm-gsymutil memory usage" (#140696)

2025-05-20T11:31:39+00:00

Reverts llvm/llvm-project#139907 as per discussion in
https://github.com/llvm/llvm-project/issues/140545 due to tests becoming
flaky

Reduce llvm-gsymutil memory usage (#139907)

2025-05-16T16:44:17+00:00

For large binaries gsymutil ends up using too much memory. This diff
adds DIE tree cleanup per compile unit to reduce memory usage.

P. S. Not sure about formatting. Maybe it hasn't been run in a while, or
I have misconfigured something.

`$ git clang-format HEAD~1
clang-format did not modify any files
$ clang-format --version
clang-format version 21.0.0git
(git@github.com:peremyach/llvm-project.git
8d945c8357e1bd9872a34f92620d4916bfd27482)
`

Co-authored-by: Arslan Khabutdinov

[llvm] Use StringRef::{starts_with,find} (NFC) (#139661)

2025-05-13T06:24:07+00:00

Calling find/contains in the StringRef domain allows us to avoid
creating temporary instances of std::string.

[llvm] Use llvm::append_range (NFC) (#135931)

2025-04-16T19:28:47+00:00

[NFC][DebugInfo] Wrap DILineInfo return type with std::optional to handle missing debug info. (#129792)

2025-03-17T21:01:06+00:00

Currently, `DIContext::getLineInfoForAddress` and
`DIContext::getLineInfoForDataAddress` returns empty DILineInfo when the
debug info is missing for the given address. This is not differentiable
with the case when debug info is found for the given address but the
debug info is default value (filename:linenum is :0).

This change wraps the return types of `DIContext::getLineInfoForAddress`
and `DIContext::getLineInfoForDataAddress` with `std::optional`.

[GSYM] Fix incorrect comparison in gSYM creation (#131197)

2025-03-14T16:13:08+00:00

There is a bug in `llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp` where
`StmtSeqVal` was being compared against `UINT32_MAX` rather than the
correct `UINT64_MAX` - thanks @nocchijiang for [pointing this
out](https://github.com/llvm/llvm-project/pull/129196#discussion_r1986203058).

We correct the issue with this patch. For testing - the issue would show
when we have a correct offset of value `UINT32_MAX` - but constructing
such a test is impractical.

[GSYM] Use debug line offsets during GSYM creation (#129196)

2025-03-03T19:22:16+00:00

This patch introduces support for the `DW_AT_LLVM_stmt_sequence`
attribute in the GSYM DWARF transformer. With this change, the DWARF
GSYM creation process can now accurately associate debug information
with the correct functions, even when multiple functions have been
merged together.

The `macho-gsym-merged-callsites-dsym.yaml` test data is regenerated to
include the fixes in the DWARF linker
(https://github.com/llvm/llvm-project/pull/128953) and the test is
updated to check that debug data is correctly associated for merged
functions.

[llvm-gsymutil] Add option to load callsites from DWARF (#119913)

2024-12-17T16:51:30+00:00

This change adds support for loading gSYM callsite information from
DWARF. Previously the only support was for loading callsites info from
YAML.

For testing, we add a pass where `macho-gsym-merged-callsites-dsym`
loads callsite info from DWARF rather than YAML.