arsenm added inline comments. ================ Comment at: lib/Basic/Targets.cpp:1959 @@ -1959,1 +1958,3 @@ + GK_VOLCANIC_ISLANDS, + GK_ARCTIC_ISLANDS } GPU; ---------------- tstellarAMD wrote: > 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. For now I think polaris10/11 should just be left as VOLCANIC_ISLANDS
https://reviews.llvm.org/D23746 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
