]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
linux: Only build fstatat fallback if required
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>
Wed, 24 Nov 2021 15:57:57 +0000 (12:57 -0300)
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>
Thu, 25 Nov 2021 12:28:27 +0000 (09:28 -0300)
For 32-bit architecture with __ASSUME_STATX there is no need to
build fstatat64_time64_stat.

Checked on i686-linux-gnu.

sysdeps/unix/sysv/linux/fstatat64.c

index f968e4ef0594852ec9710811c7b00a8eeb4ab7f0..50ae5ad74832efe1ebb1b9b06309a25a858c8030 100644 (file)
@@ -74,6 +74,17 @@ fstatat64_time64_statx (int fd, const char *file, struct __stat64_t64 *buf,
   return r;
 }
 
+#if (__WORDSIZE == 32 \
+     && (!defined __SYSCALL_WORDSIZE || __SYSCALL_WORDSIZE == 32)) \
+     || defined STAT_HAS_TIME32
+# define FSTATAT_USE_STATX 1
+#else
+# define FSTATAT_USE_STATX 0
+#endif
+
+/* Only statx supports 64-bit timestamps for 32-bit architectures with
+   __ASSUME_STATX, so there is no point in building the fallback.  */
+#if !FSTATAT_USE_STATX || (FSTATAT_USE_STATX && !defined __ASSUME_STATX)
 static inline int
 fstatat64_time64_stat (int fd, const char *file, struct __stat64_t64 *buf,
                       int flag)
@@ -134,13 +145,6 @@ fstatat64_time64_stat (int fd, const char *file, struct __stat64_t64 *buf,
 
   return r;
 }
-
-#if (__WORDSIZE == 32 \
-     && (!defined __SYSCALL_WORDSIZE || __SYSCALL_WORDSIZE == 32)) \
-     || defined STAT_HAS_TIME32
-# define FSTATAT_USE_STATX 1
-#else
-# define FSTATAT_USE_STATX 0
 #endif
 
 int