Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-11 Thread Christophe Lyon
On 11 February 2016 at 19:04, Kyrill Tkachov wrote: > > On 11/02/16 17:57, Christophe Lyon wrote: >> >> On 11 February 2016 at 14:10, Kyrill Tkachov >> wrote: >>> >>> On 10/02/16 10:39, James Greenhalgh wrote: On Wed, Feb 10, 2016 at 10:32:16AM +, Kyrill Tkachov wrote: > > H

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-11 Thread Kyrill Tkachov
On 11/02/16 17:57, Christophe Lyon wrote: On 11 February 2016 at 14:10, Kyrill Tkachov wrote: On 10/02/16 10:39, James Greenhalgh wrote: On Wed, Feb 10, 2016 at 10:32:16AM +, Kyrill Tkachov wrote: Hi James, On 10/02/16 10:11, James Greenhalgh wrote: On Thu, Feb 04, 2016 at 01:50:31PM +

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-11 Thread Christophe Lyon
On 11 February 2016 at 14:10, Kyrill Tkachov wrote: > > On 10/02/16 10:39, James Greenhalgh wrote: >> >> On Wed, Feb 10, 2016 at 10:32:16AM +, Kyrill Tkachov wrote: >>> >>> Hi James, >>> >>> On 10/02/16 10:11, James Greenhalgh wrote: On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill T

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-11 Thread James Greenhalgh
On Thu, Feb 11, 2016 at 01:10:33PM +, Kyrill Tkachov wrote: > >>>Why not just keep the last string you printed, and use a string compare > >>>to decide whether to print or not? Sure we'll end up doing a bit more > >>>work, but the logic becomes simpler to follow and we don't need to pass > >>>a

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-11 Thread Kyrill Tkachov
On 10/02/16 10:39, James Greenhalgh wrote: On Wed, Feb 10, 2016 at 10:32:16AM +, Kyrill Tkachov wrote: Hi James, On 10/02/16 10:11, James Greenhalgh wrote: On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill Tkachov wrote: Hi all, As part of the target attributes and pragmas support for GC

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-10 Thread James Greenhalgh
On Wed, Feb 10, 2016 at 10:32:16AM +, Kyrill Tkachov wrote: > Hi James, > > On 10/02/16 10:11, James Greenhalgh wrote: > >On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill Tkachov wrote: > >>Hi all, > >> > >>As part of the target attributes and pragmas support for GCC 6 I changed the > >>aarch6

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-10 Thread Kyrill Tkachov
Hi James, On 10/02/16 10:11, James Greenhalgh wrote: On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill Tkachov wrote: Hi all, As part of the target attributes and pragmas support for GCC 6 I changed the aarch64 port to emit a .arch assembly directive for each function that describes the archite

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-10 Thread James Greenhalgh
On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill Tkachov wrote: > Hi all, > > As part of the target attributes and pragmas support for GCC 6 I changed the > aarch64 port to emit a .arch assembly directive for each function that > describes the architectural features used by that function. This is

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-04 Thread Kyrill Tkachov
On 04/02/16 17:12, Andrew Pinski wrote: On Thu, Feb 4, 2016 at 5:50 AM, Kyrill Tkachov wrote: Hi all, As part of the target attributes and pragmas support for GCC 6 I changed the aarch64 port to emit a .arch assembly directive for each function that describes the architectural features used b

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-04 Thread Andrew Pinski
On Thu, Feb 4, 2016 at 5:50 AM, Kyrill Tkachov wrote: > Hi all, > > As part of the target attributes and pragmas support for GCC 6 I changed the > aarch64 port > to emit a .arch assembly directive for each function that describes the > architectural features > used by that function. This is a cha

[PATCH][AArch64] Only update assembler .arch directive when necessary

2016-02-04 Thread Kyrill Tkachov
Hi all, As part of the target attributes and pragmas support for GCC 6 I changed the aarch64 port to emit a .arch assembly directive for each function that describes the architectural features used by that function. This is a change from GCC 5 behaviour where we output a single .arch directiv