[PATCH] D93181: [NFC][AMDGPU] Reformat AMD GPU targets in cuda.cpp

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 14 10:38:57 PST 2020


tra added inline comments.


================
Comment at: clang/lib/Basic/Cuda.cpp:75
     SM(80),                          // Ampere
-    GFX(600), // tahiti
-    GFX(601), // pitcairn, verde
-    GFX(602), // oland, hainan
-    GFX(700), // kaveri
-    GFX(701), // hawaii
-    GFX(702), // 290,290x,R390,R390x
-    GFX(703), // kabini mullins
-    GFX(704), // bonaire
-    GFX(705),
-    GFX(801), // carrizo
-    GFX(802), // tonga,iceland
-    GFX(803), // fiji,polaris10
-    GFX(805), // tongapro
-    GFX(810), // stoney
-    GFX(900), // vega, instinct
-    GFX(902), GFX(904), GFX(906), GFX(908), GFX(909), GFX(90c),
-    GFX(1010), GFX(1011), GFX(1012), GFX(1030), GFX(1031), GFX(1032), GFX(1033)
+    GFX(600),  // gfx600
+    GFX(601),  // gfx601
----------------
These comments are kind of redundant now as they do not convey anything in addition to the macro itself.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93181/new/

https://reviews.llvm.org/D93181



More information about the cfe-commits mailing list