[PATCH] D24689: [ADT] Add missing const_iterator DenseSet::find() const

[David Blaikie via llvm-commits]

Worth a unit test? (could use the type based testing in GUnit to simplify
it/do the same test for const and non-const)

On Fri, Sep 16, 2016 at 4:07 PM Vitaly Buka via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> vitalybuka created this revision.
> vitalybuka added a reviewer: eugenis.
> vitalybuka added a subscriber: llvm-commits.
>
> Probably overlooked.
>
> https://reviews.llvm.org/D24689
>
> Files:
>   include/llvm/ADT/DenseSet.h
>
> Index: include/llvm/ADT/DenseSet.h
> ===================================================================
> --- include/llvm/ADT/DenseSet.h
> +++ include/llvm/ADT/DenseSet.h
> @@ -135,6 +135,7 @@
>    const_iterator end() const { return ConstIterator(TheMap.end()); }
>
>    iterator find(const ValueT &V) { return Iterator(TheMap.find(V)); }
> +  const_iterator find(const ValueT &V) const { return
> ConstIterator(TheMap.find(V)); }
>
>    /// Alternative version of find() which allows a different, and
> possibly less
>    /// expensive, key type.
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160919/55ffe4a0/attachment.html>