[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