Skip to content

Commit

Permalink
Merge pull request #532 from stweil/openmp
Browse files Browse the repository at this point in the history
openmp: Fix build with clang++ and compilers without OpenMP support
  • Loading branch information
zdenop authored Dec 7, 2016
2 parents 8cf7539 + 6140be6 commit 7f7cea1
Show file tree
Hide file tree
Showing 7 changed files with 9 additions and 14 deletions.
3 changes: 0 additions & 3 deletions api/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -88,9 +88,7 @@ tesseract_LDADD = libtesseract.la

tesseract_LDFLAGS = $(OPENCL_LDFLAGS)

if OPENMP
tesseract_LDADD += $(OPENMP_CFLAGS)
endif

if T_WIN
tesseract_LDADD += -lws2_32
Expand All @@ -99,4 +97,3 @@ endif
if ADD_RT
tesseract_LDADD += -lrt
endif

1 change: 1 addition & 0 deletions ccmain/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ AM_CPPFLAGS += \
-I$(top_srcdir)/textord -I$(top_srcdir)/opencl

AM_CPPFLAGS += $(OPENCL_CPPFLAGS)
AM_CPPFLAGS += $(OPENMP_CXXFLAGS)

if VISIBILITY
AM_CPPFLAGS += -DTESS_EXPORTS \
Expand Down
6 changes: 4 additions & 2 deletions ccmain/par_control.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@
///////////////////////////////////////////////////////////////////////

#include "tesseractclass.h"
#ifdef OPENMP
#ifdef _OPENMP
#include <omp.h>
#endif // OPENMP
#endif // _OPENMP

namespace tesseract {

Expand Down Expand Up @@ -53,7 +53,9 @@ void Tesseract::PrerecAllWordsPar(const GenericVector<WordData>& words) {
}
// Pre-classify all the blobs.
if (tessedit_parallelize > 1) {
#ifdef _OPENMP
#pragma omp parallel for num_threads(10)
#endif // _OPENMP
for (int b = 0; b < blobs.size(); ++b) {
*blobs[b].choices =
blobs[b].tesseract->classify_blob(blobs[b].blob, "par", White, NULL);
Expand Down
7 changes: 0 additions & 7 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -171,14 +171,7 @@ if test "$enable_embedded" = "yes"; then
fi
# check whether to build OpenMP support
AM_CONDITIONAL([OPENMP], false)
AC_OPENMP
AS_IF([test "x$OPENMP_CFLAGS" != "x"],
[AM_CONDITIONAL([OPENMP], true)
AM_CPPFLAGS="$OPENMP_CXXFLAGS $AM_CPPFLAGS"
AC_DEFINE([OPENMP], [], [Defined when compiled with OpenMP support])]
)
# check whether to build opencl version
AC_MSG_CHECKING([--enable-opencl argument])
Expand Down
2 changes: 1 addition & 1 deletion lstm/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ AM_CPPFLAGS += \
-I$(top_srcdir)/dict -I$(top_srcdir)/lstm
AUTOMAKE_OPTIONS = subdir-objects
SUBDIRS =
AM_CXXFLAGS = -fopenmp
AM_CXXFLAGS = $(OPENMP_CXXFLAGS)

if !NO_TESSDATA_PREFIX
AM_CXXFLAGS += -DTESSDATA_PREFIX=@datadir@/
Expand Down
2 changes: 1 addition & 1 deletion lstm/lstm.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

#include "lstm.h"

#ifndef ANDROID_BUILD
#ifdef _OPENMP
#include <omp.h>
#endif
#include <stdio.h>
Expand Down
2 changes: 2 additions & 0 deletions lstm/parallel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,9 @@

#include "parallel.h"

#ifdef _OPENMP
#include <omp.h>
#endif

#include "functions.h" // For conditional undef of _OPENMP.
#include "networkscratch.h"
Expand Down

0 comments on commit 7f7cea1

Please sign in to comment.