[libcxx-commits] [PATCH] D59572: Fix and speedup __libcpp_locale_guard on Windows

[Tom Anderson via Phabricator via libcxx-commits]

thomasanderson added a comment.

In D59572#1440417 <https://reviews.llvm.org/D59572#1440417>, @EricWF wrote:

> That said, this feels like a POSIX-ism we're leaking into our Windows implementation. So I'm open to doing whatever is more Windows-like.


Ideally we shouldn't have to use `__libcpp_locale_guard` at all.  I think its only use right now is for internally converting ints (and other numbers) to strings by setting the locale to "C", calling some variant of sprintf("%d"), and restoring the locale.  This seems like a 100 lb solution to a 1 lb problem.  setlocale() and the printf() family of functions are slow.  I think it would be much faster and not very difficult to just implement this in libc++.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59572/new/

https://reviews.llvm.org/D59572