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