On Mon, Sep 23, 2013 at 04:34:25AM -0400, Alex Smith wrote:
> Qansi-alias worked on 12.x, then caused miscompilation on 13.x, but now
> works again passing all fate tests for version icl version 14.

s/version//

> Qansi-alias is the icl equivalent of -fstrict-aliasing.

IMO redundant.

> --- a/configure
> +++ b/configure
> @@ -4021,8 +4021,9 @@ elif enabled_any msvc icl; then
>      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
> +        # -Qansi-alias is basically -fstrict-aliasing that does not work 
> (correctly) on icl 13.x

Break this line.

> +        check_cpp_condition "windows.h" "__ICL <  1300" && add_cflags 
> -Qansi-alias
> +        check_cpp_condition "windows.h" "__ICL >= 1400" && add_cflags 
> -Qansi-alias

Can't you check for "== 13" in just one line?

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

Reply via email to