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 <dje....@gmail.com> 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. >> >> Thanks, David >> >> On Mon, May 17, 2021 at 7:44 AM CHIGOT, CLEMENT <clement.chi...@atos.net> >> wrote: >> > >> > 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 <clement.chi...@atos.net> >> > >> > * configure.ac: Check sizeof ino_t and dev_t. >> > * config.in: Regenerate. >> > * configure: Regenerate. >> > * config/rs6000/aix.h (HOST_STAT_FOR_64BIT_INODES): New define. >> > * incpath.c (HOST_STAT_FOR_64BIT_INODES): New define. >> > (remove_duplicates): Use it. >> > >> > libcpp/ChangeLog: >> > 2021-05-06 Clément Chigot <clement.chi...@atos.net> >> > >> > * configure.ac: Check sizeof ino_t and dev_t. >> > * config.in: Regenerate. >> > * configure: Regenerate. >> > * include/cpplib.h (INO_T_CPP): Change for AIX. >> > (DEV_T_CPP): New macro. >> > (struct cpp_dir): Use it. >> > >> > >> >
