From owner-svn-src-all@FreeBSD.ORG Thu Dec 4 17:10:26 2014 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id B41ECED; Thu, 4 Dec 2014 17:10:26 +0000 (UTC) Received: from smtp.des.no (smtp.des.no [194.63.250.102]) by mx1.freebsd.org (Postfix) with ESMTP id 7317E3E2; Thu, 4 Dec 2014 17:10:26 +0000 (UTC) Received: from nine.des.no (smtp.des.no [194.63.250.102]) by smtp-int.des.no (Postfix) with ESMTP id 73FE4AF5E; Thu, 4 Dec 2014 17:10:25 +0000 (UTC) Received: by nine.des.no (Postfix, from userid 1001) id 2897919F9; Thu, 4 Dec 2014 18:10:23 +0100 (CET) From: =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= To: Dimitry Andric Subject: Re: svn commit: r275468 - head/sys/dev/usb/controller References: <201412032155.sB3LtjJN043364@svn.freebsd.org> <20141204005451.GM99957@funkthat.com> <1373EB55-4238-40D0-B481-53C8B46D6E6C@FreeBSD.org> Date: Thu, 04 Dec 2014 18:10:23 +0100 In-Reply-To: <1373EB55-4238-40D0-B481-53C8B46D6E6C@FreeBSD.org> (Dimitry Andric's message of "Thu, 4 Dec 2014 17:55:40 +0100") Message-ID: <86wq67e3y8.fsf@nine.des.no> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (berkeley-unix) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, John-Mark Gurney , src-committers@freebsd.org, Hans Petter Selasky X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Dec 2014 17:10:26 -0000 Dimitry Andric writes: > Maybe it is easier to alias ffs() and friends to __builtin_ffs(), > since the compilers we support have these builtins already. That should already be the case on platforms that support it. There is absolutely no reason to hand-roll an ffs() equivalent. (unfortunately, only i386 and amd64 do this; someone[tm] should fix the others) DES --=20 Dag-Erling Sm=C3=B8rgrav - des@des.no