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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
