clayborg added a comment.

Or do we

In D69820#1817768 <https://reviews.llvm.org/D69820#1817768>, @xiaobai wrote:

> In D69820#1817621 <https://reviews.llvm.org/D69820#1817621>, @clayborg wrote:
>
> > Let me know what everyone thinks of adding a "fully_qualified" argument to 
> > the TypeSystem::GetClassName()?
>
>
> I think that you would have to add more than a `fully_qualified` argument to 
> the method. You need to reference the parent `ValueObject` to fully qualify 
> something as far as I can tell, so you'd have to pass along the parent as 
> well. I think that it sounds nice for the `TypeSystem` to fully qualify the 
> class name but I don't think that it's necessarily better than what exists 
> now. You'll still have recursion walking the parent `ValueObject`s, you've 
> just split it across two different methods in different classes. Maybe I'm 
> overthinking this or missing something though?


Just for the type I am saying. Get the fully qualified name of the type. That 
should be all built into the type in the type system as well right? So no other 
value would need to be used, just ask the CompilerType for its fully qualified 
name.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69820/new/

https://reviews.llvm.org/D69820



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to