[PATCH] D40046: AMDGPU/GCN: Bring processors in sync with AMDGPUUsage

Tony Tye via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 14 15:23:01 PST 2017


t-tye accepted this revision.
t-tye added a comment.
This revision is now accepted and ready to land.

LGTM except Bonaire changes.



================
Comment at: lib/Target/AMDGPU/AMDGPU.td:529
    FeatureLDSBankCount16]>;
 
 def FeatureISAVersion8_0_0 : SubtargetFeatureISAVersion <8,0,0,
----------------
D40051 proposes making Bonaire separate from Kaveri and named gfx704.


================
Comment at: lib/Target/AMDGPU/GCNProcessors.td:87
 >;
 
 //===----------------------------------------------------------------------===//
----------------
D40051 proposes making Bonaire separate from Kaveri and named gfx704.


================
Comment at: lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:120
     return {7, 0, 3};
 
   // VI.
----------------
D40051 proposes making Bonaire separate from Kaveri and named gfx704.


https://reviews.llvm.org/D40046





More information about the llvm-commits mailing list