[Patch, microblaze]: Remove SECONDARY_MEMORY_NEEDED

David Holsgrove david.holsgrove@xilinx.com
Wed Jan 22 03:43:00 GMT 2014


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





More information about the Gcc-patches mailing list