[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