Author: Joseph Huber Date: 2022-09-06T13:56:24-05:00 New Revision: 3a623999f3ff96843f97ee300e0c94b8cbc88a9f
URL: https://github.com/llvm/llvm-project/commit/3a623999f3ff96843f97ee300e0c94b8cbc88a9f DIFF: https://github.com/llvm/llvm-project/commit/3a623999f3ff96843f97ee300e0c94b8cbc88a9f.diff LOG: [OpenMP] Fix logic error when building offloading applications Summary: A previous patch removed support for the `-fopenmp-new-driver` and accidentally used the `isHostOffloading` flag instead of `isDeviceOffloading` which lead to some build errors when compiling for the offloading device. This patch addresses that. Added: Modified: clang/lib/Driver/ToolChains/Clang.cpp Removed: ################################################################################ diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index d39f8715c7a1..d3b5f82cb5c2 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -4760,7 +4760,7 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, if (IsUsingLTO) { // Only AMDGPU supports device-side LTO. - if (IsDeviceOffloadAction && !JA.isHostOffloading(Action::OFK_OpenMP) && + if (IsDeviceOffloadAction && !JA.isDeviceOffloading(Action::OFK_OpenMP) && !Args.hasFlag(options::OPT_offload_new_driver, options::OPT_no_offload_new_driver, false) && !Triple.isAMDGPU()) { _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
