[mlpack-git] [mlpack/mlpack] LSHSearch Parallelization (#700)

[Yannis Mentekidis]

>  find_package(OpenMP)
>  if (OPENMP_FOUND)
> +  add_definitions(-DHAS_OPENMP)
> +  set(HAS_OPENMP "1")

I am defining it here to use it in `lsh_search_impl.hpp`, where I check if HAS_OPENMP == 1 or HAS_OPENMP == 0. I'm not sure this is the correct usage for `add_definitions()` - I just saw it worked and went with it.

If I remove the num_threads() directive from the pragmas, then I guess this whole thing is skippable. num_threads() was added because I was planning to do the whole hybrid parallelization scheme, but now that's out, I can simplify the code a bit.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/700/files/c4c8ff950be8a06e06084764f188095c650b7a60#r70044692
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160708/441f54ac/attachment.html>