yaxunl added a comment. In https://reviews.llvm.org/D43340#1008980, @t-tye wrote: > LGTM Plus I think there was a test with giz in its name. Should that be renamed? That's test/CodeGenOpenCL/amdgpu-env-amdgiz.cl. I already renamed it in my last commit. https://reviews.llvm.org/D43340