I'd add them for all !_LP64 architectures: arm, mips o32, mips n32, i386, and powerpc...
Warner On Jun 4, 2011, at 2:41 PM, Ben Laurie wrote: > It turns out that both clang and gcc define _LP64 when used native on amd64. > > Neither defines _ILP32 on i386 (native or cross-compiled). > > dt_popc() in cddl/contrib/opensolaris/lib/libdtrace/common/dt_subr.c > needs on or the other. clang notices because when _ILP32 is missing > there's no return. > > So ... thoughts?-- > > http://www.apache-ssl.org/ben.html http://www.links.org/ > > "There is no limit to what a man can do or how far he can go if he > doesn't mind who gets the credit." - Robert Woodruff > > _______________________________________________ > freebsd-hackers@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-hackers > To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org" > > _______________________________________________ freebsd-hackers@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-hackers To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"