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

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 20 11:44:04 PST 2017


yaxunl added a comment.

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.


https://reviews.llvm.org/D40117





More information about the llvm-commits mailing list