[PATCH] D60967: Move setTargetAttributes after setGVProperties in SetFunctionAttributes
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 23 14:25:24 PDT 2019
rjmccall added a comment.
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.
CHANGES SINCE LAST ACTION
More information about the cfe-commits