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

[Javier Lopez-Gomez via llvm-commits]

================
@@ -35,8 +35,8 @@
 ; ONE-NEXT:  [002]     1         {TypeAlias} 'INTPTR' -> '* const int'
 ; ONE-NEXT:  [002]     2         {Function} extern not_inlined 'foo' -> 'int'
 ; ONE-NEXT:  [003]                 {Block}
-; ONE-NEXT:  [004]     5             {Variable} 'CONSTANT' -> 'const INTEGER'
 ; ONE-NEXT: +[004]     4             {TypeAlias} 'INTEGER' -> 'int'
----------------
jalopezg-git wrote:

Hmmm... that is weird, given that for comparison mode sort mode is `line` (see [LVOptions.cpp:219](https://github.com/llvm/llvm-project/blob/main/llvm/lib/DebugInfo/LogicalView/Core/LVOptions.cpp#L219)), unless I am mistaken, `INTEGER` should have come before `CONSTANT`, but it was not the case.  Was this (partially) broken before?
FYI, @CarlosAlbertoEnciso.
```
; ONE-NEXT:  [004]     5             {Variable} 'CONSTANT' -> 'const INTEGER'
; ONE-NEXT: +[004]     4             {TypeAlias} 'INTEGER' -> 'int'
```

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