Code review comment for lp:~zorba-coders/zorba/clang-friend

Revision history for this message
David Graf (davidagraf) wrote :

> The changes look ok, but I'm not sure how to test.
> I tried to build this branch using clang on MacOS X, but it doesn't build for
> me.
> What exactly should work now that did not work before?

Hello Till

Sorry for not answering earlier. The 'defined __llvm__' condition makes Zorba compiling in XCode with clang/llvm. The other stuff fixes warnings.

« Back to merge proposal