On Mon, Feb 25, 2013 at 07:14:22PM +0200, Martin Storsjö wrote:
> On Mon, 25 Feb 2013, Diego Biurrun wrote:
> >--- a/configure
> >+++ b/configure
> >@@ -2364,7 +2364,7 @@ probe_cc(){
> > fi
> > _cflags_speed='-O3'
> > _cflags_size='-Os'
> >- elif $_cc --version 2>/dev/null | grep -q Intel; then
> >+ elif $_cc --version 2>/dev/null | grep -q ^icc; then
> > _type=icc
> > _ident=$($_cc --version | head -n1)
> > _depflags='-MMD'
>
> Not sure if this is better - on windows, the compiler is called icl
> and not icc. (Not sure if we currently really support that one, but
> I don't think it's worth to move further away from the possibility
> of supporting it.)
We do not currently support icl. There were some work-in-progress
patches floating around on IRC, but nothing more. Those patches added
a separate stanza to the compiler detection section and did not reuse
the icc for Linux stanza.
So this patch does not make future icl support harder to add. It does,
however, make the two compiler detection lines easier to differentiate
as the check for icl - IIRC - also grepped for "Intel" but on stderr
instead of stdout, which is easy to overlook on a quick read.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
