[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