Stephan T. Lavavej

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

I've pushed additional commits: * Attempt to avoid timeouts in `P0896R4_ranges_alg_find_end`. + Encountered in the internal test harness, probably because the "checked" compiler consumes more memory. * Drop "assert bug...

:heart_eyes_cat: :tada: :cat2: :rocket: :cake: === Thank you @cpplearner and @CaseyCarter for the massive amount of work needed to get to this long-awaited day!

Yep, this is definitely a bug - plain shortest is different from `chars_format::general`. Thanks for analyzing this!

Thanks, I'll pick that up and see what happens.

As @frederick-vs-ja mentioned in Discord, we can close this now. The STL has been fixed with #3364 (further patched by #3982) and the LLVM submodule has been updated by #4702...

We talked about this at the weekly maintainer meeting and we believe that we should Do What The Standard Says(TM) here, *except* when the Standard depicts deleted move operations (@CaseyCarter...

(Transferred this comment's info to the issue above.)

Changing a runtime test to contain this: ```cpp // Copyright (c) Microsoft Corporation. // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception #include using namespace std; int main() { puts("HEAP CORRUPTION DETECTED: before \x8d");...

I believe that changing the values of these enumerators will break ABI; labeling as `vNext`.