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

[Carlos Alberto Enciso via llvm-commits]

================
@@ -213,7 +211,17 @@ class LLVM_ABI LVScope : public LVElement {
   const LVScopes *getScopes() const { return Scopes.get(); }
   const LVSymbols *getSymbols() const { return Symbols.get(); }
   const LVTypes *getTypes() const { return Types.get(); }
-  const LVElements *getChildren() const { return Children.get(); }
+  // Return view over union of child Types, Symbols, and Scopes.
----------------
CarlosAlbertoEnciso wrote:

The comment implies the following concatenation order: Types, Symbols and Scopes.
But the `llvm::concat` uses `Scopes`, `Types` and `Symbols`.
May add a short note describing the order on which the elements are concatenated.

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