[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