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

[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 can't see a reason why you need to friend ScriptInterpreter for this patch.  And it seems wrong for the SBValue to need to friend ScriptInterpreter anyway.  Is this a bleed-through of some larger patch that this was extracted from?

If so, can we move it from here to the future patch that requires it?

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