On Sun, Jun 23, 2013 at 08:49:20PM -0400, Alex Smith wrote: > On Wed, Jun 19, 2013 at 6:07 PM, Alex Smith <alex.sm...@warpsharp.info> 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 > > add_cflags -fstrict-overflow -OPT:wrap_around_unsafe_opt=OFF > > elif enabled_any msvc icl; then > > enabled x86_32 && disable aligned_stack > > + enabled_all x86_32 debug && add_cflags -Oy- > > if enabled icl; then > > Bump
I think these changes should be split into two patches. Otherwise you guys have convinced me that this is the way to go. Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel