Remove duplicated GCN (Capeverde) entry in device list

Author: baldurk

renderdoc/driver/ihv/amd/amd_isa.cpp

@@ -144,16 +144,16 @@ static bool IsSupported(GraphicsAPI api)
 
 void GetTargets(GraphicsAPI api, std::vector<std::string> &targets)
 {
-  targets.reserve(ARRAY_COUNT(asicInfo) + 1);
+  targets.reserve(asicCount + 1);
 
   if(IsSupported(api))
   {
     // OpenGL doesn't support AMDIL
     if(api != GraphicsAPI::OpenGL)
       targets.push_back("AMDIL");
 
-    for(const asic &a : asicInfo)
-      targets.push_back(a.name);
+    for(int i = 0; i < asicCount; i++)
+      targets.push_back(asicInfo[i].name);
   }
   else
   {
@@ -179,8 +179,9 @@ std::string Disassemble(const SPVModule *spv, const std::string &entry, const st
 
   bool found = false;
 
-  for(const asic &a : asicInfo)
+  for(int i = 0; i < asicCount; i++)
   {
+    const asic &a = asicInfo[i];
     if(target == a.name)
     {
       cmdLine += " -gfxip ";
@@ -357,8 +358,9 @@ std::string Disassemble(ShaderStage stage, const std::vector<std::string> &glsl,
   }
 
   bool found = false;
-  for(const asic &a : asicInfo)
+  for(int i = 0; i < asicCount; i++)
   {
+    const asic &a = asicInfo[i];
     if(target == a.name)
     {
       cmdLine += StringFormat::Fmt("%d;%d;", a.chipFamily, a.chipRevision);

renderdoc/driver/ihv/amd/amd_isa_devices.cpp

@@ -23,14 +23,14 @@
  ******************************************************************************/
 
 #include "amd_isa_devices.h"
+#include "common/common.h"
 #include "official/RGA/Common/asic_reg/devices.h"
 
-GCNISA::asic GCNISA::asicInfo[22] = {
+const GCNISA::asic GCNISA::asicInfo[] = {
     // Southern Islands
     {"GCN (Tahiti)", "6", FAMILY_SI, SI_TAHITI_P_B1},
     {"GCN (Pitcairn)", "6", FAMILY_SI, SI_PITCAIRN_PM_A1},
     {"GCN (Capeverde)", "6", FAMILY_SI, SI_CAPEVERDE_M_A1},
-    {"GCN (Capeverde)", "6", FAMILY_SI, SI_CAPEVERDE_M_A1},
     {"GCN (Oland)", "6", FAMILY_SI, SI_OLAND_M_A0},
     {"GCN (Hainan)", "6", FAMILY_SI, SI_HAINAN_V_A0},
     // Sea Islands
@@ -52,4 +52,6 @@ GCNISA::asic GCNISA::asicInfo[22] = {
     {"GCN (Baffin)", "8", FAMILY_VI, VI_BAFFIN_M_A0},
     {"GCN (gfx804)", "8", FAMILY_VI, VI_LEXA_V_A0},
     // GDT_HW_GENERATION_GFX9 goes here, when it's supported by amdspv.
-};
+};
+
+RDCCOMPILE_ASSERT(ARRAY_COUNT(GCNISA::asicInfo) == GCNISA::asicCount, "Mismatched array count");

renderdoc/driver/ihv/amd/amd_isa_devices.h

@@ -34,6 +34,7 @@ struct asic
   int chipRevision;
 };
 
-extern asic asicInfo[22];
+const int asicCount = 21;
+extern const asic asicInfo[];
 
 };    // namespace GCNISA

renderdoc/driver/ihv/amd/amd_isa_win32.cpp

@@ -91,8 +91,9 @@ std::string GCNISA::Disassemble(const DXBC::DXBCFile *dxbc, const std::string &t
   in.numCompileOptions = 0;
   in.pCompileOptions = opts;
 
-  for(const asic &a : asicInfo)
+  for(int i = 0; i < asicCount; i++)
   {
+    const asic &a = asicInfo[i];
     if(target == a.name)
     {
       in.chipFamily = a.chipFamily;