On 05/06/2011 06:03, Warner Losh wrote: > I'd add them for all !_LP64 architectures: arm, mips o32, mips n32, i386, and > powerpc...
Forgive the stupid question, but ... add them to what? > > 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 >> >> _______________________________________________ >> [email protected] mailing list >> http://lists.freebsd.org/mailman/listinfo/freebsd-hackers >> To unsubscribe, send any mail to "[email protected]" >> >> > > -- 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 _______________________________________________ [email protected] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-hackers To unsubscribe, send any mail to "[email protected]"

