[Openmp-commits] [PATCH] D14031: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget.

[Jonas Hahnfeld via Openmp-commits]

Hahnfeld added inline comments.

================
Comment at: libomptarget/cmake/Modules/LibomptargetGetDependencies.cmake:63-78
@@ +62,18 @@
+################################################################################
+# Looking for libffi...
+################################################################################
+find_package(PkgConfig)
+
+pkg_check_modules(LIBOMPTARGET_SEARCH_LIBFFI QUIET libffi)
+
+find_path (
+  LIBOMPTARGET_DEP_LIBFFI_INCLUDE_DIR
+  NAMES
+    ffi.h
+  HINTS
+    $(LIBOMPTARGET_SEARCH_LIBFFI_INCLUDEDIR)
+    $(LIBOMPTARGET_SEARCH_LIBFFI_INCLUDE_DIRS)
+  PATHS
+    /usr/include
+    /usr/local/include
+    /opt/local/include
----------------
After looking at this quite some time and wondering why it didn't work: It's CMake, they use curly braces. With this small change everything builds correctly out-of-the-box, thanks!


http://reviews.llvm.org/D14031