[vlc-devel] [PATCH] contribs: Force optimizations flags

[Hugo Beauzée-Luyssen]

On 05/17/2016 12:50 PM, Rémi Denis-Courmont wrote:
> Le 2016-05-17 11:51, Hugo Beauzée-Luyssen a écrit :
>> Some packages aren't forcing those, leading to a bunch of contribs being
>> built without any kind of optimizations.
>
> Is -O3 safe for all contribs?
>
>> ---
>>  contrib/src/main.mak | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/contrib/src/main.mak b/contrib/src/main.mak
>> index 8d94b64..481d366 100644
>> --- a/contrib/src/main.mak
>> +++ b/contrib/src/main.mak
>> @@ -177,6 +177,9 @@ LDFLAGS := $(LDFLAGS) $(EXTRA_LDFLAGS)
>>  ifndef WITH_OPTIMIZATION
>>  CFLAGS := $(CFLAGS) -O0
>>  CXXFLAGS := $(CXXFLAGS) -O0
>> +else
>> +CFLAGS := $(CFLAGS) -O3
>> +CXXFLAGS := $(CXXFLAGS) -O3
>>  endif
>>
>>  # Do not export those! Use HOSTVARS.
>
I was originally going for -O2, but since VLC uses -O3 for release 
builds, I went with -O3 for contribs.
I'll push the -O2 version.

Thanks!