mikecrowe added a comment. In D144216#4135431 <https://reviews.llvm.org/D144216#4135431>, @carlosgalvezp wrote:
> Fair enough! I think we should merge this patch as is first, so that people > can benefit from it already, and work through the technical debt afterwards. Great. Thank you. I'm still interested in answers to the questions in my comment since I have the necessary changes to migrate the tests I mentioned to using `<string>` ready and I'd be happy to have a go at migrating more. Thanks. Mike. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144216/new/ https://reviews.llvm.org/D144216 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
