[PATCH] D52650: [clangd] NFC: Migrate to LLVM STLExtras API where possible

Kirill Bobyrev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 28 13:48:43 PDT 2018


kbobyrev added a comment.

In https://reviews.llvm.org/D52650#1249556, @sammccall wrote:

> I think it's fine to update others too, this is a trivially-safe change and
>  a nice readability improvement.
>  No need to put everything in the same patch though.


Great, thanks! I'll submit the patch once I'm home and I will make modifications in the other parts of clang-tools-extra.

Also, this making these transformations automatically might be a great idea for Clang-Tidy llvm-check (and a separate check with std::ranges later on).


https://reviews.llvm.org/D52650





More information about the cfe-commits mailing list