Re: bin/55070: compiler_rt does not build with clang with MKIPFILTER=no

[Roy Marples <roy%marples.name@localhost>]

The following reply was made to PR bin/55070; it has been noted by GNATS.

From: Roy Marples <roy%marples.name@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: bin/55070: compiler_rt does not build with clang with
 MKIPFILTER=no
Date: Sat, 14 Mar 2020 22:28:08 +0000

 This patch makes it compile for me.
 
 Index: 
 sys/external/bsd/compiler_rt/dist/lib/sanitizer_common/sanitizer_platform_limits_netbsd.cc
 ===================================================================
 RCS file: 
 /cvsroot/src/sys/external/bsd/compiler_rt/dist/lib/sanitizer_common/sanitizer_platform_limits_netbsd.cc,v
 retrieving revision 1.5
 diff -u -p -r1.5 sanitizer_platform_limits_netbsd.cc
 --- 
 sys/external/bsd/compiler_rt/dist/lib/sanitizer_common/sanitizer_platform_limits_netbsd.cc 
   31 Jan 2020 14:01:36 -0000      1.5
 +++ 
 sys/external/bsd/compiler_rt/dist/lib/sanitizer_common/sanitizer_platform_limits_netbsd.cc 
   14 Mar 2020 22:23:57 -0000
 @@ -162,9 +162,11 @@
   #include <net/slip.h>
   #include <netbt/hci.h>
   #include <netinet/ip_compat.h>
 +#if 0
   #include <netinet/ip_fil.h>
   #include <netinet/ip_nat.h>
   #include <netinet/ip_proxy.h>
 +#endif
   #include <netinet6/in6_var.h>
   #include <netinet6/nd6.h>
   #include <netsmb/smb_dev.h>
 @@ -412,7 +414,9 @@ typedef __sanitizer_nvlist_ref_t nvlist_
   // ioctl arguments
   unsigned struct_altqreq_sz = sizeof(altqreq);
   unsigned struct_amr_user_ioctl_sz = sizeof(amr_user_ioctl);
 +#if 0
   unsigned struct_ap_control_sz = sizeof(ap_control);
 +#endif
   unsigned struct_apm_ctl_sz = sizeof(apm_ctl);
   unsigned struct_apm_event_info_sz = sizeof(apm_event_info);
   unsigned struct_apm_power_info_sz = sizeof(apm_power_info);
 @@ -572,7 +576,9 @@ unsigned struct_ioc_vol_sz = sizeof(ioc_
   unsigned struct_ioctl_pt_sz = sizeof(ioctl_pt);
   unsigned struct_ioppt_sz = sizeof(ioppt);
   unsigned struct_iovec_sz = sizeof(iovec);
 +#if 0
   unsigned struct_ipfobj_sz = sizeof(ipfobj);
 +#endif
   unsigned struct_irda_params_sz = sizeof(irda_params);
   unsigned struct_isp_fc_device_sz = sizeof(isp_fc_device);
   unsigned struct_isp_fc_tsk_mgmt_sz = sizeof(isp_fc_tsk_mgmt);
 @@ -1680,11 +1686,13 @@ unsigned IOCTL_SIOCZBTSTATS = SIOCZBTSTA
   unsigned IOCTL_SIOCBTDUMP = SIOCBTDUMP;
   unsigned IOCTL_SIOCSBTSCOMTU = SIOCSBTSCOMTU;
   unsigned IOCTL_SIOCGBTFEAT = SIOCGBTFEAT;
 +#if 0
   unsigned IOCTL_SIOCADNAT = SIOCADNAT;
   unsigned IOCTL_SIOCRMNAT = SIOCRMNAT;
   unsigned IOCTL_SIOCGNATS = SIOCGNATS;
   unsigned IOCTL_SIOCGNATL = SIOCGNATL;
   unsigned IOCTL_SIOCPURGENAT = SIOCPURGENAT;
 +#endif
   unsigned IOCTL_SIOCCONNECTX = SIOCCONNECTX;
   unsigned IOCTL_SIOCCONNECTXDEL = SIOCCONNECTXDEL;
   unsigned IOCTL_SIOCSIFINFO_FLAGS = SIOCSIFINFO_FLAGS;
 
 
 
 
 Makes me wonder though, are the values even used?
 
 Roy