Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-12-07 Thread Olivier Hainque via Gcc-patches
> On 6 Dec 2021, at 04:05, Alexandre Oliva wrote: > > On Dec 3, 2021, Olivier Hainque wrote: > >> Alex, how does that look to you? > > LGTM, thanks, Great, thanks. I'll commit shortly and will soon start proposing the batch of other changes I have been mentioning. Some of this might not

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-12-05 Thread Alexandre Oliva via Gcc-patches
On Dec 3, 2021, Olivier Hainque wrote: > Alex, how does that look to you? LGTM, thanks, -- Alexandre Oliva, happy hackerhttps://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer Disinformation flourishes because many people care deeply

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-12-03 Thread Olivier Hainque via Gcc-patches
> On 3 Dec 2021, at 11:27, Rasmus Villemoes wrote: > > Reverting my fix and applying this on top of my gcc-11.2 based branch > seems to work. I haven't used the compiler for building or running any > modules (don't have the hardware handy), but I've done an 'objdump -d' > comparison on all the

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-12-03 Thread Rasmus Villemoes
On 02/12/2021 16.29, Olivier Hainque wrote: > Hi Rasmus, > > Some new on this. > >> On 20 Nov 2021, at 09:07, Olivier Hainque wrote: >> >> I'll check how our build sequence proceeds. > > Turns out our build succeeds thanks to the presence > of a vendor version of stdint.h in the VxWorks6/7 head

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-12-02 Thread Olivier Hainque via Gcc-patches
Hi Rasmus, Some new on this. > On 20 Nov 2021, at 09:07, Olivier Hainque wrote: > > I'll check how our build sequence proceeds. Turns out our build succeeds thanks to the presence of a vendor version of stdint.h in the VxWorks6/7 header dirs and we're lucky that the need to pre-include yvals.h

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-11-20 Thread Olivier Hainque via Gcc-patches
> On 19 Nov 2021, at 21:47, Rasmus Villemoes wrote: >> >> Your error triggers on O2ggnu++0x.gch and we configure with >> --disable-libstdcxx-pch. >> > > ISTR I already tried that, but just for good measure I did it again, and > no luck: > > /bin/sh ../libtool --tag CXX --tag disable-shared

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-11-19 Thread Rasmus Villemoes
On 19/11/2021 19.11, Olivier Hainque wrote: > Hi Rasmus, > >> On 12 Nov 2021, at 17:35, Olivier Hainque wrote: > > Your error triggers on O2ggnu++0x.gch and we configure with > --disable-libstdcxx-pch. > > Can you see if adding this to your configuration options helps > on your end? ISTR I alr

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-11-19 Thread Olivier Hainque via Gcc-patches
Hi Rasmus, > On 12 Nov 2021, at 17:35, Olivier Hainque wrote: > We have had to use for stdbool a similar trick as we had > for stdint (need to preinclude yyvals.h), which we will need to > propagate somehow. I'm not yet sure how to reconcile that with > your observations. >> In file included fr

Re: [PATCH] gcc: vxworks: fix providing stdint.h header

2021-11-12 Thread Olivier Hainque via Gcc-patches
Hi Rasmus, We have had to use for stdbool a similar trick as we had for stdint (need to preinclude yyvals.h), which we will need to propagate somehow. I'm not yet sure how to reconcile that with your observations. Olivier > On 12 Nov 2021, at 11:15, Rasmus Villemoes wrote: > > Commit bbbc05957