jingham added inline comments.

================
Comment at: lldb/source/Core/ValueObject.cpp:2846
+      if (llvm::isa<TypeSystemClang>(compiler_type.GetTypeSystem())) {
+        if (HasSyntheticValue()) {
+          TargetSP target_sp = GetTargetSP();
----------------
mib wrote:
> friss wrote:
> > I am understanding correctly that this condition will only trigger when 
> > there's a synthetic child provider for the type? In other words, if we have 
> > an opaque pointer that we don't know how to display, it will still cause an 
> > error, right?
> Yes, from my understanding, if the opaque pointer doesn't have any synthetic 
> child provider, it won't have a synthetic value and lldb should error saying 
> `error: dereference failed`.
We should add a test to make sure this stays true.  



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79554



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

Reply via email to