Boost logo

Boost :

Subject: Re: [boost] [config] Patch for clang.hpp to support clang in Windows using VC++ RTL
From: Edward Diener (eldiener_at_[hidden])
Date: 2013-10-14 10:13:09


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://lists.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.

>
> I suppose using them in combination with checking for _MSC_VER is safe
> at the moment, but might not always be.

Understood.


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk