[PATCH] D28726: Adding const overloads of operator* and operator-> for DenseSet iterators

[David Majnemer via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL292034: Adding const overloads of operator* and operator-> for DenseSet iterators (authored by majnemer).

Changed prior to commit:
  https://reviews.llvm.org/D28726?vs=84436&id=84464#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D28726

Files:
  llvm/trunk/include/llvm/ADT/DenseSet.h


Index: llvm/trunk/include/llvm/ADT/DenseSet.h
===================================================================
--- llvm/trunk/include/llvm/ADT/DenseSet.h
+++ llvm/trunk/include/llvm/ADT/DenseSet.h
@@ -104,7 +104,9 @@
     Iterator(const typename MapTy::iterator &i) : I(i) {}
 
     ValueT &operator*() { return I->getFirst(); }
+    const ValueT &operator*() const { return I->getFirst(); }
     ValueT *operator->() { return &I->getFirst(); }
+    const ValueT *operator->() const { return &I->getFirst(); }
 
     Iterator& operator++() { ++I; return *this; }
     Iterator operator++(int) { auto T = *this; ++I; return T; }
@@ -125,8 +127,8 @@
 
     ConstIterator(const typename MapTy::const_iterator &i) : I(i) {}
 
-    const ValueT &operator*() { return I->getFirst(); }
-    const ValueT *operator->() { return &I->getFirst(); }
+    const ValueT &operator*() const { return I->getFirst(); }
+    const ValueT *operator->() const { return &I->getFirst(); }
 
     ConstIterator& operator++() { ++I; return *this; }
     ConstIterator operator++(int) { auto T = *this; ++I; return T; }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28726.84464.patch
Type: text/x-patch
Size: 1097 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170114/e4bd34dc/attachment.bin>