Skip to content

[libc++][NFC] Remove some unused code from <map> #142408

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

philnik777
Copy link
Contributor

No description provided.

@philnik777 philnik777 marked this pull request as ready for review June 6, 2025 13:26
@philnik777 philnik777 requested a review from a team as a code owner June 6, 2025 13:27
@philnik777 philnik777 merged commit a925e90 into llvm:main Jun 6, 2025
172 of 191 checks passed
@philnik777 philnik777 deleted the map-remove_unused_code branch June 6, 2025 13:27
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Jun 6, 2025
@llvmbot
Copy link
Member

llvmbot commented Jun 6, 2025

@llvm/pr-subscribers-libcxx

Author: Nikolas Klauser (philnik777)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/142408.diff

1 Files Affected:

  • (modified) libcxx/include/map (-12)
diff --git a/libcxx/include/map b/libcxx/include/map
index 24eadbd154220..8c5799fbe6f10 100644
--- a/libcxx/include/map
+++ b/libcxx/include/map
@@ -750,21 +750,9 @@ public:
   }
 };
 
-template <class _TreeIterator>
-class __map_const_iterator;
-
 template <class _Key, class _Tp>
 struct __value_type;
 
-template <class _Tp>
-struct __extract_key_value_types;
-
-template <class _Key, class _Tp>
-struct __extract_key_value_types<__value_type<_Key, _Tp> > {
-  typedef _Key const __key_type;
-  typedef _Tp __mapped_type;
-};
-
 template <class _TreeIterator>
 class __map_iterator {
   typedef typename _TreeIterator::_NodeTypes _NodeTypes;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants