This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RS6000] Fix -m32 -Os lossage
- From: Alan Modra <amodra at gmail dot com>
- To: David Edelsohn <dje dot gcc at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Segher Boessenkool <segher at kernel dot crashing dot org>
- Date: Fri, 13 Feb 2015 10:53:39 +1030
- Subject: Re: [RS6000] Fix -m32 -Os lossage
- Authentication-results: sourceware.org; auth=none
- References: <20150212123357 dot GO4274 at bubble dot grove dot modra dot org> <CAGWvny=6iDmcHmRY3S+MdmGx4TWj2U5H1viZukrLMhiYADDV4A at mail dot gmail dot com> <20150212221030 dot GP4274 at bubble dot grove dot modra dot org> <CAGWvnyk+SyqGx6XXUox7xKk1Zq33kPxfNbxijEu-Ei-pT6MB3w at mail dot gmail dot com>
On Thu, Feb 12, 2015 at 07:01:12PM -0500, David Edelsohn wrote:
> On Thu, Feb 12, 2015 at 5:10 PM, Alan Modra <amodra@gmail.com> wrote:
> > On Thu, Feb 12, 2015 at 10:04:45AM -0500, David Edelsohn wrote:
> >> On Thu, Feb 12, 2015 at 7:33 AM, Alan Modra <amodra@gmail.com> wrote:
> >> > * config/rs6000/rs6000.c (rs6000_emit_epilogue): Use addsi3_carry
> >> > or adddi3_carry when restoring frame_reg_rtx from r0 after restvr.
> >>
> >> Okay.
> >
> > Committed revision 220665. I also intend to commit the following as
> > obvious when stage1 opens, to fix a bug in code I contributed. It's
> > not a regression nor is the generated code incorrect, but we have one
> > unnecessary instruction.
>
> > * config/rs6000/rs6000.c (rs6000_emit_epilogue): Fix typo in
> > code setting up r11 for out-of-line fp restore.
>
> Go ahead and commit the fix now.
OK, thanks. Done after bootstrap etc., revision 220672.
--
Alan Modra
Australia Development Lab, IBM