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 <[email protected]> wrote: > > @revane: happy to get a 3rd opinion and let that decide ;) > > http://llvm-reviews.chandlerc.com/D1730 > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
