Re: [PATCH] aix: handle 64bit inodes for include directories

2021-12-30 Thread Jeff Law via Gcc-patches
On 12/30/2021 9:43 AM, David Edelsohn wrote: Hi, Jeff Is the revised patch from Clement okay? I *think* so.    I don't think it's likely to affect any other system as a host, build or target.   I'm not entirely sure it's right for canadian crosses to/from involving aix, but that seems like

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-12-30 Thread David Edelsohn via Gcc-patches
Hi, Jeff Is the revised patch from Clement okay? Thanks, David On Tue, Aug 24, 2021 at 3:59 AM CHIGOT, CLEMENT wrote: > > >>> So my worry here is this is really a host property -- ie, this is > >>> behavior of where GCC runs, not the target for which GCC is generating > >>> code. > >>> > >>>

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-11-16 Thread CHIGOT, CLEMENT via Gcc-patches
Hi everyone, Gentle ping Thanks, Clément From: CHIGOT, CLEMENT Sent: Tuesday, October 26, 2021 4:51 PM To: Jeff Law ; David Malcolm Cc: gcc-patches@gcc.gnu.org ; David Edelsohn Subject: Re: [PATCH] aix: handle 64bit inodes for include directories Hi everyone

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-10-26 Thread CHIGOT, CLEMENT via Gcc-patches
Hi everyone, Gentle ping on this patch. Clément From: CHIGOT, CLEMENT Sent: Tuesday, October 12, 2021 10:35 AM To: Jeff Law ; David Malcolm Cc: gcc-patches@gcc.gnu.org ; David Edelsohn Subject: Re: [PATCH] aix: handle 64bit inodes for include directories Hi

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-10-12 Thread CHIGOT, CLEMENT via Gcc-patches
Hi Jeff, Any update on this patch ? As it's dealing with configure files, I would like to have it merged asap before any conflicts appear. Thanks, Clément

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-08-24 Thread CHIGOT, CLEMENT via Gcc-patches
>>> So my worry here is this is really a host property -- ie, this is >>> behavior of where GCC runs, not the target for which GCC is generating code. >>> >>> That implies that the change in aix.h is wrong. aix.h is for the >>> target, not the host -- you don't want to define something like >>>

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-08-19 Thread Jeff Law via Gcc-patches
On 6/28/2021 1:16 AM, CHIGOT, CLEMENT wrote: On 6/23/2021 12:53 AM, CHIGOT, CLEMENT via Gcc-patches wrote: Hi David, Did you have a chance to take look at this patch ? Thanks, Clément +DavidMalcolm Can you review this patch when you have a moment? Thanks, David On Mon, May 17, 2021

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-08-11 Thread CHIGOT, CLEMENT via Gcc-patches
: handle 64bit inodes for include directories >On 6/23/2021 12:53 AM, CHIGOT, CLEMENT via Gcc-patches wrote: >> Hi David, >> >> Did you have a chance to take look at this patch ? >> >> Thanks, >> Clément >> >> >>> +DavidMalcolm >>>

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-06-28 Thread CHIGOT, CLEMENT via Gcc-patches
>On 6/23/2021 12:53 AM, CHIGOT, CLEMENT via Gcc-patches wrote: >> Hi David, >> >> Did you have a chance to take look at this patch ? >> >> Thanks, >> Clément >> >> >>> +DavidMalcolm >>> >>> Can you review this patch when you have a moment? >>> >>> Thanks, David >>> >>> On Mon, May 17, 2021 at 3:05

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-06-24 Thread Jeff Law via Gcc-patches
On 6/23/2021 12:53 AM, CHIGOT, CLEMENT via Gcc-patches wrote: Hi David, Did you have a chance to take look at this patch ? Thanks, Clément +DavidMalcolm Can you review this patch when you have a moment? Thanks, David On Mon, May 17, 2021 at 3:05 PM David Edelsohn wrote: The aix.h

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-06-23 Thread CHIGOT, CLEMENT via Gcc-patches
Hi David, Did you have a chance to take look at this patch ? Thanks, Clément > > +DavidMalcolm > > Can you review this patch when you have a moment? > > Thanks, David > > On Mon, May 17, 2021 at 3:05 PM David Edelsohn wrote: >> >> The aix.h change is okay with me, but you need to get approval

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-05-22 Thread David Edelsohn via Gcc-patches
+DavidMalcolm Can you review this patch when you have a moment? Thanks, David On Mon, May 17, 2021 at 3:05 PM David Edelsohn wrote: > > The aix.h change is okay with me, but you need to get approval for the > incpath.c and cpplib.h parts of the patch from the appropriate > maintainers. > >

Re: [PATCH] aix: handle 64bit inodes for include directories

2021-05-17 Thread David Edelsohn via Gcc-patches
The aix.h change is okay with me, but you need to get approval for the incpath.c and cpplib.h parts of the patch from the appropriate maintainers. Thanks, David On Mon, May 17, 2021 at 7:44 AM CHIGOT, CLEMENT wrote: > > On AIX, stat will store inodes in 32bit even when using LARGE_FILES. > If

[PATCH] aix: handle 64bit inodes for include directories

2021-05-17 Thread CHIGOT, CLEMENT via Gcc-patches
On AIX, stat will store inodes in 32bit even when using LARGE_FILES. If the inode is larger, it will return -1 in st_ino. Thus, in incpath.c when comparing include directories, if several of them have 64bit inodes, they will be considered as duplicated. gcc/ChangeLog: 2021-05-06  Clément Chigot