[clang] [OpenMP] Prevent AMDGPU from overriding visibility on DT_nohost variables (PR #68264)
Jon Chesterfield via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 5 07:45:18 PDT 2023
================
@@ -308,12 +308,13 @@ static bool requiresAMDGPUProtectedVisibility(const Decl *D,
if (GV->getVisibility() != llvm::GlobalValue::HiddenVisibility)
return false;
- return D->hasAttr<OpenCLKernelAttr>() ||
- (isa<FunctionDecl>(D) && D->hasAttr<CUDAGlobalAttr>()) ||
- (isa<VarDecl>(D) &&
- (D->hasAttr<CUDADeviceAttr>() || D->hasAttr<CUDAConstantAttr>() ||
- cast<VarDecl>(D)->getType()->isCUDADeviceBuiltinSurfaceType() ||
- cast<VarDecl>(D)->getType()->isCUDADeviceBuiltinTextureType()));
+ return !D->hasAttr<OMPDeclareTargetDeclAttr>() &&
----------------
JonChesterfield wrote:
is this a spurious whitespace change?
https://github.com/llvm/llvm-project/pull/68264
More information about the cfe-commits
mailing list