Address comments

Author: jakeegan

compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.cpp

@@ -1093,6 +1093,9 @@ CHECK_SIZE_AND_OFFSET(addrinfo, ai_protocol);
 CHECK_SIZE_AND_OFFSET(addrinfo, ai_addrlen);
 CHECK_SIZE_AND_OFFSET(addrinfo, ai_canonname);
 CHECK_SIZE_AND_OFFSET(addrinfo, ai_addr);
+#if SANITIZER_AIX
+CHECK_SIZE_AND_OFFSET(addrinfo, ai_eflags);
+#endif
 
 CHECK_TYPE_SIZE(hostent);
 CHECK_SIZE_AND_OFFSET(hostent, h_name);

compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.h

@@ -487,11 +487,19 @@ struct __sanitizer_msghdr {
   struct __sanitizer_iovec *msg_iov;
   uptr msg_iovlen;
   void *msg_control;
+#    if !SANITIZER_AIX
   uptr msg_controllen;
+#    else
+  unsigned msg_controllen;
+#    endif
   int msg_flags;
 };
 struct __sanitizer_cmsghdr {
+#    if !SANITIZER_AIX
   uptr cmsg_len;
+#    else
+  unsigned cmsg_len;
+#    endif
   int cmsg_level;
   int cmsg_type;
 };
@@ -563,6 +571,8 @@ typedef long __sanitizer_clock_t;
 #  if SANITIZER_LINUX || SANITIZER_HAIKU
 typedef int __sanitizer_clockid_t;
 typedef unsigned long long __sanitizer_eventfd_t;
+#  elif SANITIZER_AIX
+typedef int __sanitizer_clockid_t;
 #  endif
 
 #  if SANITIZER_LINUX
@@ -1148,6 +1158,13 @@ extern unsigned fpos_t_sz;
 // when it can not be determined without including any system headers.
 extern const unsigned IOCTL_NOT_PRESENT;
 
+ // On AIX, some variables are unsigned long types.
+#if SANITIZER_AIX
+using ioctl_alttype = uptr;
+#else
+using ioctl_alttype = unsigned;
+#endif
+
 extern ioctl_alttype IOCTL_FIOASYNC;
 extern unsigned IOCTL_FIOCLEX;
 extern unsigned IOCTL_FIOGETOWN;

compiler-rt/lib/sanitizer_common/sanitizer_posix.cpp

@@ -30,6 +30,7 @@
 #if SANITIZER_FREEBSD || SANITIZER_AIX
 // The MAP_NORESERVE define has been removed in FreeBSD 11.x, and even before
 // that, it was never implemented.  So just define it to zero.
+// Similarly, AIX does not define MAP_NORESERVE.
 #undef  MAP_NORESERVE
 #define MAP_NORESERVE 0
 #endif