[PATCH] D56871: [AMDGPU] Require at least protected visibility for certain symbols

[Scott Linder via Phabricator via cfe-commits]

scott.linder created this revision.
scott.linder added reviewers: b-sumner, arsenm, kzhuravl, t-tye, yaxunl.
Herald added subscribers: cfe-commits, tpr, dstuttard, wdng.

This allows the global visibility controls to be restrictive while still populating the dynamic symbol table where required. Depends in-part on a patch to apply global symbol visibility to declarations, although this patch is conservatively correct without that change.


Repository:
  rC Clang

https://reviews.llvm.org/D56871

Files:
  lib/CodeGen/CodeGenModule.cpp
  lib/CodeGen/TargetInfo.cpp
  test/CodeGenOpenCL/visibility.cl

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56871.182361.patch
Type: text/x-patch
Size: 5893 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190117/068ba34c/attachment.bin>