================
@@ -440,12 +440,6 @@ static void GetTypeLookupContextImpl(DWARFDIE die,
       continue;
     }
 
-    // If there is no name, then there is no need to look anything up for this
-    // DIE.
-    const char *name = die.GetName();
-    if (!name || !name[0])
-      return;
-
----------------
labath wrote:

unnamed structs cannot contain [named 
structs](https://godbolt.org/z/sEfdMEvsz), so the only place these can appear 
is at the top (bottom?) level. We don't currently have a way to look them up 
from our APIs taking strings (we'd need something like an `(anonymous 
struct)`), but they could be looked up internally by code which constructs 
CompilerContexts directly. So yeah, I think including them here is the right 
choice for this function.

https://github.com/llvm/llvm-project/pull/102111
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to