================
@@ -71,6 +66,11 @@ class OptionValueArch : public Cloneable<OptionValueArch, 
OptionValue> {
   void SetDefaultValue(const ArchSpec &value) { m_default_value = value; }
 
 protected:
+  void ClearImpl() override {
----------------
da-viper wrote:

Dropped the `ClearImpl` and called the recursive mutex in the derived classes. 
I initially didn't want to expose the mutex but I realised that some of the 
derived classes may change values not available in the base `OptionValue` and 
would require a lock. 

https://github.com/llvm/llvm-project/pull/208471
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to