Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM. Other clean up can be done as a separate patch.
test/CodeGenOpenCL/address-spaces.cl | ||
---|---|---|
37 ↗ | (On Diff #133802) | Suggest using the same name across all the OpenCL tests as some are using GIZ, some AMD and some AMDGCN. AMDGCN seems the clearer now that only have a single address space mapping? Similar comment for other places GIZ/AMD is being mentioned. |
test/CodeGenOpenCL/amdgpu-env-amdgiz.cl | ||
4 ↗ | (On Diff #133802) | Should this test be renamed to amdgpu-env-amdgcn.cl now there is a single address space mapping (and delete any relating to the old mapping if they exist)? |