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

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 28 12:16:35 PDT 2018


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.

On Fri, Sep 28, 2018, 16:23 Kirill Bobyrev via Phabricator <
reviews at reviews.llvm.org> wrote:

> kbobyrev added a comment.
>
> Also, I'm not sure whether I should update Clang-Tidy and other tools in
> the scope of this patch. It makes sense to me, but I don't know whether the
> maintainers of these projects are happy with the change. WDYT?
>
>
> https://reviews.llvm.org/D52650
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180928/35c09aa5/attachment.html>


More information about the cfe-commits mailing list