This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/GCN: Bring processors in sync with AMDGPUUsage
ClosedPublic

Authored by kzhuravl on Nov 14 2017, 12:03 PM.

Diff Detail

Event Timeline

kzhuravl created this revision.Nov 14 2017, 12:03 PM
t-tye accepted this revision.Nov 14 2017, 3:19 PM

LGTM except for Bonaire.

lib/Basic/Targets/AMDGPU.cpp
271

D40051 proposes making Bonaire separate from Kaveri and named gfx704.

This revision is now accepted and ready to land.Nov 14 2017, 3:19 PM
kzhuravl planned changes to this revision.Nov 17 2017, 10:23 AM
This revision was not accepted when it landed; it landed in state Changes Planned.Feb 8 2018, 11:04 PM
This revision was automatically updated to reflect the committed changes.
kzhuravl marked an inline comment as done.

LGTM except for Bonaire.

Done. Also removed gfx800 as per latest docs.