Thanks again for this amazing work which I would not have been able to accomplish! 😻 I pushed minor nitpicky changes, nothing substantial (FYI @strega-nil-ms anyways).

With the addition of the charconv test cases, and the "if it doesn't match, at least it should match strtod/strtof" workaround, I am much more confident that we won't be damaging correctness here. Thanks for making your test changes totally non-intrusive too!

The change is still higher risk than most changes, but I believe we're preserving ABI here, and this is a significant improvement over the status quo. Let's do it.

I've had to push an additional commit to restore the dllexport surface.

_Parse_fp_with_locale() needs to be templated, otherwise it'll be dllexported when the whole class is.

_Getffld() and _Getffldx() needed special treatment - their static constexpr char _Src[] arrays were implicitly dllexported, which shows up separately from the functions themselves. Accordingly, we need to restore the arrays even though they're unused, and we need to prevent the compiler eliding them, so I'm storing the address of the first element in a top-level volatile pointer, which is sufficient to keep the array in the DLL. (This does not work without volatile.)

@frederick-vs-ja The compiler back-end's internal test suite (specifically spec2k eon) found a bug in this PR:

D:\GitHub\STL\out\x64>type meow.cpp

#include <cstdio>
#include <sstream>
using namespace std;

int main() {
    printf("_MSVC_STL_UPDATE: %d\n", static_cast<int>(_MSVC_STL_UPDATE));
    istringstream iss{"0\n"};
    float flt = 1729.0f;
    iss >> flt;
    if (iss) {
        printf("flt: %.1000g\n", flt);
    } else {
        printf("Parsing failure!\n");
    }
}

Before this PR (VS 2022 17.6 Preview 5):

D:\GitHub\STL\out\x64>cl /EHsc /nologo /W4 /MTd /Od meow.cpp && meow
meow.cpp
_MSVC_STL_UPDATE: 202302
flt: 0

After this PR:

D:\GitHub\STL\out\x64>cl /EHsc /nologo /W4 /MTd /Od meow.cpp && meow
meow.cpp
_MSVC_STL_UPDATE: 202304
Parsing failure!

We definitely need test coverage for this scenario.

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

Thank you for this absolutely massive correctness fix and LWG issue resolution, which the maintainers would not have been able to accomplish! We really appreciate it! 😻 🎉 🚀