On 24/03/13 07:08, Reinhard Tartler wrote:
> On Sun, Mar 24, 2013 at 5:38 AM, Derek Buitenhuis
> <[email protected]> wrote:
>> On 2013-03-23 3:41 AM, Reinhard Tartler wrote:
>>> +    gcov)
>>> +        add_cflags  -fprofile-arcs -ftest-coverage
>>> +        add_ldflags -fprofile-arcs -ftest-coverage
>>> +    ;;
>>
>> I feel like last time I tried this with any compiler except GCC, it
>> didn't work. Is it worth it to restrict it to certain compilers...?
> 
> AFAIUI, gcov is very specific to GCC currently, but we cannot say for
> sure that there doesn't exist some other compiler, maybe a derivative,
> that implemented this option in this way. We could maybe issue a
> warning if we had a mechanism for that.

the common alternative is using callgrind or freya (from valgrind)

> But I'm sure we have a number of other gcc specific things that could
> make use of such compiler specific warnings as well, so I'd rather
> keep that out of this change, if you don't mind.

I'd keep the patch as is for now.

lu


_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to