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

[Samuel Antao via Openmp-commits]

sfantao added a comment.

Hi Jonas sorry for the mess up with the braces. It should be good now.

Thanks!
Samuel


================
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
----------------
Hahnfeld wrote:
> 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!
Oh, right, sorry, my bad.... The new diff fixes it.


http://reviews.llvm.org/D14031