From owner-freebsd-arch@FreeBSD.ORG Wed Apr 20 09:15:32 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 BA5BB106564A; Wed, 20 Apr 2011 09:15:32 +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 6C5408FC0A; Wed, 20 Apr 2011 09:15:32 +0000 (UTC) Received: from outgoing.leidinger.net (p5B155769.dip.t-dialin.net [91.21.87.105]) by mail.ebusiness-leidinger.de (Postfix) with ESMTPSA id 65637844015; Wed, 20 Apr 2011 11:15:15 +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 8B8CB1E07; Wed, 20 Apr 2011 11:15:11 +0200 (CEST) Received: (from www@localhost) by webmail.leidinger.net (8.14.4/8.13.8/Submit) id p3K9FB6D026294; Wed, 20 Apr 2011 11:15:11 +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; Wed, 20 Apr 2011 11:15:10 +0200 Message-ID: <20110420111510.31806jq0ffewdrgo@webmail.leidinger.net> Date: Wed, 20 Apr 2011 11:15:10 +0200 From: Alexander Leidinger To: Dimitry Andric References: <4DAC3FD9.1060603@FreeBSD.org> <20110418103416.76b07b36@kan.dnsalias.net> <4DAC9534.60009@FreeBSD.org> <20110419132114.43009vpwaeidet68@webmail.leidinger.net> <4DADCBB4.8060301@FreeBSD.org> In-Reply-To: <4DADCBB4.8060301@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: 65637844015.ADCC7 X-EBL-MailScanner: Found to be clean X-EBL-MailScanner-SpamCheck: not spam, spamhaus-ZEN, SpamAssassin (not cached, score=0.077, required 6, autolearn=disabled, TW_TD 0.08) X-EBL-MailScanner-From: alexander@leidinger.net X-EBL-MailScanner-Watermark: 1303895718.86926@UBtFnLfq1mwvwQFcEk5elA 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: Wed, 20 Apr 2011 09:15:32 -0000 Quoting Dimitry Andric (from Tue, 19 Apr 2011 19:51:48 +0200): > On 2011-04-19 13:21, Alexander Leidinger wrote: > ... >>>> 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. > > Isn't that the case for the other headers too? Except the in_cksum.[ch] > files, where the 'fix' for ICC is apparently not needed at all anymore. The stdbool header is somewhat tied to our system I would say. The math one is AFAIK about building the math lib itself. Just from reading the diff I would assume this could manifest itself by compile errors (duplicate definition). I do not know how valuable it is to build this somewhere else. IMO they can stay in or they can be removed. Both should be easy to debug/redo in case of problems. The cdefs header is differntly in this regard, only for some parts you will get easy to debug error cases, other cases are silent (you don't know where it happens) corruption or bad performance. Bye, Alexander. -- Coming together is a beginning; keeping together is progress; working together is success. http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID = B0063FE7 http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID = 72077137