Change check for protected device_global members

Author: schittir

clang/lib/Sema/SemaTemplateInstantiateDecl.cpp

@@ -1621,7 +1621,7 @@ Decl *TemplateDeclInstantiator::VisitVarDecl(VarDecl *D,
         SemaRef.Diag(D->getLocation(),
                      diag::err_sycl_device_global_incorrect_scope);
 
-      if (Var->getAccess() != AS_public)
+      if (Var->getAccess() == AS_private || Var->getAccess() == AS_protected)
         SemaRef.Diag(D->getLocation(),
                      diag::err_sycl_device_global_not_publicly_accessible)
             << Var;

clang/test/SemaSYCL/device_global.cpp

@@ -69,10 +69,8 @@ struct ABar {
 };
 
 template <typename T> void fooBar() {
-  // expected-error@+2{{'device_global' member variable 'c' should be publicly accessible from namespace scope}}
   // expected-error@+1{{'device_global' variable must be a static data member or declared in global or namespace scope}}
   static device_global<T> c;
-  // expected-error@+2{{'device_global' member variable 'd' should be publicly accessible from namespace scope}}
   // expected-error@+1{{'device_global' variable must be a static data member or declared in global or namespace scope}}
   device_global<T> d;
 }