yaxunl added inline comments.
================
Comment at: lib/Basic/Targets.cpp:2015
-static const unsigned AMDGPUAddrSpaceMap[] = {
- 1, // opencl_global
- 3, // opencl_local
- 2, // opencl_constant
- 4, // opencl_generic
- 1, // cuda_device
- 2, // cuda_constant
- 3 // cuda_shared
+static LangAS::Map AMDGPUPrivateIsZeroMap = {
+ 1, // opencl_global
----------------
t-tye wrote:
> Is there a reason this is no longer const?
sorry, my omission. Will fix it when commit.
================
Comment at: lib/Basic/Targets.cpp:2357
+
+ AddrSpace AS;
};
----------------
t-tye wrote:
> Could this be const since it is not changed after construction?
Yes. will do.
https://reviews.llvm.org/D31210
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
