[aspectc-user] Build failure with Clang 3.7.1

[Dan Fandrich]

Thanks for the fix--it's working fine for me.

On Sun, Feb 21, 2016 at 10:16:21AM +0100, Olaf Spinczyk wrote:
> Hi Dan!
> 
> Thanks a lot for this effort. In AspectC++ we don't print attributes of
> types at the moment. Therefore, I could comment out two critical switch
> statements. We will, of course, work on a better solution, but that has
> to wait until after the 2.0 release. I tested that the compiled ac++
> actually works and added 3.7.1 to the list of supported Clang versions.
> 
> - Olaf
> 
> 
> Am 20.02.2016 um 16:32 schrieb Dan Fandrich:
> > I'm getting a build failure with the daily source package of aspectc++ with
> > clang 3.7.1, starting with the check for specific clang versions:
> >
> >  #error "Invalid clang version used! Only 3.4 and 3.6.2 are supported by this code."
> >
> > It would be nice if the upcoming release supported the latest Clang, especially
> > since Aspect C++ releases are pretty infrequent. In fact, version 3.8.0
> > of clang was supposed to have been tagged two days ago should be released any
> > day now.
> >
> > The following patch makes it compile, but there are warnings about enumeration
> > value not handled so may need to be adjusted to properly handle the new enums
> > rather than just remove the removed ones.
> >
> > --- aspectc++/AspectC++/ClangAdjustedTypePrinter.cc.orig	2016-02-20 12:53:41.603561765 +0000
> > +++ aspectc++/AspectC++/ClangAdjustedTypePrinter.cc	2016-02-20 12:56:37.057777807 +0000
> > @@ -642,9 +642,6 @@
> >        case CC_AAPCS_VFP:
> >          OS << " __attribute__((pcs(\"aapcs-vfp\")))";
> >          break;
> > -      case CC_PnaclCall:
> > -        OS << " __attribute__((pnaclcall))";
> > -        break;
> >        case CC_IntelOclBicc:
> >          OS << " __attribute__((intel_ocl_bicc))";
> >          break;
> > @@ -1256,7 +1253,6 @@
> >       OS << ')';
> >       break;
> >      }
> > -    case AttributedType::attr_pnaclcall: OS << "pnaclcall"; break;
> >      case AttributedType::attr_inteloclbicc: OS << "inteloclbicc"; break;
> >      }
> >      OS << "))";
> > --- aspectc++/AspectC++/ClangBinding.cc.orig	2016-02-20 13:51:25.088377696 +0000
> > +++ aspectc++/AspectC++/ClangBinding.cc	2016-02-20 13:51:33.445301183 +0000
> > @@ -3,7 +3,7 @@
> >  #if CLANG_VERSION_MAJOR == 3 && CLANG_VERSION_MINOR == 4 && !defined(CLANG_VERSION_PATCHLEVEL)
> >  #elif CLANG_VERSION_MAJOR == 3 && CLANG_VERSION_MINOR == 6 && CLANG_VERSION_PATCHLEVEL == 2
> >  #else
> > -#error "Invalid clang version used! Only 3.4 and 3.6.2 are supported by this code."
> > +#warning "Invalid clang version used! Only 3.4 and 3.6.2 are supported by this code."
> >  #endif
> >  
> >  #include "ClangIntroSema.h"
> > _______________________________________________
> > aspectc-user mailing list
> > aspectc-user at aspectc.org
> > http://www.aspectc.org/mailman/listinfo/aspectc-user
> 
> _______________________________________________
> aspectc-user mailing list
> aspectc-user at aspectc.org
> http://www.aspectc.org/mailman/listinfo/aspectc-user