jdemeule added a comment. In D43500#1546089 <https://reviews.llvm.org/D43500#1546089>, @lebedev.ri wrote:
> In D43500#1546077 <https://reviews.llvm.org/D43500#1546077>, @jdemeule wrote: > > > In D43500#1244518 <https://reviews.llvm.org/D43500#1244518>, @JonasToth > > wrote: > > > > > What is the status here? Will you continue to work on this patch > > > @jdemeule (which would be great!) ? > > > > > > Hi, now, this patch contains only tests which are already covered with lit > > tests. > > The cleanup is now applied in clang-apply-replacements. > > > How does that work with `clang-tidy -fix` ? If I remember by calling 'format::cleanupAroundReplacements' when applying the fix in memory. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D43500/new/ https://reviews.llvm.org/D43500 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits