labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
Looks fine to me. I guess this would be the place I'd start thinking about an `OptionalSize` class (to replace the two fields in the Type class). But I don't think that's necessary yet... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57273/new/ https://reviews.llvm.org/D57273 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
