[Lldb-commits] [PATCH] D104951: [lldb] Use the non-locking variant of objc_copyRealizedClassList

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 25 14:19:04 PDT 2021


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



More information about the lldb-commits mailing list