Subject: Re: [boost] [config] Patch for clang.hpp to support clang in Windows using VC++ RTL
From: Daniel James (daniel_at_[hidden])
Date: 2013-10-14 10:29:03
On 14 October 2013 15:13, Edward Diener <eldiener_at_[hidden]> wrote:
> On 10/14/2013 6:26 AM, Daniel James wrote:
>>
>> On 14 October 2013 02:52, Edward Diener <eldiener_at_[hidden]> wrote:
>>>
>>>
>>> I would have used the BOOST_WORKAROUND macro but clang does not support a
>>> value which gives a single number for the compiler version so I felt that
>>> drumming up multiple BOOST_WORKAROUNDs using major/minor/patch version
>>> numbers was too much of a hack to bother about.
>>
>>
>> It's not recommended to use Clang version numbers at all:
>>
>> http://listarchives.boost.org/Archives/boost/2011/06/182574.php
>
>
> I am aware of that. I could not find a __has_feature/__has_extension for
> checking support for char16_t or char32_t in clang. Anyway my patch just
> checks for _MSC_VER for the time being.
Sorry, that wasn't meant to be criticism, I was saying that you're
doing the right thing. If they add support, hopefully they can give
some guidance as to how to detect it.