Don't include <algorithm> in <chrono>

[frederick-vs-ja]

Separated from #3623. Towards #3599.

It seems to me that we don't need to use transform for tzdb_list. emplace_back in loop looks better.

[strega-nil-ms]

This is a good change, imo.

[StephanTLavavej]

Agreed! Even without the throughput considerations, this is more efficient and easier to understand. The reserve() calls are safe (note that careless use of reserve() can lead to quadratic complexity, but not in the pattern here.)

I checked all of <chrono>'s _STD calls, and I see no other <algorithm> dependencies. We're using _Copy_unchecked, fill, find_if, and upper_bound from <xutility>, as they are widely needed.

Finally, this is unlikely to be an impactful breaking change, as it's limited to C++20.

[StephanTLavavej]

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

[StephanTLavavej]

Thanks for finding and making this major improvement! <algorithm> is huge so this should be a nice win. 🎉 📉 😻