[flac-dev] [PATCH] Fix compilation on OS/2

Erik de Castro Lopo mle+la at mega-nerd.com
Mon Feb 8 03:25:11 PST 2016


Dave Yeo wrote:

> On 01/24/16 12:29 PM, Erik de Castro Lopo wrote:> Dave Yeo wrote:
> > 
> >> After this the build dies with,
> >> util.c: In function 'benchmark_function':
> >> util.c:124:17: error: 'CLOCK_PROCESS_CPUTIME_ID' undeclared (first use
> >> in this function)
> >>     clock_gettime (CLOCK_PROCESS_CPUTIME_ID, &start) ;
> >>
> >> Would using gettimeofday() be accurate enough?
> > 
> > AS a fall back yes, but not as replacement on platforms with clock_gettime().
> > 
> >> Perhaps simpler just to
> >> disable the microbench test on OS/2 as gettimeofday() would require a
> >> configure test and OS/2 is a minor platform?
> > 
> > 
> > I would accept a patch that disables the microsbenchmark on OS/2 I would
> > also accept one that uses gettimeofday() when clock_gettime() is not
> > available.

I'm happy with the first patch (making AVX and AVX2 a configure option) and
I've committed and pushed it to the repo.

The configure stuff in the second seems overly complex. Is something like
this:

     AC_CHECK_LIB(rt, clock_gettime, have_clock_gettime=yes)
     if test x$have_clock_gettime = xyes; then
         AC_DEFINE(HAVE_CLOCK_GETTIME)
     fi

not sufficient? You can either test this and provide an updated patch or
I can fix it here.

Cheers,
Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/


More information about the flac-dev mailing list