[PATCH] D40117: [AMDGPU] Update test nullptr.ll to use amdgiz environment

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 22 07:50:38 PST 2017


yaxunl added a comment.

In https://reviews.llvm.org/D40117#930580, @yaxunl wrote:

> In https://reviews.llvm.org/D40117#928951, @arsenm wrote:
>
> > Changing the size of the other address spaces seems like a problem, but I'm not sure what they are for. If we actually care about them, they need to be added to the datalayout
>
>
> It seems the backend uses these address spaces. I will update the datalayout instead.


Actually addrspace 6 to 23 are only used by r600. I will add separate run line for amdgcn and r600.


https://reviews.llvm.org/D40117





More information about the llvm-commits mailing list