[PATCH] D53153: [OpenCL] Mark kernel functions with default visibility

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 5 16:56:20 PST 2018


arsenm added a comment.

In https://reviews.llvm.org/D53153#1288059, @rjmccall wrote:

> I agree with Richard that I'm not sure what the point of supporting frontend visibility settings in OpenCL is.  If you want the "everything is internal to the image" optimization, presumably you can just infer visibility on everything in a pass over the IR.


Eventually we want to support linking object files, and setting the visibility in the IR won't be aware of any user specified visibility


https://reviews.llvm.org/D53153





More information about the cfe-commits mailing list