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

Javier Lopez-Gomez via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 16 10:27:08 PDT 2025


jalopezg-git wrote:

@CarlosAlbertoEnciso The `LLVM.tools/llvm-debuginfo-analyzer/COFF/01-coff-print-basic-details.test` test is failing (see [here](https://github.com/llvm/llvm-project/actions/runs/17772405724?pr=144750)).

This test uses `--output-sort=offset`; for CodeView, this might be ambiguous (see, e.g. [here](https://github.com/llvm/llvm-project/blob/main/llvm/lib/DebugInfo/LogicalView/Readers/LVCodeViewVisitor.cpp#L3138)).  I think `--output-sort=id` (https://github.com/llvm/llvm-project/pull/145761) has to be landed first to fix this test.

Other than that, a second round of review can already start :slightly_smiling_face: -- apologies for coming back to this a bit late.

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


More information about the llvm-commits mailing list