[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.
+  auto getChildren() const {
+    return llvm::concat<LVElement *const>(Scopes ? *Scopes : EmptyScopes,
+                                          Types ? *Types : EmptyTypes,
+                                          Symbols ? *Symbols : EmptySymbols);
+  }
+  // Return view over union of child Types, Symbols, and Scopes that is sorted
----------------
CarlosAlbertoEnciso wrote:

Same comment about the concatenation order.

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