The methods visited for a special class must have an identifier.
authorZahira Ammarguellat <zahira.ammarguellat@intel.com>
Wed, 2 Feb 2022 19:28:44 +0000 (11:28 -0800)
committerZahira Ammarguellat <zahira.ammarguellat@intel.com>
Wed, 2 Feb 2022 21:12:33 +0000 (13:12 -0800)
clang/lib/Sema/SemaDecl.cpp
clang/test/SemaSYCL/special-class-attribute.cpp

index cbd9df4..a86947a 100644 (file)
@@ -9193,7 +9193,7 @@ Sema::ActOnFunctionDeclarator(Scope *S, Declarator &D, DeclContext *DC,
       }
       if ((Parent->isClass() || Parent->isStruct()) &&
           Parent->hasAttr<SYCLSpecialClassAttr>() &&
-          NewFD->getKind() == Decl::Kind::CXXMethod &&
+          NewFD->getKind() == Decl::Kind::CXXMethod && NewFD->getIdentifier() &&
           NewFD->getName() == "__init" && D.isFunctionDefinition()) {
         if (auto *Def = Parent->getDefinition())
           Def->setInitMethod(true);
index f06a7b1..1bfe7cc 100644 (file)
@@ -30,6 +30,14 @@ class class5 {
   void __init(){}
 };
 
+struct __attribute__((sycl_special_class)) struct6 {
+  struct6();
+  bool operator==(const struct6 &);
+  struct6 &operator()();
+  ~struct6();
+  void __init(){}
+};
+
 // Must have one and only one __init method defined
 class __attribute__((sycl_special_class)) class6 { // expected-error {{types with 'sycl_special_class' attribute must have one and only one '__init' method defined}}
   class6() {}
@@ -47,6 +55,14 @@ class [[clang::sycl_special_class]] class8 { // expected-error {{types with 'syc
 struct __attribute__((sycl_special_class)) struct3;
 struct struct3 {}; // expected-error {{types with 'sycl_special_class' attribute must have one and only one '__init' method defined}}
 
+// expected-error@+1{{'sycl_special_class' attribute must have one and only one '__init' method defined}}
+struct __attribute__((sycl_special_class)) struct7 {
+  struct7();
+  bool operator==(const struct7 &);
+  struct7 &operator()();
+  ~struct7();
+};
+
 // Only classes
 [[clang::sycl_special_class]] int var1 = 0;       // expected-warning {{'sycl_special_class' attribute only applies to classes}}
 __attribute__((sycl_special_class)) int var2 = 0; // expected-warning {{'sycl_special_class' attribute only applies to classes}}