[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