Skip to content

Commit

Permalink
Revert 59fb337 (-ffast-math)
Browse files Browse the repository at this point in the history 
It breaks intsimdmatrix_test.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
  • Loading branch information
@stweil
stweil committed Jan 14, 2019
1 parent 5d3d251 commit 9adf6e4
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/arch/Makefile.am
View file
Expand Up @@ -22,15 +22,15 @@ noinst_LTLIBRARIES += libtesseract_arch.la
libtesseract_arch_la_CPPFLAGS = $(AM_CPPFLAGS)
if AVX_OPT
libtesseract_arch_la_CPPFLAGS += -DAVX
libtesseract_avx_la_CXXFLAGS = -ffast-math -mavx
libtesseract_avx_la_CXXFLAGS = -mavx
endif
if AVX2_OPT
libtesseract_arch_la_CPPFLAGS += -DAVX2
libtesseract_avx2_la_CXXFLAGS = -ffast-math -mavx2
libtesseract_avx2_la_CXXFLAGS = -mavx2
endif
if SSE41_OPT
libtesseract_arch_la_CPPFLAGS += -DSSE4_1
libtesseract_sse_la_CXXFLAGS = -ffast-math -msse4.1
libtesseract_sse_la_CXXFLAGS = -msse4.1
endif

libtesseract_native_la_CXXFLAGS = -O3 -ffast-math
Expand Down

2 comments on commit 9adf6e4

@amitdo
Copy link
Collaborator

@amitdo amitdo commented on 9adf6e4 Mar 4, 2019 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about
libtesseract_native_la_CXXFLAGS = -O3 -ffast-math
?

@stweil
Copy link
Contributor Author

@stweil stweil commented on 9adf6e4 Mar 4, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not revert that part because its source code does not include explicit SIMD instructions. The native implementation is also never used automatically, but only if requested by the user. If there are problems with the native implementation and -ffast-math, it would have to be removed there, too, of course.

Please sign in to comment.