Starting from VxWorks 7, the system comes with a Dinkumware environment which requires the inclusion of "yvals.h" before other system headers. We provide wrapped versions of a few headers to accommodate such constraints. Initially proposed by Jérôme, this helps fix dozens of build issues when activating libstdc++ on such environments.
Tested in accordance with the description in https://gcc.gnu.org/ml/gcc-patches/2019-12/msg00911.html Committing soon to mainline. Olivier 2019-12-13 Jerome Lambourg <lambo...@adaccore.com> Olivier Hainque <hain...@adacore.com> gcc/ * config/vxworks/_yvals.h: New file. * config/vxworks/_yvals-wrapper.h: New file. * gcc/config/vxworks/math.h: Use it to wrap the VxWorks math.h header. * gcc/config/vxworks/complex.h: Likewise. * gcc/config/vxworks/setjmp.h: Likewise. * gcc/config/vxworks/inttypes.h: Likewise. * config.gcc (*-*-vxworks*): Add system header wrappers to extra_headers. (powerpc-*-vxworks*): Reuse the common extra_headers.
gcc/config.gcc | 16 +++++++++- gcc/config/vxworks/_yvals-wrapper.h | 44 ++++++++++++++++++++++++++++ gcc/config/vxworks/_yvals.h | 45 +++++++++++++++++++++++++++++ gcc/config/vxworks/complex.h | 28 ++++++++++++++++++ gcc/config/vxworks/inttypes.h | 28 ++++++++++++++++++ gcc/config/vxworks/math.h | 28 ++++++++++++++++++ gcc/config/vxworks/setjmp.h | 34 ++++++++++++++++++++++ 7 files changed, 222 insertions(+), 1 deletion(-) create mode 100644 gcc/config/vxworks/_yvals-wrapper.h create mode 100644 gcc/config/vxworks/_yvals.h create mode 100644 gcc/config/vxworks/complex.h create mode 100644 gcc/config/vxworks/inttypes.h create mode 100644 gcc/config/vxworks/math.h create mode 100644 gcc/config/vxworks/setjmp.h diff --git a/gcc/config.gcc b/gcc/config.gcc index b09a3656be8..4790247a868 100644 --- a/gcc/config.gcc +++ b/gcc/config.gcc @@ -953,12 +953,15 @@ case ${target} in *-*-vxworks*) tmake_file=t-vxworks xm_defines=POSIX + extra_options="${extra_options} vxworks.opt" extra_objs="$extra_objs vxworks.o" + c_target_objs="${c_target_objs} vxworks-c.o" cxx_target_objs="${cxx_target_objs} vxworks-c.o" extra_headers="${extra_headers} ../vxworks/vxworks-predef.h" target_has_targetcm="yes" + use_gcc_stdint=provide tm_file="${tm_file} vxworks-stdint.h" @@ -968,6 +971,17 @@ case ${target} in extra_headers="${extra_headers} ../vxworks/_vxworks-versions.h" + # Starting from VxWorks 7, the system comes with a Dinkumware + # environment which requires the inclusion of "yvals.h" before other + # system headers. We provide wrapped versions of a few headers to + # accomodate such constraints: + + extra_headers="${extra_headers} ../vxworks/_yvals.h" + extra_headers="${extra_headers} ../vxworks/_yvals-wrapper.h" + + extra_headers="${extra_headers} ../vxworks/math.h ../vxworks/complex.h" + extra_headers="${extra_headers} ../vxworks/inttypes.h ../vxworks/setjmp.h" + case ${enable_threads} in no) ;; "" | yes | vxworks) thread_file='vxworks' ;; @@ -2918,7 +2932,7 @@ powerpc-wrs-vxworks*) tm_file="${tm_file} elfos.h gnu-user.h freebsd-spec.h rs6000/sysv4.h" tmake_file="${tmake_file} rs6000/t-fprules rs6000/t-ppccomm rs6000/t-vxworks" extra_options="${extra_options} rs6000/sysv4.opt" - extra_headers=ppc-asm.h + extra_headers="${extra_headers} ppc-asm.h" case ${target} in *-vxworksmils*) tm_file="${tm_file} vx-common.h vxworksae.h rs6000/vxworks.h rs6000/vxworksmils.h" diff --git a/gcc/config/vxworks/_yvals-wrapper.h b/gcc/config/vxworks/_yvals-wrapper.h new file mode 100644 index 00000000000..e7b84c2174d --- /dev/null +++ b/gcc/config/vxworks/_yvals-wrapper.h @@ -0,0 +1,44 @@ +/* This file is part of GCC. + +GCC is free software; you can redistribute it and/or modify it under +the terms of the GNU General Public License as published by the Free +Software Foundation; either version 3, or (at your option) any later +version. + +GCC is distributed in the hope that it will be useful, but WITHOUT ANY +WARRANTY; without even the implied warranty of MERCHANTABILITY or +FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +for more details. + +Under Section 7 of GPL version 3, you are granted additional +permissions described in the GCC Runtime Library Exception, version +3.1, as published by the Free Software Foundation. + +You should have received a copy of the GNU General Public License and +a copy of the GCC Runtime Library Exception along with this program; +see the files COPYING3 and COPYING.RUNTIME respectively. If not, see +<http://www.gnu.org/licenses/>. */ + +/* The VxWorks 7 environment for C++ is based on a Dinkumware toolchain, + where quite a few configuration parameters are exposed in a yvals.h header + file which needs to be included before other standard headers. + + This header file provides a wrapper facility to achieve this in addition + to the common requirement to include vxWorks.h before anything else. */ + +#include <_vxworks-versions.h> + +#if _VXWORKS_MAJOR_GE(7) && defined(__cplusplus) + +#include <_yvals.h> + +#pragma push_macro("__cplusplus") +#undef __cplusplus +#include_next __HEADER_TO_WRAP +#pragma pop_macro("__cplusplus") + +#else + +#include_next __HEADER_TO_WRAP + +#endif diff --git a/gcc/config/vxworks/_yvals.h b/gcc/config/vxworks/_yvals.h new file mode 100644 index 00000000000..f8b0818185c --- /dev/null +++ b/gcc/config/vxworks/_yvals.h @@ -0,0 +1,45 @@ +/* This file is part of GCC. + +GCC is free software; you can redistribute it and/or modify it under +the terms of the GNU General Public License as published by the Free +Software Foundation; either version 3, or (at your option) any later +version. + +GCC is distributed in the hope that it will be useful, but WITHOUT ANY +WARRANTY; without even the implied warranty of MERCHANTABILITY or +FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +for more details. + +Under Section 7 of GPL version 3, you are granted additional +permissions described in the GCC Runtime Library Exception, version +3.1, as published by the Free Software Foundation. + +You should have received a copy of the GNU General Public License and +a copy of the GCC Runtime Library Exception along with this program; +see the files COPYING3 and COPYING.RUNTIME respectively. If not, see +<http://www.gnu.org/licenses/>. */ + +#ifndef __GCC_YVALS_H +#define __GCC_YVALS_H + +#include <_vxworks-versions.h> + +#if _VXWORKS_MAJOR_GE(7) + +/* We need to deactivate the definitions tailored for the Dinkumware + intrinsics, incompatible with a compilation by G++. */ + +#include <yvals.h> + +#undef _HAS_CPP17 +#define _HAS_CPP17 0 + +#undef _HAS_CPP14 +#define _HAS_CPP14 0 + +#undef _HAS_CPP11 +#define _HAS_CPP11 0 + +#endif /* VxWorks MAJOR >= 7 */ + +#endif /* __GCC_YVALS_H */ diff --git a/gcc/config/vxworks/complex.h b/gcc/config/vxworks/complex.h new file mode 100644 index 00000000000..394671a8b49 --- /dev/null +++ b/gcc/config/vxworks/complex.h @@ -0,0 +1,28 @@ +/* This file is part of GCC. + +GCC is free software; you can redistribute it and/or modify it under +the terms of the GNU General Public License as published by the Free +Software Foundation; either version 3, or (at your option) any later +version. + +GCC is distributed in the hope that it will be useful, but WITHOUT ANY +WARRANTY; without even the implied warranty of MERCHANTABILITY or +FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +for more details. + +Under Section 7 of GPL version 3, you are granted additional +permissions described in the GCC Runtime Library Exception, version +3.1, as published by the Free Software Foundation. + +You should have received a copy of the GNU General Public License and +a copy of the GCC Runtime Library Exception along with this program; +see the files COPYING3 and COPYING.RUNTIME respectively. If not, see +<http://www.gnu.org/licenses/>. */ + +#ifndef __GCC_COMPLEX_H +#define __GCC_COMPLEX_H + +#define __HEADER_TO_WRAP <complex.h> +#include <_yvals-wrapper.h> + +#endif diff --git a/gcc/config/vxworks/inttypes.h b/gcc/config/vxworks/inttypes.h new file mode 100644 index 00000000000..d7862a48253 --- /dev/null +++ b/gcc/config/vxworks/inttypes.h @@ -0,0 +1,28 @@ +/* This file is part of GCC. + +GCC is free software; you can redistribute it and/or modify it under +the terms of the GNU General Public License as published by the Free +Software Foundation; either version 3, or (at your option) any later +version. + +GCC is distributed in the hope that it will be useful, but WITHOUT ANY +WARRANTY; without even the implied warranty of MERCHANTABILITY or +FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +for more details. + +Under Section 7 of GPL version 3, you are granted additional +permissions described in the GCC Runtime Library Exception, version +3.1, as published by the Free Software Foundation. + +You should have received a copy of the GNU General Public License and +a copy of the GCC Runtime Library Exception along with this program; +see the files COPYING3 and COPYING.RUNTIME respectively. If not, see +<http://www.gnu.org/licenses/>. */ + +#ifndef __GCC_INTTYPES_H +#define __GCC_INTTYPES_H + +#define __HEADER_TO_WRAP <inttypes.h> +#include <_yvals-wrapper.h> + +#endif diff --git a/gcc/config/vxworks/math.h b/gcc/config/vxworks/math.h new file mode 100644 index 00000000000..8cf31b477ba --- /dev/null +++ b/gcc/config/vxworks/math.h @@ -0,0 +1,28 @@ +/* This file is part of GCC. + +GCC is free software; you can redistribute it and/or modify it under +the terms of the GNU General Public License as published by the Free +Software Foundation; either version 3, or (at your option) any later +version. + +GCC is distributed in the hope that it will be useful, but WITHOUT ANY +WARRANTY; without even the implied warranty of MERCHANTABILITY or +FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +for more details. + +Under Section 7 of GPL version 3, you are granted additional +permissions described in the GCC Runtime Library Exception, version +3.1, as published by the Free Software Foundation. + +You should have received a copy of the GNU General Public License and +a copy of the GCC Runtime Library Exception along with this program; +see the files COPYING3 and COPYING.RUNTIME respectively. If not, see +<http://www.gnu.org/licenses/>. */ + +#ifndef __GCC_MATH_H +#define __GCC_MATH_H + +#define __HEADER_TO_WRAP <math.h> +#include <_yvals-wrapper.h> + +#endif diff --git a/gcc/config/vxworks/setjmp.h b/gcc/config/vxworks/setjmp.h new file mode 100644 index 00000000000..c625e612948 --- /dev/null +++ b/gcc/config/vxworks/setjmp.h @@ -0,0 +1,34 @@ +/* This file is part of GCC. + +GCC is free software; you can redistribute it and/or modify it under +the terms of the GNU General Public License as published by the Free +Software Foundation; either version 3, or (at your option) any later +version. + +GCC is distributed in the hope that it will be useful, but WITHOUT ANY +WARRANTY; without even the implied warranty of MERCHANTABILITY or +FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +for more details. + +Under Section 7 of GPL version 3, you are granted additional +permissions described in the GCC Runtime Library Exception, version +3.1, as published by the Free Software Foundation. + +You should have received a copy of the GNU General Public License and +a copy of the GCC Runtime Library Exception along with this program; +see the files COPYING3 and COPYING.RUNTIME respectively. If not, see +<http://www.gnu.org/licenses/>. */ + +#ifndef __GCC_SETJMP_H +#define __GCC_SETJMP_H + +/* Getting proper namespaces for c++ requires that don't alter the perception + that we are compiling for c++, unlike what _yvals-wrapper would do. + setjmp.h redoes a few of a the yvals.h things in its own fashion, so the + pre-inclusion of yvals.h on VxWorks 7 with __cplusplus defined isn't a + problem in this particular case. */ + +#include <_yvals.h> +#include_next <setjmp.h> + +#endif -- 2.17.1