[FFmpeg-devel] [PATCH] add cflags filtering rules for suncc

[Michael Kostylev]

On Sat Oct 31 11:44:54 2009
M?ns Rullg?rd wrote:

>> The -Os mapping may be ajusted/safely removed, -O2 is set just because 
>> the -O[3-5] options are currently unusable on x86.
> 
> Are you sure you attached the patch you're talking about there?  I see
> nothing about 3-5.

Theoretically, `-O[3-5] -xspace' should work too, -xspace is an additional
flag, it is used to disable optimizations which increase code size.
Maybe `-O5 -xspace' works ok on sparc, I don't know.

>> --- ./configure.orig
>> +++ ./configure
>> @@ -1613,6 +1618,17 @@
>>      cc_version="AV_STRINGIFY(__SUNPRO_C)"
>>      DEPEND_CMD='$(DEPCC) $(DEPFLAGS) $< | sed -e "1s,^.*: ,$@: ," -e "\$$!s,\$$, \\\\\\," -e "1!s,^.*: , ," > $(@:.o=.d)'
>>      DEPFLAGS='$(CPPFLAGS) $(CFLAGS) -xM1'
>> +    filter_cflags=suncc_flags
>> +    suncc_flags(){
>> +        for flag; do
>> +            case $flag in
>> +                -std=c99)             echo -xc99           ;;
>> +                -fomit-frame-pointer) echo -xregs=frameptr ;;
>> +                -Os)                  echo -O2 -xspace     ;;
>> +                -O[1-5]|-g)           echo $flag           ;;
>> +            esac
>> +        done
>> +    }
>>  fi
>
> Do you realise that this will drop all other flags?

I do. If you need more details:
almost all warning options are on by default;
it is not very easy to map -march and -mcpu, it can de done later.

Michael