[PATCH] D142138: [OpenMP] Clean up AMD handling for `-fopenmp-targets=amdgcn` arch inference

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 20 14:39:23 PST 2023


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

In a follow up we should default to all system archs not the first one. But let's keep them in sync. LG



================
Comment at: clang/lib/Driver/ToolChains/AMDGPUOpenMP.cpp:100
+          Arch = Args.MakeArgString(ArchsOrErr->front());
+        }
+      }
----------------
Swap, shorter case first.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142138



More information about the cfe-commits mailing list