Re: [PATCH] libstdc++: Ensure headers used by fast_float are included
On Wed, Apr 12, 2023 at 11:04 AM Jonathan Wakely wrote: > > On Wed, 12 Apr 2023 at 14:45, Patrick Palka via Libstdc++ > wrote: > > > > This makes floating_from_chars.cc explicitly include all headers > > that are used by the original fast_float amalgamation according to > > r12-6647-gf5c8b82512f9d3, except: > > > > 1. since fast_float doesn't seem to use anything from it > > 2. since fast_float doesn't seem to use anything directly > > from it (as opposed to from ) > > 3. since std::errc is naturally already available > > from > > > > This avoids potential build failures on platforms for which some > > required headers (namely ) end up not getting transitively > > included from elsewhere. > > > > Tested on x86_64-pc-linux-gnu, does this look OK for trunk/12? > > Yes for both, thanks. Thanks, pushed as r13-7159-g13669111e7219e and r12-9396-g43ab94d20e1f68 > > > > > > libstdc++-v3/ChangeLog: > > > > * src/c++17/floating_from_chars.cc: Include , > > , and . > > --- > > libstdc++-v3/src/c++17/floating_from_chars.cc | 4 > > 1 file changed, 4 insertions(+) > > > > diff --git a/libstdc++-v3/src/c++17/floating_from_chars.cc > > b/libstdc++-v3/src/c++17/floating_from_chars.cc > > index 5d18ca32dbb..3a411cf546a 100644 > > --- a/libstdc++-v3/src/c++17/floating_from_chars.cc > > +++ b/libstdc++-v3/src/c++17/floating_from_chars.cc > > @@ -30,14 +30,18 @@ > > // Prefer to use std::pmr::string if possible, which requires the cxx11 > > ABI. > > #define _GLIBCXX_USE_CXX11_ABI 1 > > > > +#include > > #include > > #include > > #include > > +#include > > +#include > > #include > > #include > > #include > > #include > > #include > > +#include > > #include > > #include > > #include > > -- > > 2.40.0.315.g0607f793cb > > >
Re: [PATCH] libstdc++: Ensure headers used by fast_float are included
On Wed, 12 Apr 2023 at 14:45, Patrick Palka via Libstdc++ wrote: > > This makes floating_from_chars.cc explicitly include all headers > that are used by the original fast_float amalgamation according to > r12-6647-gf5c8b82512f9d3, except: > > 1. since fast_float doesn't seem to use anything from it > 2. since fast_float doesn't seem to use anything directly > from it (as opposed to from ) > 3. since std::errc is naturally already available > from > > This avoids potential build failures on platforms for which some > required headers (namely ) end up not getting transitively > included from elsewhere. > > Tested on x86_64-pc-linux-gnu, does this look OK for trunk/12? Yes for both, thanks. > > libstdc++-v3/ChangeLog: > > * src/c++17/floating_from_chars.cc: Include , > , and . > --- > libstdc++-v3/src/c++17/floating_from_chars.cc | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libstdc++-v3/src/c++17/floating_from_chars.cc > b/libstdc++-v3/src/c++17/floating_from_chars.cc > index 5d18ca32dbb..3a411cf546a 100644 > --- a/libstdc++-v3/src/c++17/floating_from_chars.cc > +++ b/libstdc++-v3/src/c++17/floating_from_chars.cc > @@ -30,14 +30,18 @@ > // Prefer to use std::pmr::string if possible, which requires the cxx11 ABI. > #define _GLIBCXX_USE_CXX11_ABI 1 > > +#include > #include > #include > #include > +#include > +#include > #include > #include > #include > #include > #include > +#include > #include > #include > #include > -- > 2.40.0.315.g0607f793cb >
[PATCH] libstdc++: Ensure headers used by fast_float are included
This makes floating_from_chars.cc explicitly include all headers that are used by the original fast_float amalgamation according to r12-6647-gf5c8b82512f9d3, except: 1. since fast_float doesn't seem to use anything from it 2. since fast_float doesn't seem to use anything directly from it (as opposed to from ) 3. since std::errc is naturally already available from This avoids potential build failures on platforms for which some required headers (namely ) end up not getting transitively included from elsewhere. Tested on x86_64-pc-linux-gnu, does this look OK for trunk/12? libstdc++-v3/ChangeLog: * src/c++17/floating_from_chars.cc: Include , , and . --- libstdc++-v3/src/c++17/floating_from_chars.cc | 4 1 file changed, 4 insertions(+) diff --git a/libstdc++-v3/src/c++17/floating_from_chars.cc b/libstdc++-v3/src/c++17/floating_from_chars.cc index 5d18ca32dbb..3a411cf546a 100644 --- a/libstdc++-v3/src/c++17/floating_from_chars.cc +++ b/libstdc++-v3/src/c++17/floating_from_chars.cc @@ -30,14 +30,18 @@ // Prefer to use std::pmr::string if possible, which requires the cxx11 ABI. #define _GLIBCXX_USE_CXX11_ABI 1 +#include #include #include #include +#include +#include #include #include #include #include #include +#include #include #include #include -- 2.40.0.315.g0607f793cb