[PATCH] D15121: A new clang-tidy module to find calls to `std::swap`, and change them to use ADL

Marshall Clow via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 3 08:08:25 PST 2015


mclow.lists added a comment.

I ran this on LLVM + clang, and it changed calls in 61 files.  The changed LLVM codebase compiled successfully, and passed all the tests.

Re: @dblaikie's comment, I'd rather call such a beast `llvm::swap`, and it would have to go into a header file that everyone already includes.


http://reviews.llvm.org/D15121





More information about the cfe-commits mailing list