On Fri, Nov 16, 2012 at 12:29 AM, Gennadiy Rozental <rogeeff@gmail.com> wrote:

Dean Michael Berris <mikhailberis <at> gmail.com> writes:

...

I'm trying to build Boost.Test in trunk in C++11 mode with Apple's released Clang compiler with Xcode 4.5. Is this supported by Boost.Test?

Well is is intended to work ;)

Thanks, that's good to know. :)

...

I keep getting this error somewhere deep in some preprocessor parts which is far outside my understanding.

Would help if you can show it. I do not have access to this configuration

BTW,

clang seems to be working fine in general according to this:

http://www.boost.org/development/tests/trunk/developer/test.html

Interesting. Is there a way to find out exactly what version of clang Sandia is using? If anything this may be a regression on clang I'm encountering. I have added more details to https://svn.boost.org/trac/boost/ticket/7697 which shows exactly what the error I encounter is. It seems to have something to do with decltype and boost::declval. Also, if it helps I'm on r81355. Thanks Gennadiy! -- Dean Michael Berris www.deanberris.com

On 15 November 2012 14:54, Daniel James <dnljms@gmail.com> wrote:

I tried building with Clang 3.1 in C++11 mode, and I got the same errors as you.

I think it's a clang bug, here's a test case: template <typename T> struct foo {}; foo<decltype(1 > 0)> x; It thinks the '>' from '1 > 0' is an angle bracket, I guess it has been fixed on clang trunk. A workaround is just to double the round brackets. I've attached a patch that I used to get Boost.Test to compile. It's a bit messy though.

On 15 November 2012 15:32, Andrey Semashev <andrey.semashev@gmail.com> wrote:

On Thu, Nov 15, 2012 at 7:24 PM, Daniel James <dnljms@gmail.com> wrote:

...

On 15 November 2012 14:54, Daniel James <dnljms@gmail.com> wrote:

...

I tried building with Clang 3.1 in C++11 mode, and I got the same errors as you.

I think it's a clang bug, here's a test case:

template <typename T> struct foo {}; foo<decltype(1 > 0)> x;

It thinks the '>' from '1 > 0' is an angle bracket, I guess it has been fixed on clang trunk. A workaround is just to double the round brackets. I've attached a patch that I used to get Boost.Test to compile. It's a bit messy though.

Note there is difference between decltype(x) and decltype((x)), so the change may alter the behavior. I didn't examine the code to confirm that though, just a heads up.

Well, that's annoying. Just had a very quick read up on the issue, and I think it's okay. The expression is: typename boost::remove_reference< decltype((boost::declval<Lhs>() oper boost::declval<Rhs>() ))>::type result_type; 'oper' is a macro parameter, which is never '.' or '->' (obviously, given that it wouldn't work). So I don't think it counts as an id or member expression. 'oper' is '->*' in one case, but I don't think that's affected? If it is, the 'remove_reference' should help, maybe it would need to use 'remove_const' as well? Btw. please can someone with clang from subversion try the test case?

On 15 November 2012 20:01, Marshall Clow <mclow.lists@gmail.com> wrote:

On Nov 15, 2012, at 11:32 AM, Daniel James <dnljms@gmail.com> wrote:

...

Btw. please can someone with clang from subversion try the test case?

Coming in late - where's the test case?

Can you compile this is C++11 mode? template <typename T> struct foo {}; foo<decltype(1 > 0)> x;

On 15 November 2012 20:15, Marshall Clow <mclow.lists@gmail.com> wrote:

Yes, I can.

$ cat junk2.cpp template <typename T> struct foo {}; foo<decltype(1 > 0)> x; $ totclang11 junk2.cpp -c -Wall

Thanks, it's good to confirm that it's fixed.

On 16 November 2012 00:52, Gennadiy Rozental <rogeeff@gmail.com> wrote:

I checked in a fix, which just split above into two lines. The rest should work as is I believe.

It isn't going to work for people using a C++11 compiler with an older standard library. This is quite common for people using clang, since libc++ is not easy to install on anything other than a Mac. I don't know how much of a problem it will be, but I did get bug reports when I didn't deal with that situation.