X-Original-To: alpine-devel@lists.alpinelinux.org Received: from mx1.tetrasec.net (mx1.tetrasec.net [74.117.190.25]) by lists.alpinelinux.org (Postfix) with ESMTP id 873555C0048 for ; Wed, 25 Oct 2017 14:46:20 +0000 (GMT) Received: from mx1.tetrasec.net (mail.local [127.0.0.1]) by mx1.tetrasec.net (Postfix) with ESMTP id 2CA1F9E3174 for ; Wed, 25 Oct 2017 14:46:20 +0000 (GMT) Received: from ncopa-desktop.copa.dup.pw (15.63.200.37.customer.cdi.no [37.200.63.15]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: n@tanael.org) by mx1.tetrasec.net (Postfix) with ESMTPSA id A2C519E25CD for ; Wed, 25 Oct 2017 14:46:19 +0000 (GMT) Date: Wed, 25 Oct 2017 16:46:14 +0200 From: Natanael Copa To: alpine-devel@lists.alpinelinux.org Subject: [alpine-devel] force compile flag for musl? Message-ID: <20171025164614.14c4c57e@ncopa-desktop.copa.dup.pw> X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-alpine-linux-musl) X-Mailinglist: alpine-devel Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Hi, I wonder what you think about overriding the -Os compile flag for musl, and hardcode it to -O2. I think this makes sense since the functions in libc are so often used that we want trade better performance at the cost of slightly bigger binary. This means that we override whatever user as set CFLAGS to in /etc/abuild.conf We already do this with zlib. What do you think? diff --git a/main/musl/APKBUILD b/main/musl/APKBUILD index 1938bbb3ca..193002186d 100644 --- a/main/musl/APKBUILD +++ b/main/musl/APKBUILD @@ -54,6 +54,8 @@ build() { fi # note: not autotools + # force -O2 compile flag for better performance + CFLAGS="-O2" \ LDFLAGS="$LDFLAGS -Wl,-soname,libc.musl-${CARCH}.so.1" \ ./configure \ --build=$CBUILD \ -nc --- Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org Help: alpine-devel+help@lists.alpinelinux.org ---