[libcxx-commits] [libcxx] [libc++] Simplify overload sets where C::iterator == C::const_iterator (PR #66080)

[Louis Dionne via libcxx-commits]

ldionne wrote:

I'm very reluctant to do this. We don't want to bake in the assumption that our `iterator`s and `const_iterator`s are the same, even though that's the case right now. Furthermore, even though this patch removes some code, I think it makes the code more complicated because it makes it differ more from the Standard specification. Now, every time we make a change to these containers, people will be wondering why half of the functions are seemingly missing.

I think we could consider doing this *if* there was a large compile-time (or other) benefit, but it's not clear to me that this is the case. I'll close this for now, but please reopen with more information if you still want to pursue this.

https://github.com/llvm/llvm-project/pull/66080