Re: [PATCH] ARM fixed-point support [6/6]: target-specific parts

2011-07-29 Thread Richard Earnshaw
On 29/07/11 16:47, Julian Brown wrote: > On Thu, 30 Jun 2011 14:42:54 +0100 > Richard Earnshaw wrote: > >>> OK to apply? Tested alongside the rest of the patch series, in both >>> big & little-endian mode. >>> [snip] >> >> Please put the iterator definitions in iterators.md > > I've done this, a

Re: [PATCH] ARM fixed-point support [6/6]: target-specific parts

2011-07-29 Thread Julian Brown
On Thu, 30 Jun 2011 14:42:54 +0100 Richard Earnshaw wrote: > > OK to apply? Tested alongside the rest of the patch series, in both > > big & little-endian mode. > > [snip] > > Please put the iterator definitions in iterators.md I've done this, and made a few other changes as required by the new

Re: [PATCH] ARM fixed-point support [6/6]: target-specific parts

2011-06-30 Thread Richard Earnshaw
On 26/05/11 17:45, Julian Brown wrote: > On Fri, 13 May 2011 17:31:18 +0100 > Julian Brown wrote: > >> On Fri, 13 May 2011 14:54:47 + (UTC) >> "Joseph S. Myers" wrote: >> >>> On Fri, 13 May 2011, Julian Brown wrote: >>> although Joseph's comments have (hopefully) all been addressed. >>>

Re: [PATCH] ARM fixed-point support [6/6]: target-specific parts

2011-05-26 Thread Julian Brown
On Fri, 13 May 2011 17:31:18 +0100 Julian Brown wrote: > On Fri, 13 May 2011 14:54:47 + (UTC) > "Joseph S. Myers" wrote: > > > On Fri, 13 May 2011, Julian Brown wrote: > > > > > although Joseph's comments have (hopefully) all been addressed. > > > One must now configure with "--enable-fixe

Re: [PATCH] ARM fixed-point support [6/6]: target-specific parts

2011-05-13 Thread Julian Brown
On Fri, 13 May 2011 14:54:47 + (UTC) "Joseph S. Myers" wrote: > On Fri, 13 May 2011, Julian Brown wrote: > > > although Joseph's comments have (hopefully) all been addressed. One > > must now configure with "--enable-fixed-point" to enable the > > fixed-point support. > > No, that's not how

Re: [PATCH] ARM fixed-point support [6/6]: target-specific parts

2011-05-13 Thread Joseph S. Myers
On Fri, 13 May 2011, Julian Brown wrote: > although Joseph's comments have (hopefully) all been addressed. One > must now configure with "--enable-fixed-point" to enable the > fixed-point support. No, that's not how this configure logic works; it enables fixed point by default for the given targ

[PATCH] ARM fixed-point support [6/6]: target-specific parts

2011-05-13 Thread Julian Brown
This is the last patch to add fixed-point support to the ARM backend, the actual ARM-specific backend parts. It's functionally unchanged from the earlier patch: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00837.html although Joseph's comments have (hopefully) all been addressed. One must now con