Hi Michael, > -----Original Message----- > From: Michael Eager [mailto:ea...@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.holsgr...@xilinx.com> wrote: > >> Hi Michael, > >> > >> On 18 March 2013 22:49, David Holsgrove <david.holsgr...@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.igles...@xilinx.com> > >>> > >>> * gcc/config/microblaze/microblaze.h: Remove > SECONDARY_MEMORY_NEEDED > >>> definition. > >>> > >>> Signed-off-by: Edgar E. Iglesias <edgar.igles...@xilinx.com> > >>> Signed-off-by: Peter A. G. Crosthwaite <peter.crosthwa...@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 ea...@eagercon.com > 1960 Park Blvd., Palo Alto, CA 94306 650-325-8077