[OpenMP] Fix logic error when building offloading applications
authorJoseph Huber <jhuber6@vols.utk.edu>
Tue, 6 Sep 2022 18:56:24 +0000 (13:56 -0500)
committerJoseph Huber <jhuber6@vols.utk.edu>
Tue, 6 Sep 2022 18:56:24 +0000 (13:56 -0500)
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.

clang/lib/Driver/ToolChains/Clang.cpp

index d39f871..d3b5f82 100644 (file)
@@ -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()) {