Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Matthias Kaehlcke
El Fri, Apr 28, 2017 at 03:33:33PM +0100 Mark Rutland ha dit: > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > > On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > > >> On Wed, Apr 26,

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Matthias Kaehlcke
El Fri, Apr 28, 2017 at 03:33:33PM +0100 Mark Rutland ha dit: > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > > On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > > >> On Wed, Apr 26, 2017 at 02:46:16PM

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Will Deacon
On Fri, Apr 28, 2017 at 03:43:56PM +0100, Ard Biesheuvel wrote: > On 28 April 2017 at 15:33, Mark Rutland wrote: > > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > OK, good. That is a departure from previous behavior of Clang, which > was causing build

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Will Deacon
On Fri, Apr 28, 2017 at 03:43:56PM +0100, Ard Biesheuvel wrote: > On 28 April 2017 at 15:33, Mark Rutland wrote: > > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > OK, good. That is a departure from previous behavior of Clang, which > was causing build errors before due to the

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Ard Biesheuvel
On 28 April 2017 at 15:33, Mark Rutland wrote: > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: >> On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: >> > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: >> >> On Wed, Apr

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Ard Biesheuvel
On 28 April 2017 at 15:33, Mark Rutland wrote: > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: >> On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: >> > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: >> >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Mark Rutland
On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > >> > Many inline

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Mark Rutland
On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > >> > Many inline assembly statements

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Mark Rutland
On Fri, Apr 28, 2017 at 11:20:21AM +0100, Ard Biesheuvel wrote: > On 28 April 2017 at 10:53, Mark Rutland wrote: > > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > >> On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > >> > El Thu,

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Mark Rutland
On Fri, Apr 28, 2017 at 11:20:21AM +0100, Ard Biesheuvel wrote: > On 28 April 2017 at 10:53, Mark Rutland wrote: > > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > >> On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > >> > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Ard Biesheuvel
On 28 April 2017 at 10:53, Mark Rutland wrote: > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: >> On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: >> > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: >> >> On Wed, Apr

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Ard Biesheuvel
On 28 April 2017 at 10:53, Mark Rutland wrote: > On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: >> On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: >> > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: >> >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Mark Rutland
On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > >> > - asm

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Mark Rutland
On Fri, Apr 28, 2017 at 08:18:52AM +0100, Ard Biesheuvel wrote: > On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > >> > - asm volatile("strb %w0, [%1]"

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Ard Biesheuvel
On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > Hi Mark, > > Thanks for your comments. > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > >> Hi, >> >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: >> > Many inline assembly statements

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-28 Thread Ard Biesheuvel
On 27 April 2017 at 23:52, Matthias Kaehlcke wrote: > Hi Mark, > > Thanks for your comments. > > El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > >> Hi, >> >> On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: >> > Many inline assembly statements don't include the

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-27 Thread Matthias Kaehlcke
Hi Mark, Thanks for your comments. El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > Hi, > > On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > > Many inline assembly statements don't include the 'x' modifier when > > using xN registers as operands. This is

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-27 Thread Matthias Kaehlcke
Hi Mark, Thanks for your comments. El Thu, Apr 27, 2017 at 12:02:56PM +0100 Mark Rutland ha dit: > Hi, > > On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > > Many inline assembly statements don't include the 'x' modifier when > > using xN registers as operands. This is

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-27 Thread Mark Rutland
Hi, On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > Many inline assembly statements don't include the 'x' modifier when > using xN registers as operands. This is perfectly valid, however it > causes clang to raise warnings like this: > > warning: value size does not match

Re: [PATCH v2] arm64: Add ASM modifier for xN register operands

2017-04-27 Thread Mark Rutland
Hi, On Wed, Apr 26, 2017 at 02:46:16PM -0700, Matthias Kaehlcke wrote: > Many inline assembly statements don't include the 'x' modifier when > using xN registers as operands. This is perfectly valid, however it > causes clang to raise warnings like this: > > warning: value size does not match