On Thu, Jun 20, 2013 at 2:00 PM, Diego Biurrun <di...@biurrun.de> wrote:
> On 2013-06-20 11:47, Hendrik Leppkes wrote:
>>
>> On Thu, Jun 20, 2013 at 11:42 AM, Diego Biurrun <di...@biurrun.de> wrote:
>>>
>>> On 2013-06-20 08:09, Hendrik Leppkes wrote:
>>>>
>>>>
>>>> On Thu, Jun 20, 2013 at 2:12 AM, Diego Biurrun <di...@biurrun.de> wrote:
>>>>>
>>>>>
>>>>> On 2013-06-20 00:07, Alex Smith wrote:
>>>>>>
>>>>>>
>>>>>> Instead, since it is implied by O1 or O2, selectively disable frame
>>>>>> pointer omission for x86 debug builds.  Oy is ignored for x64 targets.
>>>>>> This also silences warnings with icl targeting x64.
>>>>>> --- a/configure
>>>>>> +++ b/configure
>>>>>> @@ -2353,7 +2353,7 @@ msvc_common_flags(){
>>>>>>                 -std=c99)             ;;
>>>>>>                 # Common flags
>>>>>> -            -fomit-frame-pointer) echo -Oy ;;
>>>>>> +            -fomit-frame-pointer) ;;
>>>>>>                 -g)                   echo -Z7 ;;
>>>>>>                 -fno-math-errno)      ;;
>>>>>> @@ -3947,6 +3947,7 @@ elif enabled pathscale; then
>>>>>>     elif enabled_any msvc icl; then
>>>>>>         enabled x86_32 && disable aligned_stack
>>>>>> +    enabled_all x86_32 debug && add_cflags -Oy-
>>>>>
>>>>>
>>>>> This changes the semantics of --disable-optimizations, which will no
>>>>> longer
>>>>> disable this flag.
>>>>
>>>>
>>>> How so?
>>>
>>>
>>> Have a look at configure line 3350:
>>>
>>> disabled optimizations || check_cflags -fomit-frame-pointer
>>
>>
>> Did you read the patch and my comments?
>> Oy is implicit in both O1 and O2, and either one of those is required
>> for building. So it was never disabled before.
>
>
> So neither msvc nor icl build with --disable-optimizations?
>
>

I can't be sure about ICL, it may have a separate option to enable DCE
without all the optimizations, but MSVC folds DCE into the general
optimization, so you need at least O1 or O2

- Hendrik
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to