This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Update support patch for ACML vectorized intrinsic library
On Fri, 24 Aug 2007, Uros Bizjak wrote:
> On 8/24/07, Richard Guenther <rguenther@suse.de> wrote:
>
> > So, the following extra hunk for the patch automatically links the
> > library.
> >
> > Is the patch ok for mainline?
>
> > * config/i386/linux64.h (LIB_SPEC): Copy from config/linux.h.
> > Link with libacml_mv as needed, if building with -mveclib=acml.
>
> I still think that having -mveclib=acml_mv and to pass acml_mv
> automatically to -l as a variable is better. But I'll leave this to
> your choice (but perhaps we should wait for another opinion).
I'm somewhat unsure here - for one -mveclib was supposed to select
an ABI for the vectorization library, but of course automatically
linking against some specific library name makes this pointless
(that is, it doesn't allow for an alternate implementation with
a different name). Now if we want some automatic linking then still
the option should be easy enough to recognize (acml_mv is not from
my point of view).
So I am fine with either of two options - not linking automatically
or linking against libacml_mv with -mveclib=acml.
Richard.