Diego Biurrun <[email protected]> writes: > On Thu, Jul 19, 2012 at 11:43:32AM +0100, Mans Rullgard wrote: >> This variable is only used once, so setting it there is clearer. >> >> --- a/configure >> +++ b/configure >> @@ -1975,7 +1975,6 @@ elif $cc -v 2>&1 | grep -qi ^gcc; then >> elif $cc --version 2>/dev/null | grep -q Intel; then >> cc_type=icc >> cc_ident=$($cc --version | head -n1) >> - icc_version=$($cc -dumpversion) >> CC_DEPFLAGS='-MMD' >> AS_DEPFLAGS='-MMD' >> @@ -3149,6 +3148,7 @@ if enabled icc; then >> # icc 11.0 and 11.1 work with ebp_available, but don't pass the test >> enable ebp_available >> if enabled x86_32; then >> + icc_version=$($cc -dumpversion) >> test ${icc_version%%.*} -ge 11 && >> check_cflags -falign-stack=maintain-16-byte || > > Well, it also makes sense to set this in the compiler section, but > if you prefer it this way, I don't mind...
I have my reasons for this which will become clear in due course. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
