aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
In D150840#4359570 <https://reviews.llvm.org/D150840#4359570>, @tbaeder wrote: > Side-note: The `ToPrint` stuff here is just useful to "cache" the output > string, isn't it? Is that really useful? I'd expect the output stream to be > cached anyway? Yeah, that does seem like a kind of caching operation, I'm not certain what it's purpose is though. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150840/new/ https://reviews.llvm.org/D150840 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits