On 2018-09-05 11:38, Olivier Hainque wrote: > Hi Rasmus, > > I think we should either do a fixinclude that would "work" for > C and ASM (like #include vxCpu for ASM, vxTypesOld otherwise), or > simply remove this hack (just using the fixinclude parlance here). > > My inclination would be for the latter. > [snip] > > What happens on your end if you just remove the hack ? >
Unfortunately, the libstdc++ build breaks: In file included from /usr/powerpc-wrs-vxworks/wind_base/target/h/regs.h:66:0, from /bld/vestas/auto/work.20180912.214646/gcc-build/gcc/include-fixed/setjmp.h:57, from /bld/vestas/auto/work.20180912.214646/gcc-build/powerpc-wrs-vxworks/libstdc++-v3/include/csetjmp:42, from /bld/vestas/auto/work.20180912.214646/gcc-src/libstdc++-v3/include/precompiled/stdc++.h:42: /usr/powerpc-wrs-vxworks/wind_base/target/h/arch/ppc/regsPpc.h:33:5: error: 'UINT32' does not name a type UINT32 cr; /* condition register */ ^~~~~~ I'm happy to add an include of vxCpu in the _ASMLANGUAGE case, along with a big comment. But, it's also a small enough patch that we can carry it internally, if you prefer that we don't touch this hack upstream. Thanks, Rasmus