This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
RE: [PATCH] boehm gc support for CRIS
- From: "Boehm, Hans" <hans dot boehm at hp dot com>
- To: "Hans-Peter Nilsson" <hans-peter dot nilsson at axis dot com>
- Cc: <simon dot posnjak at cetrtapot dot si>, <tromey at redhat dot com>,<rmathew at gmail dot com>, <green at redhat dot com>, <java-patches at gcc dot gnu dot org>
- Date: Fri, 12 Nov 2004 14:08:09 -0800
- Subject: RE: [PATCH] boehm gc support for CRIS
I finally put this into my source trees. It will show up in GC6.4
and later, and in 7.0alpha2 and later. I made the changes suggested
below.
I did not check this into gcc, but that would be a good idea, if it
hasn't been done.
(The 7.0alpha2 version will probably require some work when we get
there. I tried, but I probably broke something.)
Hans
> -----Original Message-----
> From: java-patches-owner@gcc.gnu.org
> [mailto:java-patches-owner@gcc.gnu.org]On Behalf Of Hans-Peter Nilsson
> Sent: Friday, October 29, 2004 4:01 AM
> To: Boehm, Hans
> Cc: simon.posnjak@cetrtapot.si; hans-peter.nilsson@axis.com;
> tromey@redhat.com; rmathew@gmail.com; green@redhat.com;
> java-patches@gcc.gnu.org
> Subject: Re: [PATCH] boehm gc support for CRIS
>
>
> > From: Simon Posnjak <simon.posnjak@cetrtapot.si>
> > Date: Fri, 29 Oct 2004 10:00:04 +0200
>
> > Content-Type: text/plain; charset=UTF-8
> > Content-Transfer-Encoding: 8bit
> >
> > V Ä?et, 28.10.2004 ob 21:08 je Boehm, Hans napisal(a):
> > > In that case, I'm basically fine with the patch.
> > >
> > > Simon - Can you resubmit the patch with the small change from
> > > CRIS32 to CRIS? That will reduce the chance of a misapplication
> > > on my part, or divergence between the gcc and my trees.
> > Attached.
>
> When this is applied to the master GC sources, is there any
> chance of also fixing:
>
> The misspelling in the gc_locks.h part of the patch? "Riped" ->
> "Ripped".
>
> The comment above the asm needs updating too; it refers to a
> variable "spinlock" which here is "addr".
>
> Thanks!
>
> brgds, H-P
> PS. If you'd rather have an updated patch, I can do that.
>