> -----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
