[PATCH] D42711: AMDGPU: Support target triple OS component cuda

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 31 09:16:22 PST 2018


jlebar added a comment.

In https://reviews.llvm.org/D42711#993457, @b-sumner wrote:

> Changing those selections make sense to me since was are adding support for a completely new and different target.


As one of the CUDA maintainers, this also makes sense to me, although we're going to need tests and (to the extent possible) wrapper functions so that it doesn't break.


https://reviews.llvm.org/D42711





More information about the llvm-commits mailing list