[libav-devel] [PATCH 2/2] configure: Do not omit frame pointers for msvc/icl debug builds

2013-06-24 Thread Alex Smith 
Because O1 or O2 are required to build libav with msvc/icl, this must be
explicitly set instead of just omitting Oy.
---
 configure | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configure b/configure
index fdbea76..db591e8 100755
--- a/configure
+++ b/configure
@@ -3956,6 +3956,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
 # basically -fstrict-aliasing that does not work (correctly) on icl 
13.x
 check_cpp_condition windows.h __ICL  1300  add_cflags 
-Qansi-alias
-- 
1.8.1.msysgit.1

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

Re: [libav-devel] [PATCH 2/2] configure: Do not omit frame pointers for msvc/icl debug builds

2013-06-24 Thread Derek Buitenhuis 
On 2013-06-24 6:10 PM, Alex Smith wrote:
 Because O1 or O2 are required to build libav with msvc/icl, this must be
 explicitly set instead of just omitting Oy.
 ---
  configure | 1 +
  1 file changed, 1 insertion(+)

LGTM.

- Derek
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 2/2] configure: Do not omit frame pointers for msvc/icl debug builds

2013-06-24 Thread Diego Biurrun 
On Mon, Jun 24, 2013 at 06:10:12PM -0400, Alex Smith wrote:
 Because O1 or O2 are required to build libav with msvc/icl, this must be
 explicitly set instead of just omitting Oy.
 ---
  configure | 1 +
  1 file changed, 1 insertion(+)

LGTM, thanks for splitting.

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel