Mailing-List: contact stdcxx-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: stdcxx-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of Farid_Zaripov@epam.com
 designates 217.21.63.3 as permitted sender)
Content-class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Subject: RE: svn commit: r574015 -
 /incubator/stdcxx/trunk/etc/config/windows/projectdef.js
Date: Mon, 10 Sep 2007 20:41:27 +0300
Message-ID: <7BDB2168BEAEF14C98F1901FD2DE6438E9A88F@epmsa009.minsk.epam.com>
In-Reply-To: <46E5641C.7000205@roguewave.com>
Thread-Topic: svn commit: r574015 -
 /incubator/stdcxx/trunk/etc/config/windows/projectdef.js
Thread-Index: AcfzwFhld4iOfPqiRTSQmTYpznKZvAAC3vQg
From: "Farid Zaripov" <Farid_Zaripov@epam.com>
To: <stdcxx-dev@incubator.apache.org>

> -----Original Message-----
> From: Martin Sebor [mailto:sebor@roguewave.com]=20
> Sent: Monday, September 10, 2007 6:35 PM
> To: stdcxx-dev@incubator.apache.org
> Subject: Re: svn commit: r574015 -=20
> /incubator/stdcxx/trunk/etc/config/windows/projectdef.js
>=20
> faridz@apache.org wrote:
> > Author: faridz
> > Date: Sun Sep  9 06:57:32 2007
> > New Revision: 574015
> >=20
> > URL: http://svn.apache.org/viewvc?rev=3D574015&view=3Drev
> > Log:
> > 2007-09-09 Farid Zaripov <Farid_Zaripov@epam.com>
> >=20
> > 	* projectdef.js (projectCreateVCProject): Enable using of the
> > 	intrinsic functions in release builds.
>=20
> Unless it's obvious, it's helpful to say why a change was made.
> You might know why now and some of the rest of us who read=20
> the list might be able to guess (to enable the Interlocked functions,
> right?) but others who just read the ChangeLog may not.

  No. The Interlocked functions will be intrinsic in all builds, because
of
presence of the #pragma intrinsic (Interlockedxxx) in _mutex.h

  The purpose of this patch is enable using of some CRT functions
(memcmp, memcpy, memset, ...) as intrinsic in release builds.
The full list of intrinsic CRT functions here:
http://msdn2.microsoft.com/en-us/library/tzkfha43(VS.80).aspx

  But now I can say, that this patch is useless. The projects in
release builds are compiled with /O1 optimization, which not includes
/Oi option. And setting of the EnableIntrinsicFunctions property to true
doesn't leads to compiling with /Oi option unless /O1 or /O2 options
are not specified. So we should append /Oi to the command line manually
(using AdditionalOptions property) to enable using of CRT functions
as intrinsic (or use /O2 optimization, because /O2 implies /Oi).

Farid.