Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 19 Apr 2011 13:21:14 +0200
From:      Alexander Leidinger <Alexander@Leidinger.net>
To:        Dimitry Andric <dim@FreeBSD.org>
Cc:        "arch@freebsd.org" <arch@FreeBSD.org>, "current@freebsd.org" <current@FreeBSD.org>
Subject:   Re: Removal of ICC (intel compiler) bits from mk
Message-ID:  <20110419132114.43009vpwaeidet68@webmail.leidinger.net>
In-Reply-To: <4DAC9534.60009@FreeBSD.org>
References:  <4DAC3FD9.1060603@FreeBSD.org> <20110418103416.76b07b36@kan.dnsalias.net> <4DAC9534.60009@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Quoting Dimitry Andric <dim@FreeBSD.org> (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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110419132114.43009vpwaeidet68>