use _Make_unsigned_like_t instead of make_unsigned_t for _Max_possible_v

[Igor Zhukov (fsb4000)]

Fixes #3025

Yes, I saw A. Jiang (@frederick-vs-ja) 's comment and we can change every use _Make_unsigned_like_t to make_unsigned_t when the C++ standard will be changed.

[Stephan T. Lavavej (StephanTLavavej)]

Thanks! I pushed a conflict-free merge with main, followed by a rename to avoid shadowing std::prev, moving the test to the views::iota test, and testing it both at run-time and compile-time.

[nicole mazzuca (strega-nil-ms)]

Thanks for this!

[Stephan T. Lavavej (StephanTLavavej)]

Casey Carter (@CaseyCarter) nicole mazzuca (@strega-nil-ms) I pushed your suggested change. I refrained from adding a _To_unsigned_like() fallback function, but we'll notice if we ever need it in the future and it can easily be added then.

[Stephan T. Lavavej (StephanTLavavej)]

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

[Stephan T. Lavavej (StephanTLavavej)]

I had to push an additional commit to work around the previously reported VSO-1666180 "/clr C++20 x64 System.AccessViolationException with views::iota and views::transform".

[Stephan T. Lavavej (StephanTLavavej)]

Thanks for fixing this bug and improving ranges usability! 🐞 🛠️ 😸