This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 1/3] sparc: build with -mvis on sparc32/sparcv9 and sparc64

From: Aurelien Jarno <aurelien at aurel32 dot net>
To: David Miller <davem at davemloft dot net>
Cc: libc-alpha at sourceware dot org
Date: Fri, 5 Aug 2016 22:36:01 +0200
Subject: Re: [PATCH 1/3] sparc: build with -mvis on sparc32/sparcv9 and sparc64
Authentication-results: sourceware.org; auth=none
References: <1470272940-22128-1-git-send-email-aurelien@aurel32.net> <20160803.221934.947600832121916069.davem@davemloft.net>

On 2016-08-03 22:19, David Miller wrote:
> From: Aurelien Jarno <aurelien@aurel32.net>
> Date: Thu,  4 Aug 2016 03:08:58 +0200
> 
> > When building for sparc32/sparcv9 or sparc64, we assume that VIS
> > instructions are available and use them in the sparc specific assembly
> > code. However we do not tell GCC to use such instructions, resulting in
> > slightly suboptimal code.
> > 
> > Fix that by passing -Wa,-Av9a -mvis to GCC.
> > 
> > Changelog:
> > 	* sysdeps/sparc/sparc32/sparcv9/Makefile (sysdep-CFLAGS): Add -mvis.
> > 	* sysdeps/sparc/sparc64/Makefile (sysdep-CFLAGS): New. Define to
> > 	-Wa,-Av9a -mvis.
> 
> All three patches look great, please commit them and thanks for doing
> this work.

Thanks for the review, I have just pushed the patches.

-- 
Aurelien Jarno                          GPG: 4096R/1DDD8C9B
aurelien@aurel32.net                 http://www.aurel32.net

References:
- [PATCH 1/3] sparc: build with -mvis on sparc32/sparcv9 and sparc64
  - From: Aurelien Jarno
- Re: [PATCH 1/3] sparc: build with -mvis on sparc32/sparcv9 and sparc64
  - From: David Miller

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]