Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 4 Oct 1999 08:47:01 -0400 (EDT)
From:      Luoqi Chen <luoqi@watermarkgroup.com>
To:        alc@cs.rice.edu, eivind@FreeBSD.org
Cc:        cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org
Subject:   Re: cvs commit: src/sys/i386/include atomic.h
Message-ID:  <199910041247.IAA29554@lor.watermarkgroup.com>

next in thread | raw e-mail | index | archive | help
> On Sun, Oct 03, 1999 at 04:35:44PM -0500, Alan Cox wrote:
> > On Sun, Oct 03, 1999 at 02:15:25PM -0700, Eivind Eklund wrote:
> > > eivind      1999/10/03 14:15:25 PDT
> > > 
> > >   Modified files:
> > >     sys/i386/include     atomic.h 
> > >   Log:
> > >   Allow compilation with older versions of GCC, in order to make it possible
> > >   to bootstrap and work with -current from older versions of FreeBSD.
> > >   
> > 
> > I'm curious.  Why didn't you base this change on revision 1.1.2.1?  The
> > implementation that you used is known to produce non-atomic code under
> > gcc 2.7.2.x.
> 
> Because I didn't notice 1.1.2.1; I reviewed the diffs backwards along
> -current only :-( I'll fix it tomorrow - I'm too tired now to want to
> mess around in the code.
> 
> Eivind.
> 
Please also append another underscore to __GNUC_MINOR_, and shouldn't the
gcc minor version test be ">= 9" instead of "> 9"?

-lq


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199910041247.IAA29554>