[Openmp-commits] [openmp] 8a77056 - [OpenMP][Libomptarget] Fix conditional in CMake for remote plugin

[Atmn Patel via Openmp-commits]

Author: Atmn Patel
Date: 2021-01-27T21:28:25-05:00
New Revision: 8a77056256d9970387595a5c729d894e3fe07131

URL: https://github.com/llvm/llvm-project/commit/8a77056256d9970387595a5c729d894e3fe07131
DIFF: https://github.com/llvm/llvm-project/commit/8a77056256d9970387595a5c729d894e3fe07131.diff

LOG: [OpenMP][Libomptarget] Fix conditional in CMake for remote plugin

The remote offloading plugin's CMakeLists was trying to build if its
flag was enabled even if it didn't find gRPC/protobuf. The conditional
was wrong, it's fixed by this.

Differential Revision: https://reviews.llvm.org/D95574

Added: 
    

Modified: 
    openmp/libomptarget/plugins/remote/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/openmp/libomptarget/plugins/remote/CMakeLists.txt b/openmp/libomptarget/plugins/remote/CMakeLists.txt
index 1baa1125f44c..989c74642c66 100644
--- a/openmp/libomptarget/plugins/remote/CMakeLists.txt
+++ b/openmp/libomptarget/plugins/remote/CMakeLists.txt
@@ -42,12 +42,13 @@ if (Protobuf_FOUND AND gRPC_FOUND AND PROTOC AND GRPC_CPP_PLUGIN)
   set(GRPC_INCLUDE_DIR
       ${directory}
   )
+
+  set(RPC_INCLUDE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/include/)
+  set(RPC_SRC_DIR ${CMAKE_CURRENT_SOURCE_DIR}/lib/)
+  
+  add_subdirectory(src)
+  add_subdirectory(server)
 else()
   libomptarget_say("Not building remote offloading plugin: required libraries were not found.")
 endif()
 
-set(RPC_INCLUDE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/include/)
-set(RPC_SRC_DIR ${CMAKE_CURRENT_SOURCE_DIR}/lib/)
-
-add_subdirectory(src)
-add_subdirectory(server)