[PATCH] clang-apply-replacements: Add code-formatting post processing step

David Blaikie dblaikie at gmail.com
Thu Sep 26 12:55:16 PDT 2013


Not a big deal either way, but like Manuel, I prefer the return by value -
it avoids any confusion about whether it's append or clear/initialize
semantics, etc. And even in the absence of C++11 (though I believe the
general idea is that we care that LLVM projects /build/ with various
compliant C++98 compilers, but only really care about their performance
when built with Clang as C++11) this is trivially NRVOable.


On Thu, Sep 26, 2013 at 12:45 PM, Manuel Klimek <klimek at google.com> wrote:

>
>   @revane: happy to get a 3rd opinion and let that decide ;)
>
> http://llvm-reviews.chandlerc.com/D1730
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130926/9644e0cf/attachment.html>


More information about the cfe-commits mailing list