Re: [PATCHv2][ARM] -mpure-code option for ARM

2016-10-04 Thread Andre Vieira (lists)
On 04/10/16 14:24, mickael guene wrote: > Hi Andre, > > I can't see new testsuite files in trunk : > gcc.target/arm/pure-code/ffunction-sections.c > gcc.target/arm/pure-code/no-literal-pool.c > gcc.target/arm/pure-code/pure-code.exp > > It seems you forgot to include them in your patch. > Can

Re: [PATCHv2][ARM] -mpure-code option for ARM

2016-10-04 Thread mickael guene
Hi Andre, I can't see new testsuite files in trunk : gcc.target/arm/pure-code/ffunction-sections.c gcc.target/arm/pure-code/no-literal-pool.c gcc.target/arm/pure-code/pure-code.exp It seems you forgot to include them in your patch. Can you post a new one with those files ? Regards Mickael

Re: [PATCHv2][ARM] -mpure-code option for ARM

2016-09-23 Thread Sandra Loosemore
On 09/23/2016 04:39 AM, Andre Vieira (lists) wrote: Hi Sandra, Is this better? Cheers, Andre gcc/ChangeLog 2016-09-23 Andre Vieira * target.def(elf_flags_numeric): Change documentation to present tense. And here is the patch with the

Re: [PATCHv2][ARM] -mpure-code option for ARM

2016-09-23 Thread Andre Vieira (lists)
On 23/09/16 09:33, Andre Vieira (lists) wrote: > On 23/09/16 02:21, Sandra Loosemore wrote: >> On 09/22/2016 07:52 AM, Richard Earnshaw (lists) wrote: >>> On 11/07/16 17:56, Andre Vieira (lists) wrote: + diff --git a/gcc/target.def b/gcc/target.def index

Re: [PATCHv2][ARM] -mpure-code option for ARM

2016-09-23 Thread Andre Vieira (lists)
On 23/09/16 02:21, Sandra Loosemore wrote: > On 09/22/2016 07:52 AM, Richard Earnshaw (lists) wrote: >> On 11/07/16 17:56, Andre Vieira (lists) wrote: >>> + >>> diff --git a/gcc/target.def b/gcc/target.def >>> index >>>

Re: [PATCHv2][ARM] -mpure-code option for ARM

2016-09-22 Thread Sandra Loosemore
On 09/22/2016 07:52 AM, Richard Earnshaw (lists) wrote: On 11/07/16 17:56, Andre Vieira (lists) wrote: + diff --git a/gcc/target.def b/gcc/target.def index a4df363698ce776b51d11c187baed2069ba88a52..a3d46fa48d919a16699c33b2b78236e62a33e025 100644 --- a/gcc/target.def +++ b/gcc/target.def @@

Re: [PATCHv2][ARM] -mpure-code option for ARM

2016-09-22 Thread Richard Earnshaw (lists)
On 11/07/16 17:56, Andre Vieira (lists) wrote: > On 07/07/16 13:30, mickael guene wrote: >> Hi Andre, >> >> Another feedback on your purecode patch. >> You have to disable casesi pattern since then it will >> generate wrong code with -mpure-code option. >> Indeed it will generate an 'adr rx,

[PATCHv2][ARM] -mpure-code option for ARM

2016-07-11 Thread Andre Vieira (lists)
On 07/07/16 13:30, mickael guene wrote: > Hi Andre, > > Another feedback on your purecode patch. > You have to disable casesi pattern since then it will > generate wrong code with -mpure-code option. > Indeed it will generate an 'adr rx, .Lx' (aka > 'subs rx, PC, #offset') which will not work