[PATCH] D61274: [Sema][AST] Explicit visibility for OpenCL/CUDA kernels/variables

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 30 15:27:22 PDT 2019


yaxunl added inline comments.


================
Comment at: lib/AST/Decl.cpp:738
+      (isa<FunctionDecl>(D) && D->hasAttr<CUDAGlobalAttr>()) ||
+      (isa<VarDecl>(D) && D->hasAttr<CUDADeviceAttr>())) {
+    Visibility Vis = LV.getVisibility();
----------------
we also need this for `__constant__` variables.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61274/new/

https://reviews.llvm.org/D61274





More information about the cfe-commits mailing list