[PATCH] D60967: Move setTargetAttributes after setGVProperties in SetFunctionAttributes

[John McCall via Phabricator via cfe-commits]

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.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60967/new/

https://reviews.llvm.org/D60967