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:04 PM.

Details

Summary
  • Add gfx704
    • Change bonaire to gfx704
  • Remove gfx804
  • Remove gfx901
  • Remove gfx903

Diff Detail

Repository
rL LLVM

Event Timeline

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

LGTM except Bonaire changes.

lib/Target/AMDGPU/AMDGPU.td
529 ↗(On Diff #122894)

D40051 proposes making Bonaire separate from Kaveri and named gfx704.

lib/Target/AMDGPU/GCNProcessors.td
87 ↗(On Diff #122894)

D40051 proposes making Bonaire separate from Kaveri and named gfx704.

lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
120 ↗(On Diff #122894)

D40051 proposes making Bonaire separate from Kaveri and named gfx704.

This revision is now accepted and ready to land.Nov 14 2017, 3:22 PM
kzhuravl updated this revision to Diff 123378.Nov 17 2017, 10:22 AM
kzhuravl marked 3 inline comments as done.
kzhuravl edited the summary of this revision. (Show Details)

Change bonaire to gfx704

kzhuravl requested review of this revision.Nov 17 2017, 10:23 AM
kzhuravl edited edge metadata.
tstellar accepted this revision.Nov 28 2017, 11:00 AM
This revision is now accepted and ready to land.Nov 28 2017, 11:00 AM
This revision was automatically updated to reflect the committed changes.