Re: [Boost-build] Patch to clear Intel /Op deprecated warning

Subject: Re: [Boost-build] Patch to clear Intel /Op deprecated warning
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2011-11-27 18:11:20

AMDG

On 11/27/2011 08:06 AM, Beman Dawes wrote:
> Beman Dawes <bdawes <at> acm.org> writes:
>> Steven Watanabe <watanabesj <at> gmail.com> writes:
>>
>>>
>>> On 11/26/2011 06:03 AM, Beman Dawes wrote:
>>>> The patch below clears a warning from Intel 12.1, and works fine for
>>>> MSVC 8, 9, and 10.
>>>>
>>>> Unless someone objects, I'll apply it tomorrow. I'm posting advance
>>>> notice in case any of the math folks want to give it a try beforehand.
>>>>
>>>
>>> I object.  This branch is for msvc 6 and 7, which
>>> only have /Op.  /fp:precise is for msvc 8 and
>>> up.  The real problem is that intel-win is assuming
>>> vc7.1 compatibility.
>>
>> What would a better fix be? A patch for intel-win? Can you provide it?
> 
> I tried changing intel-win to default compatibility to vc10, and that apparently
> works. But I don't understand the implications of that change, so would like
> confirmation that it is correct before committing it.
> 

It's not any worse than the current default, anyway.

>From http://tinyurl.com/7ndl8lj, it looks
like icl.exe picks the default vc version
by searching PATH for cl.exe.  I think the
correct thing for Boost.Build is to use
the most recent installed version of msvc
and always pass /QvcXX explicitly.

In Christ,
Steven Watanabe