[llvm] [llvm-debuginfo-analyzer] Remove `LVScope::Children` container (PR #144750)

Javier Lopez-Gomez via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 19 06:09:33 PDT 2025


jalopezg-git wrote:

@CarlosAlbertoEnciso This is finally ready for review!

Let me clarify the changes in `llvm/test/tools/llvm-debuginfo-analyzer/COFF/01-coff-print-basic-details.test` and `llvm/test/tools/llvm-debuginfo-analyzer/DWARF/01-dwarf-print-basic-details.test`:
- After https://github.com/llvm/llvm-project/pull/145761, `--output-sort=id` can be used to compare against reference output.  For CodeView, this avoids issues w/ `LVElement`s that have same `offset` value (see, e.g. https://github.com/llvm/llvm-project/blob/main/llvm/lib/DebugInfo/LogicalView/Readers/LVCodeViewVisitor.cpp#L3138).
- I have reordered lines a bit to leave the `--output-sort=none` test at the end.

Let me know what you think!

https://github.com/llvm/llvm-project/pull/144750


More information about the llvm-commits mailing list