scott.linder added a comment. In D60967#1476226 <https://reviews.llvm.org/D60967#1476226>, @rjmccall wrote:
> I suspect that other OpenCL and CUDA implementations don't care at all about > symbol visibility for device-side code generation, and giving kernel > functions default visibility seems like the right thing to do for the > (relatively few) things at the AST level that are sensitive to that, like > template visibility. Would you mind reaching out to other implementors about > that? > > This patch seems fine to me regardless. Yes, I can certainly identify who would be interested in terms of OpenCL and Cuda and work on moving this up to the AST. If you don't object to this patch then is it reasonable for me to submit it? It will get us the required behavior for AMDGPU while I work on the more general solution. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60967/new/ https://reviews.llvm.org/D60967 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
