Re: [PATCH v2 11/58] bsd-user: Fix __i386__ test for TARGET_HAS_STAT_TIME_T_EXT

[Michael Tokarev]

On 1/16/26 06:32, Richard Henderson wrote:

The target test is TARGET_I386, not __i386__.

Cc: Warner Losh <i...@bsdimp.com>
Cc: Kyle Evans <kev...@freebsd.org>
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
Reviewed-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>
Signed-off-by: Richard Henderson <richard.hender...@linaro.org>

This looks like a qemu-stable material (but probably of very
low priority).  Please let me know if it is not.

Thanks,

/mjt

---
  bsd-user/syscall_defs.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bsd-user/syscall_defs.h b/bsd-user/syscall_defs.h
index 52f84d5dd1..c49be32bdc 100644
--- a/bsd-user/syscall_defs.h
+++ b/bsd-user/syscall_defs.h
@@ -247,7 +247,7 @@ struct target_freebsd11_stat {
      unsigned int:(8 / 2) * (16 - (int)sizeof(struct target_freebsd_timespec));
  } __packed;

-#if defined(__i386__)

+#if defined(TARGET_I386)
  #define TARGET_HAS_STAT_TIME_T_EXT       1
  #endif