This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [Patch, microblaze]: Remove SECONDARY_MEMORY_NEEDED
- From: David Holsgrove <david dot holsgrove at xilinx dot com>
- To: Michael Eager <eager at eagerm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Edgar Iglesias <edgari at xilinx dot com>, John Williams <jwilliams at xilinx dot com>, Vidhumouli Hunsigida <vidhum at xilinx dot com>, Nagaraju Mekala <nmekala at xilinx dot com>
- Date: Wed, 22 Jan 2014 03:43:13 +0000
- Subject: RE: [Patch, microblaze]: Remove SECONDARY_MEMORY_NEEDED
- Authentication-results: sourceware.org; auth=none
- References: <122ea32d-6164-4635-99a8-c563dc1368a2 at AM1EHSMHS005 dot ehs dot local> <CAM=EW8Zf_msEBF-JpCV=jnM5xC5b5-BKAg69q3=Vy7E1bqY+3w at mail dot gmail dot com> <CAM=EW8aCSxX3p5=RO+SrogdjJNcBAs7iM-AaGuy4K+2KrrbdOw at mail dot gmail dot com> <52D82865 dot 6050908 at eagerm dot com>
Hi Michael,
> -----Original Message-----
> From: Michael Eager [mailto:eager@eagerm.com]
> Sent: Friday, 17 January 2014 4:44 am
> To: David Holsgrove; gcc-patches@gcc.gnu.org
> Cc: Edgar Iglesias; John Williams; Vidhumouli Hunsigida; Nagaraju Mekala
> Subject: Re: [Patch, microblaze]: Remove SECONDARY_MEMORY_NEEDED
>
> On 11/25/13 23:51, David Holsgrove wrote:
> > Hi Michael,
> >
> > I've attached patch based on latest gcc master. Please let me know if
> > you need anything further.
> >
> > thanks,
> > David
> >
> > On 15 July 2013 14:44, David Holsgrove <david.holsgrove@xilinx.com> wrote:
> >> Hi Michael,
> >>
> >> On 18 March 2013 22:49, David Holsgrove <david.holsgrove@xilinx.com>
> wrote:
> >>> MicroBlaze doesn't have restrictions that would force us to
> >>> reload regs via memory. Don't define SECONDARY_MEMORY_NEEDED.
> >>> Fixes an ICE when compiling OpenSSL for linux.
> >>>
> >>> Changelog
> >>>
> >>> 2013-03-18 Edgar E. Iglesias <edgar.iglesias@xilinx.com>
> >>>
> >>> * gcc/config/microblaze/microblaze.h: Remove
> SECONDARY_MEMORY_NEEDED
> >>> definition.
> >>>
> >>> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
> >>> Signed-off-by: Peter A. G. Crosthwaite <peter.crosthwaite@xilinx.com>
> >>>
> >>
> >> Patch remains the same, please apply when ready.
> >>
> >> thanks,
> >> David
>
> Hi David --
>
> Is it possible to add a test case which shows the ICE?
>
I'm afraid I donât still have my test environment for this patch from last March, I'll attempt to recreate and distil into a small test case if possible, based on the error encountered whilst building openssl.
I'll update again when I have some further detail.
thanks,
David
> Thanks.
>
>
> --
> Michael Eager eager@eagercon.com
> 1960 Park Blvd., Palo Alto, CA 94306 650-325-8077