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
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 th
+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.
>
> Than
: 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
>>>
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 at
>>> 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
>>> HO
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.
> >>>
> >>> T
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
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
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
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
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
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 cha
>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
14 matches
Mail list logo