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

[Måns Rullgård]

Michael Kostylev <michael.kostylev at gmail.com> writes:

> 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.

The bugs should be tracked down and fixed or worked around.  Disabling
optimisations is a bad idea (except for the gcc vectoriser, which will
never be of any use).

>>> --- ./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.

What do the equivalent options look like?  What about other options?

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