From owner-freebsd-arch@FreeBSD.ORG Tue Apr 19 11:21:33 2011 Return-Path: Delivered-To: arch@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D156C106566B; Tue, 19 Apr 2011 11:21:33 +0000 (UTC) (envelope-from alexander@leidinger.net) Received: from mail.ebusiness-leidinger.de (mail.ebusiness-leidinger.de [217.11.53.44]) by mx1.freebsd.org (Postfix) with ESMTP id 844E68FC15; Tue, 19 Apr 2011 11:21:33 +0000 (UTC) Received: from outgoing.leidinger.net (p5B1557F8.dip.t-dialin.net [91.21.87.248]) by mail.ebusiness-leidinger.de (Postfix) with ESMTPSA id 648F6844015; Tue, 19 Apr 2011 13:21:18 +0200 (CEST) Received: from webmail.leidinger.net (webmail.Leidinger.net [IPv6:fd73:10c7:2053:1::2:102]) by outgoing.leidinger.net (Postfix) with ESMTP id 19ECA1CDF; Tue, 19 Apr 2011 13:21:15 +0200 (CEST) Received: (from www@localhost) by webmail.leidinger.net (8.14.4/8.13.8/Submit) id p3JBLEEB085729; Tue, 19 Apr 2011 13:21:14 +0200 (CEST) (envelope-from Alexander@Leidinger.net) Received: from pslux.ec.europa.eu (pslux.ec.europa.eu [158.169.9.14]) by webmail.leidinger.net (Horde Framework) with HTTP; Tue, 19 Apr 2011 13:21:14 +0200 Message-ID: <20110419132114.43009vpwaeidet68@webmail.leidinger.net> Date: Tue, 19 Apr 2011 13:21:14 +0200 From: Alexander Leidinger To: Dimitry Andric References: <4DAC3FD9.1060603@FreeBSD.org> <20110418103416.76b07b36@kan.dnsalias.net> <4DAC9534.60009@FreeBSD.org> In-Reply-To: <4DAC9534.60009@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; DelSp="Yes"; format="flowed" Content-Disposition: inline Content-Transfer-Encoding: 7bit User-Agent: Dynamic Internet Messaging Program (DIMP) H3 (1.1.6) X-EBL-MailScanner-Information: Please contact the ISP for more information X-EBL-MailScanner-ID: 648F6844015.AE694 X-EBL-MailScanner: Found to be clean X-EBL-MailScanner-SpamCheck: not spam, SpamAssassin (not cached, score=0, required 6, autolearn=disabled) X-EBL-MailScanner-From: alexander@leidinger.net X-EBL-MailScanner-Watermark: 1303816878.61879@3G7XlhemNpacWYtsV3GnLA X-EBL-Spam-Status: No Cc: "arch@freebsd.org" , "current@freebsd.org" Subject: Re: Removal of ICC (intel compiler) bits from mk X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Apr 2011 11:21:33 -0000 Quoting Dimitry Andric (from Mon, 18 Apr 2011 21:47:00 +0200): > On 2011-04-18 16:34, Alexander Kabaev wrote: > ... >>> Please review the attached patch, which cleans up the ICC bits. >> You probably also want to remove the code that uses __INTEL_COMPILER >> in source tree if you want to be really thorough. > > Ok, new patch attached. Some of the contributed sources also contain > instances of __INTEL_COMPILER, but these are all from upstream, so I > left them in. Is it really necessary to remove them from cdefs.h? If someone takes parts of our source and wants to compile them on linux or windows with icc, the icc parts in cdefs.h are a valuable part to have. Info: AFAIR I didn't add icc stuff to csup, and AFAIR csup was added when our icc in the ports collection was outdated, so I assume someone tested csup with icc on linux and addid this stuff there. Removing it from csup seems to be a step backwards to me in this case. Regarding the build infrastructure part I just skimmed over it, and the math/std* headers and 1s-complement-checksum part looks ok to me. Bye, Alexander. -- Intel CPUs are not defective, they just act that way. -- Henry Spencer http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID = B0063FE7 http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID = 72077137