[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