hokein added a comment. In https://reviews.llvm.org/D34206#790406, @alexfh wrote:
> In https://reviews.llvm.org/D34206#783673, @hokein wrote: > > > In https://reviews.llvm.org/D34206#780455, @Eugene.Zelenko wrote: > > > > > It'll be good idea to run modernize-make-unique on LLVM/Clang/etc for > > > llvm::make_unique. > > > > > > +1. See https://reviews.llvm.org/D34334, https://reviews.llvm.org/D34333. > > And found a few bugs in the check, will work on fixes on them. > > > Ping me once you're done fixing the bugs. I have a fix patch under review: https://reviews.llvm.org/D34286, and the other will come afterwards. I think it is fine to check in this patch before these two bugs are fixed. https://reviews.llvm.org/D34206 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
