JDevlieghere marked 11 inline comments as done. JDevlieghere added a comment.
In D56322#1382971 <https://reviews.llvm.org/D56322#1382971>, @labath wrote: > Thank you. I think this looks much better now. > > It occurred to me that the (de)serializer classes are fully standalone. Since > they already have a comprehensive test suite, do you think it would make > sense to split them off into a separate patch, which can be committed > separately? Sure, makes sense to me, please see D57714 <https://reviews.llvm.org/D57714> CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56322/new/ https://reviews.llvm.org/D56322 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
