[Lldb-commits] [PATCH] D63240: [Core] Generalize ValueObject::IsRuntimeSupportValue

[Adrian Prantl via Phabricator via lldb-commits]

aprantl added inline comments.


================
Comment at: include/lldb/Target/LanguageRuntime.h:157
+  virtual bool IsRuntimeSupportValue(ValueObject &valobj) {
+    return valobj.GetVariable() && valobj.GetVariable()->IsArtificial();
+  }
----------------
I think this function should not be part of LanguageRuntime any more since there is nothing runtime-specific about it any more.Instead, it should probably be a function implemented by ValueObjectVariable. The Whitelist make still sense in the runtime of course.


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

https://reviews.llvm.org/D63240