On Wed, Feb 25, 2015 at 06:53:14PM -0800, Matt Turner wrote: > On Wed, Feb 25, 2015 at 5:37 PM, Jonathan Gray <[email protected]> wrote: > > If it isn't going to be configure checks could someone merge the > > original patch in this thread? > > I committed > > commit 3492e88090d2d0c0bfbc934963b8772b45fc8880 > Author: Matt Turner <[email protected]> > Date: Fri Feb 20 18:46:43 2015 -0800 > > gallium/util: Use HAVE___BUILTIN_* macros. > > Reviewed-by: Eric Anholt <[email protected]> > Reviewed-by: Jose Fonseca <[email protected]> > > which switched over a bunch of preprocessor checks around __builtin* > calls to use the macros defined by autotools. > > So I think cleaning it up to use __builtin_ffs* first #ifdef > HAVE___BUILTIN_* can go forward now.
Yes but there is no HAVE_FFSLL for constructs like #if !defined(HAVE_FFSLL) && defined(HAVE___BUILTIN_FFSLL) or is it ok to always use the builtin? _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
