[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.


https://reviews.llvm.org/D53153





More information about the cfe-commits mailing list