[PATCH] D53153: [OpenCL] Mark kernel functions with default visibility
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 5 17:28:15 PST 2018
rjmccall added a comment.
In https://reviews.llvm.org/D53153#1288112, @rjmccall wrote:
> But do you want to support *dynamically* linking object files? Because that's what visibility is about.
To be specific, if you don't have multiple levels of linking — doing a slower and relatively more expensive link to form a self-contained unit of code distribution, then doing a faster link to form a runnable program from multiple independently-distributed such units — then visibility isn't really doing anything for you.
More information about the cfe-commits