tstellarAMD requested changes to this revision.
This revision now requires changes to proceed.

================
Comment at: lib/Basic/Targets.cpp:1959
@@ -1959,1 +1958,3 @@
+    GK_VOLCANIC_ISLANDS,
+    GK_ARCTIC_ISLANDS
   } GPU;
----------------
We're trying to move to more descriptive GPU family names, so this should be 
GK_GFX8 instead of GK_ARCTIC_ISLANDS.  GK_VOLCANIC_ISLANDS should also be 
changed to GK_GFX8, but that can be done in another patch, and would not be a 
prerequisite for this patch. 


https://reviews.llvm.org/D23746



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to