[llvm] 3eacaa3 - [ADT] Refactor DenseMapIterator::operator-> (NFC) (#157227)

via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 6 09:51:48 PDT 2025


Author: Kazu Hirata
Date: 2025-09-06T09:51:44-07:00
New Revision: 3eacaa3d1073a28e719e7d57f5aeb4267c4d9235

URL: https://github.com/llvm/llvm-project/commit/3eacaa3d1073a28e719e7d57f5aeb4267c4d9235
DIFF: https://github.com/llvm/llvm-project/commit/3eacaa3d1073a28e719e7d57f5aeb4267c4d9235.diff

LOG: [ADT] Refactor DenseMapIterator::operator-> (NFC) (#157227)

This patchs implement DenseMapIterator::operator-> in terms of
operator* to reduce code duplication.

Added: 
    

Modified: 
    llvm/include/llvm/ADT/DenseMap.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/ADT/DenseMap.h b/llvm/include/llvm/ADT/DenseMap.h
index 0650dc2044ca9..848e76feb20c6 100644
--- a/llvm/include/llvm/ADT/DenseMap.h
+++ b/llvm/include/llvm/ADT/DenseMap.h
@@ -1242,13 +1242,7 @@ class DenseMapIterator : DebugEpochBase::HandleBase {
       return Ptr[-1];
     return *Ptr;
   }
-  pointer operator->() const {
-    assert(isHandleInSync() && "invalid iterator access!");
-    assert(Ptr != End && "dereferencing end() iterator");
-    if (shouldReverseIterate<KeyT>())
-      return &(Ptr[-1]);
-    return Ptr;
-  }
+  pointer operator->() const { return &operator*(); }
 
   friend bool operator==(const DenseMapIterator &LHS,
                          const DenseMapIterator &RHS) {


        


More information about the llvm-commits mailing list