This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [Patch,AVR]: Fix PR50449
- 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>
- Cc: "Anatoly Sokolov" <aesok at post dot ru>, "Denis Chertykov" <chertykov at gmail dot com>
- Date: Tue, 20 Sep 2011 12:41:30 -0600
- Subject: RE: [Patch,AVR]: Fix PR50449
- References: <4E78DB15.3040803@gjlay.de>
> -----Original Message-----
> From: Georg-Johann Lay [mailto:avr@gjlay.de]
> Sent: Tuesday, September 20, 2011 12:28 PM
> To: gcc-patches@gcc.gnu.org
> Cc: Weddington, Eric; Anatoly Sokolov; Denis Chertykov
> Subject: [Patch,AVR]: Fix PR50449
>
> This is fix for a minor performance regression introduced by my changes in
> trunk r175956: To load a 32-bit constant like 1 into R2, 4.6 uses
>
> CLR R2
> CLR R3
> MOVW R4,R2
> INC R2
>
> whereas trunk prints the longer
>
> CLR R2
> INC R2
> CLR R3
> CLR R4
> CLR R5
>
> This patch fixes it. The insns affected (*reload_insi and *reload_insf)
> whose
> instruction length must be adjusted now use an insn attribute to express
> how
> the adjustment has to be performed.
>
>
> Tested without regressions.
>
> Ok to commit?
>
Please commit.
Eric