[SYCL][ESIMD] Fix ESIMD private global detection. (#1909)

DenisBakhvalov · web-flow · commit 616a396db061 · 2020-06-19T14:01:21.000+03:00
Signed-off-by: Denis Bakhvalov &lt;denis.bakhvalov@intel.com&gt;
diff --git a/clang/include/clang/Sema/Sema.h b/clang/include/clang/Sema/Sema.h
@@ -12700,7 +12700,7 @@ class Sema final {
   bool isSYCLEsimdPrivateGlobal(VarDecl *VDecl) {
     return getLangOpts().SYCLIsDevice && getLangOpts().SYCLExplicitSIMD &&
            VDecl->hasGlobalStorage() &&
-           (VDecl->getType().getAddressSpace() != LangAS::opencl_constant);
+           (VDecl->getType().getAddressSpace() == LangAS::opencl_private);
   }
 };
 
diff --git a/clang/test/SemaSYCL/esimd-private-glob.cpp b/clang/test/SemaSYCL/esimd-private-glob.cpp
@@ -0,0 +1,3 @@
+// RUN: %clang_cc1 -fsycl -fsycl-is-device -fsycl-explicit-simd -fsyntax-only -verify %s
+// expected-no-diagnostics
+int x = 0;

Original file line number	Diff line number	Diff line change
`@@ -12700,7 +12700,7 @@ class Sema final {`
`12700`	`12700`	`bool isSYCLEsimdPrivateGlobal(VarDecl *VDecl) {`
`12701`	`12701`	`return getLangOpts().SYCLIsDevice && getLangOpts().SYCLExplicitSIMD &&`
`12702`	`12702`	`VDecl->hasGlobalStorage() &&`
`12703`		`- (VDecl->getType().getAddressSpace() != LangAS::opencl_constant);`
	`12703`	`+ (VDecl->getType().getAddressSpace() == LangAS::opencl_private);`
`12704`	`12704`	`}`
`12705`	`12705`	`};`
`12706`	`12706`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+// RUN: %clang_cc1 -fsycl -fsycl-is-device -fsycl-explicit-simd -fsyntax-only -verify %s`
	`2`	`+// expected-no-diagnostics`
	`3`	`+int x = 0;`