Re: Removal of ICC (intel compiler) bits from mk

From: Alexander Leidinger <Alexander_at_Leidinger.net>
Date: Wed, 20 Apr 2011 11:15:10 +0200
Quoting Dimitry Andric <dim_at_FreeBSD.org> (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 _at_ Leidinger.net: PGP ID = B0063FE7
http://www.FreeBSD.org       netchild _at_ FreeBSD.org  : PGP ID = 72077137
Received on Wed Apr 20 2011 - 07:15:32 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:13 UTC