jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land.
That seems fine to me. We already only allow one thread to run when we make this call, and copyRealizedClassList is read only, so at worst this will crash if the list is getting edited, but it won't do any harm and we'll just try again later. That's a much better failure mode than potentially stranding the ObjC runtime lock. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104951/new/ https://reviews.llvm.org/D104951 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
