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

2017-04-25 Thread Will Deacon
On Mon, Apr 24, 2017 at 12:13:45PM -0700, Matthias Kaehlcke wrote: > El Mon, Apr 24, 2017 at 06:34:14PM +0100 Will Deacon ha dit: > > On Mon, Apr 24, 2017 at 06:22:51PM +0100, Ard Biesheuvel wrote: > > > AIUI, Clang now always complains for missing register width modifiers, > > > not just for

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

2017-04-24 Thread Matthias Kaehlcke
Hi, El Mon, Apr 24, 2017 at 06:34:14PM +0100 Will Deacon ha dit: > On Mon, Apr 24, 2017 at 06:22:51PM +0100, Ard Biesheuvel wrote: > > On 24 April 2017 at 18:00, Will Deacon wrote: > > > Hi Matthias, > > > > > > On Thu, Apr 20, 2017 at 11:30:53AM -0700, Matthias Kaehlcke

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

2017-04-24 Thread Will Deacon
On Mon, Apr 24, 2017 at 06:22:51PM +0100, Ard Biesheuvel wrote: > On 24 April 2017 at 18:00, Will Deacon wrote: > > Hi Matthias, > > > > On Thu, Apr 20, 2017 at 11:30:53AM -0700, Matthias Kaehlcke wrote: > >> Many inline assembly statements don't include the 'x' modifier when

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

2017-04-24 Thread Ard Biesheuvel
On 24 April 2017 at 18:00, Will Deacon wrote: > Hi Matthias, > > On Thu, Apr 20, 2017 at 11:30:53AM -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 >>

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

2017-04-24 Thread Will Deacon
Hi Matthias, On Thu, Apr 20, 2017 at 11:30:53AM -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

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

2017-04-20 Thread Matthias Kaehlcke
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 register size specified by the constraint and modifier [-Wasm-operand-widths] ...