Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 19 Jan 2010 08:56:57 -0700
From:      Scott Long <scottl@samsco.org>
To:        Ed Schouten <ed@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r202628 - in head: . sys/amd64/amd64 sys/i386/i386 sys/i386/xen sys/sys
Message-ID:  <9FF943CE-9FCB-48C8-A048-23F23620F4E5@samsco.org>
In-Reply-To: <201001191531.o0JFVI6n029716@svn.freebsd.org>
References:  <201001191531.o0JFVI6n029716@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Jan 19, 2010, at 8:31 AM, Ed Schouten wrote:
> Author: ed
> Date: Tue Jan 19 15:31:18 2010
> New Revision: 202628
> URL: http://svn.freebsd.org/changeset/base/202628
>
> Log:
>  Recommit r193732:
>
>    Remove __gnu89_inline.
>
>    Now that we use C99 almost everywhere, just use C99-style in the  
> pmap
>    code. Since the pmap code is the only consumer of __gnu89_inline,  
> remove
>    it from cdefs.h as well. Because the flag was only introduced 17  
> months
>    ago, I don't expect any problems.
>
>    Reviewed by:    alc
>
>  It was backed out, because it prevented us from building kernels  
> using a
>  7.x compiler. Now that most people use 8.x, there is nothing that  
> holds
>  us back. Even if people run 7.x, they should be able to build a  
> kernel
>  if they run `make kernel-toolchain' or `make buildworld' first.

Ok, so you've broken a legitimate piece of compatibility.  What's the  
gain?

Scott




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?9FF943CE-9FCB-48C8-A048-23F23620F4E5>