[Lldb-commits] [lldb] Improve type and namespace lookup using parent chain (PR #108907)

[Felipe de Azevedo Piovezan via lldb-commits]

================
@@ -402,6 +420,36 @@ bool DebugNamesDWARFIndex::SameParentChain(
   return true;
 }
 
+bool DebugNamesDWARFIndex::WithinParentChain(
+    llvm::ArrayRef<llvm::StringRef> query_parent_names,
+    llvm::ArrayRef<DebugNames::Entry> parent_chain) const {
+  if (parent_chain.size() < query_parent_names.size())
+    return false;
+
+  size_t query_idx = 0, chain_idx = 0;
+  while (query_idx < query_parent_names.size() &&
+         chain_idx < parent_chain.size()) {
+    if (SameAsEntryATName(query_parent_names[query_idx],
+                          parent_chain[chain_idx])) {
+      ++query_idx;
+      ++chain_idx;
+    } else {
+      // Name does not match, try next parent_chain entry if the current entry
+      // is namespace because the current one can be an inline namespace.
----------------
felipepiovezan wrote:

do we have a way of guaranteeing it *is*?
If the comparison that failed to match is with a real (non inline) namespace, we would return a false result here.

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