search

bcgsc / ntHash

Fast hash function for DNA/RNA sequences
http://bcgsc.github.io/ntHash/
MIT License
96 stars 13 forks source link

Calls to omp timing functions when _OPENMP is not defined #2

Closed rsharris closed 7 years ago

rsharris commented 7 years ago

In nttest,cpp, in the function nthashBF(), omp_get_wtime() is called even if _OPENMP is not defined.

If not having OpenMP is a supported configuration, it seems like these four calls should be redirected to some other function (get_time?) if _OPENMP is not defined.

rsharris commented 7 years ago

Similarly, an attempt to set up the build with ./configure --disable-openmp , suggested by ./configure --help, fails with the message "configure: error: NTHASH must be compiled with a C++ compiler that supports OpenMP threading".

mohamadi commented 7 years ago

@rsharris Thanks Bob for reporting these issues. I'll fix them shortly.

mohamadi commented 7 years ago

@rsharris just fixed the openmp issue.