[PATCH] D31771: [AMDGPU] Temporarily change constant address space from 4 to 2 for the new address space mapping
Tony Tye via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 6 09:56:43 PDT 2017
t-tye accepted this revision.
t-tye added a comment.
This revision is now accepted and ready to land.
Other than one comment:
LGTM
================
Comment at: lib/Basic/Targets.cpp:2083
Local = 3;
- Constant = 4;
+ Constant = 2;
Private = 5;
----------------
Since Constant is now the same regardless of the GIZ setting, should it be moved to be a literal constant like the other values that do not change?
https://reviews.llvm.org/D31771
More information about the cfe-commits
mailing list