This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Temporarily change constant address space from 4 to 2
ClosedPublic

Authored by yaxunl on Apr 6 2017, 9:32 AM.

Details

Summary

Our final address space mapping is to let constant address space to be 4 to match nvptx. However for now we will make it 2 to avoid unnecessary work in FE/BE/devlib about intrinsics returning constant pointers.

Diff Detail

Event Timeline

yaxunl created this revision.Apr 6 2017, 9:32 AM
arsenm accepted this revision.Apr 6 2017, 9:42 AM

LGTM

This revision is now accepted and ready to land.Apr 6 2017, 9:42 AM
t-tye accepted this revision.Apr 6 2017, 10:01 AM

LGTM

This revision was automatically updated to reflect the committed changes.