From 92a7f46b09d825e9c18b3f7e9154495f82b01e5c Mon Sep 17 00:00:00 2001 From: Rasmus Villemoes Date: Tue, 16 Oct 2018 11:40:27 +0200 Subject: [PATCH] fixincludes: vxworks: regs.h: Fix includes in regs.h wrapper A quick experiment reveals that this hack is needed for C code - simply removing this hack entirely breaks the build of libstdc++, since regs.h (more accurately, the cpu-specific header it pulls in) defines structs in terms of types from vxTypesOld. Those definitions are properly guarded by #ifndef _ASMLANGUAGE, but the cpu-files do not take care to include vxTypesOld.h for the types they depend on. But when using regs.h from some assembly file, the assembler chokes on the typedefs in vxTypesOld.h. We can fix that by guarding the include of vxTypesOld by !_ASMLANGUAGE. This should not affect existing C code. Now, the OS' regs.h contains preprocessor conditionals such as #if CPU_FAMILY==I960 ... #endif /* CPU_FAMILY==I960 */ #if CPU_FAMILY==MC680X0 ... #endif /* CPU_FAMILY==MC680X0 */ Without definitions of CPU_FAMILY, I960 etc., these would all be true, which will not end well. Code using the fix-included regs.h automatically get vxCpu.h via a chain of includes from vxTypesOld.h, but we can make regs.h a little more self-contained for both C and asm users by doing an explicit include of vxCpu.h. From-SVN: r265187 --- fixincludes/ChangeLog | 7 +++++++ fixincludes/fixincl.x | 14 ++++++++++++-- fixincludes/inclhack.def | 10 ++++++++++ 3 files changed, 29 insertions(+), 2 deletions(-) diff --git a/fixincludes/ChangeLog b/fixincludes/ChangeLog index 4f5f1fefb67..e7223659ab6 100644 --- a/fixincludes/ChangeLog +++ b/fixincludes/ChangeLog @@ -1,3 +1,10 @@ +2018-10-16 Rasmus Villemoes + + * inclhack.def (AAB_vxworks_regs_vxtypes): Add unconditional + include of vxCpu.h, guard include of vxTypesOld.h by + !_ASMLANGUAGE. + * fixincl.x: Regenerate. + 2018-09-03 Rasmus Villemoes * inclhack.def (vxworks_ioctl_macro): Remove parentheses from diff --git a/fixincludes/fixincl.x b/fixincludes/fixincl.x index 73324211534..ad5824da015 100644 --- a/fixincludes/fixincl.x +++ b/fixincludes/fixincl.x @@ -2,11 +2,11 @@ * * DO NOT EDIT THIS FILE (fixincl.x) * - * It has been AutoGen-ed September 3, 2018 at 03:24:05 PM by AutoGen 5.18.7 + * It has been AutoGen-ed October 16, 2018 at 11:38:39 AM by AutoGen 5.18.7 * From the definitions inclhack.def * and the template file fixincl */ -/* DO NOT SVN-MERGE THIS FILE, EITHER Mon Sep 3 15:24:05 CEST 2018 +/* DO NOT SVN-MERGE THIS FILE, EITHER Tue Oct 16 11:38:39 CEST 2018 * * You must regenerate it. Use the ./genfixes script. * @@ -579,7 +579,17 @@ tSCC* apzAab_Vxworks_Regs_VxtypesMachs[] = { static const char* apzAab_Vxworks_Regs_VxtypesPatch[] = { "#ifndef _REGS_H\n\ #define _REGS_H\n\ +/* regs.h depends on CPU_FAMILY being properly defined, which\n\ + is done by vxCpu.h. */\n\ +#include \n\ +/* regs.h includes a CPU_FAMILY-specific header that requires\n\ + vxTypesOld.h to already have been included. Those headers\n\ + contain proper _ASMLANGUAGE guards around their typedefs,\n\ + but vxTypesOld.h itself does not. So we avoid including\n\ + vxTypesOld.h from assembly. */\n\ +#ifndef _ASMLANGUAGE\n\ #include \n\ +#endif\n\ #include_next \n\ #endif", (char*)NULL }; diff --git a/fixincludes/inclhack.def b/fixincludes/inclhack.def index f9ba9774f32..8fd9f7ef295 100644 --- a/fixincludes/inclhack.def +++ b/fixincludes/inclhack.def @@ -426,7 +426,17 @@ fix = { replace = <<- _EndOfHeader_ #ifndef _REGS_H #define _REGS_H + /* regs.h depends on CPU_FAMILY being properly defined, which + is done by vxCpu.h. */ + #include + /* regs.h includes a CPU_FAMILY-specific header that requires + vxTypesOld.h to already have been included. Those headers + contain proper _ASMLANGUAGE guards around their typedefs, + but vxTypesOld.h itself does not. So we avoid including + vxTypesOld.h from assembly. */ + #ifndef _ASMLANGUAGE #include + #endif #include_next #endif _EndOfHeader_; -- 2.30.2