[PATCH] D31210: [AMDGPU] Add new address space mapping

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 22 12:26:40 PDT 2017


yaxunl added a comment.

In https://reviews.llvm.org/D31210#707832, @rampitec wrote:

> I also do not exactly like names "old" and "new". This implies we are going to switch to "new" permanently and doing transition. That is not clear yet, however.


How about we call the old address space mapping "Alloca Is Zero" (AIZ) address space mapping, whereas the new address space mapping "Generic Is Zero" (GIZ) address space mapping?

I can change the variable names. Would you suggest to change the target triple environment name too? e.g. amdnas => amdgiz, amdnascl => amdgizcl?


https://reviews.llvm.org/D31210





More information about the cfe-commits mailing list