tbaeder added a comment. Generally LGTM, but the `erase` call could do with a comment before. And I'll let someone with more concepts experience handle whether this should be accepted or not.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157526/new/ https://reviews.llvm.org/D157526 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits