From owner-freebsd-current@FreeBSD.ORG  Fri Dec 23 19:04:31 2011
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Delivered-To: freebsd-current@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 2A82D1065688;
	Fri, 23 Dec 2011 19:04:31 +0000 (UTC) (envelope-from dim@FreeBSD.org)
Received: from tensor.andric.com (cl-327.ede-01.nl.sixxs.net
	[IPv6:2001:7b8:2ff:146::2])
	by mx1.freebsd.org (Postfix) with ESMTP id DCD848FC17;
	Fri, 23 Dec 2011 19:04:30 +0000 (UTC)
Received: from [IPv6:2001:7b8:3a7:0:8108:6e02:8e8c:934f] (unknown
	[IPv6:2001:7b8:3a7:0:8108:6e02:8e8c:934f])
	(using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits))
	(No client certificate requested)
	by tensor.andric.com (Postfix) with ESMTPSA id 1F6B35C37;
	Fri, 23 Dec 2011 20:04:30 +0100 (CET)
Message-ID: <4EF4D0C0.7080808@FreeBSD.org>
Date: Fri, 23 Dec 2011 20:04:32 +0100
From: Dimitry Andric <dim@FreeBSD.org>
Organization: The FreeBSD Project
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
	rv:9.0) Gecko/20111214 Thunderbird/9.0
MIME-Version: 1.0
To: Kostik Belousov <kostikbel@gmail.com>
References: <4EF34E52.2040905@FreeBSD.org> <20111223005932.GA65042@freebsd.org>
	<25FBBF23-CDFA-429E-966D-A90409D8F2BD@gmail.com>
	<201112230937.08971.jhb@freebsd.org>
	<20111223160032.GA18839@freebsd.org> <4EF4B46E.7000405@FreeBSD.org>
	<20111223175504.GK50300@deviant.kiev.zoral.com.ua>
In-Reply-To: <20111223175504.GK50300@deviant.kiev.zoral.com.ua>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Cc: Garrett Cooper <yanegomi@gmail.com>, Alexander Best <arundel@freebsd.org>,
	freebsd-current@freebsd.org, Benjamin Kaduk <kaduk@mit.edu>
Subject: Re: [patch] Cleaning up amd64 kernel optimization options
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>, 
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 23 Dec 2011 19:04:31 -0000

On 2011-12-23 18:55, Kostik Belousov wrote:
> On Fri, Dec 23, 2011 at 06:03:42PM +0100, Dimitry Andric wrote:
...
>> The only thing my patch makes sure of, is that amd64 does the same thing
>> as all other arches, e.g.: compile with a low optimization settings for
>> debug (-O, which is equivalent to -O1), compile with arch-specific high
>> optimization settings for release (-O2 plus whatever is required for the
>> arch, or lower if optimization breaks things).
>
> Release is built with -g for long time, this is where the symbol files
> in /boot/kernel comes from.

Ah, that is done via 'makeoptions DEBUG=-g' in the kernel configuration
file, right?  I didn't realize that was kept in for a release.  But even
in that case, amd64 is somehow different from the other arches, which
all get compiled with -O instead.

If people prefer that to stay as it is, I'll change the diff so only
-frename-registers gets removed when clang is used, as clang does not
support this flag.