[PATCH 6/N v2] [x86_64] Vectorized math functions

Joseph Myers joseph@codesourcery.com
Wed Dec 3 18:05:00 GMT 2014


On Tue, 2 Dec 2014, Andrew Senkevich wrote:

> +ifeq ($(build-mathvec),yes)
> +# We need to install libm.so as linker script
> +# for more comfortable use of vector math library.
> +install-lib-ldscripts := libm.so
> +libmvec-so-ver = libmvec.so$(libmvec.so-version)
> +install_subdir: $(inst_libdir)/libm.so
> +$(inst_libdir)/libm.so: $(common-objpfx)format.lds $(libm) \
> +			$(common-objpfx)mathvec/$(libmvec-so-ver) \
> +			$(+force)
> +	(echo '/* GNU ld script'; echo '*/';\
> +	cat $<; \
> +	echo 'GROUP ( $(slibdir)/libm.so$(libm.so-version) ' \
> +	'AS_NEEDED ( $(slibdir)/libmvec.so$(libmvec.so-version) ) )' \

Watch indentation.  Commands inside () should be indented one space after 
the initial TAB so they all line up; the 'AS_NEEDED...' argument to echo 
should be indented to line up with the 'GROUP...' argument.

I think patches 5 and 6 in the series should be combined (that is, 
building and installing the library should be in the same patch).

-- 
Joseph S. Myers
joseph@codesourcery.com



More information about the Libc-alpha mailing list