[FFmpeg-devel] [PATCH] Add CFLAGS needed by PathScale compiler

Måns Rullgård mans
Mon Jan 24 02:59:02 CET 2011


"Ronald S. Bultje" <rsbultje at gmail.com> writes:

> Hi,
>
> On Sun, Jan 23, 2011 at 6:13 PM, Mans Rullgard <mans at mansr.com> wrote:
>> The PathScale compiler miscompiles wrapping arithmetic without
>> these flags.
>>
>> Signed-off-by: Mans Rullgard <mans at mansr.com>
>> ---
>> ?configure | ? ?2 ++
>> ?1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 4c24152..7854659 100755
>> --- a/configure
>> +++ b/configure
>> @@ -2987,6 +2987,8 @@ elif enabled armcc; then
>> ? ? add_cflags -W${armcc_opt},--diag_suppress=3343 # hardfp compat
>> ?elif enabled tms470; then
>> ? ? add_cflags -pds=824 -pds=837
>> +elif enabled pathscale; then
>> + ? ?add_cflags -fstrict-overflow -OPT:wrap_around_unsafe_opt=OFF
>> ?fi
>>
>> ?enabled_any $THREADS_LIST ? ? ?&& enable threads
>> --
>> 1.7.3.5
>
> OK. Which fate tests does this fix?

I don't remember, but was many.  I've had those flags in the fate
config file for a while to try them out.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list