clayborg wrote:

> This seems like a much wider ranging change than is required to fix this bug. 
> I'm all for rethinking the 
> ExecutionContextScope/ExecutionContext/ExecutionContextRef nexus, but I don't 
> think that's a trivial rethink, and I don't have the time to undertake that 
> right now.

My only point in mentioning this is many of the changed files in your patch 
goes away if we fix ExecutionContext. 

Change look fine to me if we don't want to change ExecutionContext, but i will 
let others do the final accept.


https://github.com/llvm/llvm-project/pull/65822
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to