teemperor accepted this revision. teemperor added a comment. This revision is now accepted and ready to land. Herald added a subscriber: JDevlieghere.
LGTM. I think the Read/Write methods should return a Status instead of some out parameter, but that seems like a NFC follow-up refactoring (I can take care of that). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98170/new/ https://reviews.llvm.org/D98170 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits