[libcxx-commits] [libcxx] [libc++] Implement P3168R2: Give optional range support (PR #149441)

[William Tran-Viet via libcxx-commits]

smallp-o-p wrote:

> The patch LGTM as-is or with the `__optional_iterator_aliases` removed per @philnik777 's comment. Either way looks fine to me.
> 
> What I care strongly about is that we then tackle the iterator type confusion problem.
> 
> Thanks a lot for the patch, this is a great (first?) contribution.

Technically second, but we'll count the first one as part of this one. Thanks for taking the time to review this!

(Also, who would I need to ping to get this merged?)

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