[Lldb-commits] [lldb] [lldb] Move ValueImpl and ValueLocker to ValueObject, NFC. (PR #178573)

[via lldb-commits]

================
@@ -506,8 +505,11 @@ class LLDB_API SBValue {
   void SetSP(const lldb::ValueObjectSP &sp, lldb::DynamicValueType use_dynamic,
              bool use_synthetic, const char *name);
 
+protected:
----------------
jimingham wrote:

I made the same comment on the previous version of the PR, but this "friend" statement doesn't seem germane to this patch.

Can you either explain why it's necessary here, or move this into the subsequent PR where it really belongs.  

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