This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [Patch,AVR] PR54222: Add fixed point support
- From: "Weddington, Eric" <Eric dot Weddington at atmel dot com>
- To: Georg-Johann Lay <avr at gjlay dot de>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Denis Chertykov <chertykov at gmail dot com>, Sean D'Epagnier <sean at depagnier dot com>, Joerg Wunsch <joerg_wunsch at uriah dot heep dot sax dot de>
- Date: Fri, 10 Aug 2012 16:08:59 +0000
- Subject: RE: [Patch,AVR] PR54222: Add fixed point support
- References: <50252E25.3020901@gjlay.de>
> -----Original Message-----
> From: Georg-Johann Lay
> Sent: Friday, August 10, 2012 9:52 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Denis Chertykov; Weddington, Eric; Sean D'Epagnier; Joerg Wunsch
> Subject: [Patch,AVR] PR54222: Add fixed point support
>
> This patch adds fixed point support to the avr target.
Hi Johann,
Thanks for doing this work to get Sean's patch into GCC! :-)
> It's based on the work of Sean, see
>
> http://lists.gnu.org/archive/html/avr-gcc-list/2012-07/msg00030.html
>
> This patch has several changes compared to Sean's patch:
>
<snip>
> * The patch neither implements NEG nor ABS. The standard requires
> saturation, e.g. -0x80 must not become 0x80 again. This is work
> still to be done, but just a matter of optimization.
Can I assume that you have plans to do this in the near-ish future?
<snip>
> The patch works out fine. However, because of PR53923 which
> shreds the AVR port, currently no reasonable testing is possible.
>
> Work to be done is better testing after PR53923 is fixed and the
> AVR port works properly again.
Is there a reasonable time frame as to when PR53923 will be fixed? My only real concern is that this is a major feature addition and the AVR port is currently broken.
Thanks,
Eric Weddington