On 11/13/20 12:03 PM, John David Anglin wrote: > On 2020-11-13 1:20 p.m., Jeff Law wrote: >> On 11/13/20 10:29 AM, Jakub Jelinek via Gcc-patches wrote: >>> Hi! >>> >>> The following patch predefines __STDCPP_THREADS__ macro to 1 if c++11 or >>> later and thread model (e.g. printed by gcc -v) is not single. >>> There are two targets not handled by this patch, those that define >>> THREAD_MODEL_SPEC. In one case - QNX - it looks just like a mistake >>> to me, instead of setting thread_model=posix in config.gcc it uses >>> THREAD_MODEL_SPEC macro to set it unconditionally to posix. >>> The other is hpux10, which uses -threads option to decide if threads >>> are enabled or not, but that option isn't really passed to the compiler. >>> I think that is something that really should be solved in config/pa/ >>> instead, e.g. in the config/xxx/xxx-c.c targets usually set their own >>> predefined macros and it could handle this, and either pass the option >>> also to the compiler, or say predefine __STDCPP_THREADS__ if _DCE_THREADS >>> macro is defined already (or -D_DCE_THREADS found on the command line), >>> or whatever else. >>> >>> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? >>> >>> 2020-11-13 Jakub Jelinek <ja...@redhat.com> >>> >>> * c-cppbuiltin.c: Include configargs.h. >>> (c_cpp_builtins): For C++11 and later if THREAD_MODEL_SPEC is not >>> defined, predefine __STDCPP_THREADS__ to 1 unless thread_model is >>> "single". >> OK. Note that hpux10 should be considered long dead. I wouldn't let >> that get in the way of anything. One could argue we should remove >> hpux10 and earlier, leaving just hpux11. > In principle, I agree. But there are some intereactions in the header > defines and I have limited > time at the moment.
ACK. I don't think removing the old hpux stuff is a high priority. My primary point was that I think hpux10 is dead and we shouldn't let it get in the way of making progress on platforms that are still viable. jeff