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"

Reply via email to