From 361fde346a1b8e058f53c19040754cc5cf2a3e65 Mon Sep 17 00:00:00 2001 From: Sven Verdoolaege Date: Wed, 25 Feb 2015 14:40:34 +0100 Subject: [PATCH] remove clang from blacklist for openmp support In ce4c754 (polybench_test.sh.in: do not perform openmp tests when using clang, Thu Feb 5 13:10:48 2015 +0100) clang was blacklisted because there were some issues with some PolyBench benchmarks. However, it turned out that the problem was not caused by clang, but by a problem in the initialization of the input for those benchmarks. In particular, part of the array was not being initialized and while the contents apparently defaulted to zero on most systems, this failed to be the case when adding the -fopenmp flag on the clang command line. Refrain from blacklisting clang. Signed-off-by: Sven Verdoolaege --- m4/ax_check_openmp.m4 | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/m4/ax_check_openmp.m4 b/m4/ax_check_openmp.m4 index 4b2b748..509861e 100644 --- a/m4/ax_check_openmp.m4 +++ b/m4/ax_check_openmp.m4 @@ -1,6 +1,4 @@ # Check if $CC supports openmp. -# While clang (3.5) accepts the -fopenmp flag, it produces bad code -# on some PolyBench benchmarks, so it is blacklisted. AC_DEFUN([AX_CHECK_OPENMP], [ AC_SUBST(HAVE_OPENMP) HAVE_OPENMP=no @@ -9,12 +7,4 @@ AC_DEFUN([AX_CHECK_OPENMP], [ if test $? -eq 0; then HAVE_OPENMP=yes fi - AC_MSG_RESULT($HAVE_OPENMP) - if test $HAVE_OPENMP = yes; then - $CC --version | grep clang > /dev/null - if test $? -eq 0; then - AC_MSG_NOTICE([clang blacklisted]) - HAVE_OPENMP=no - fi - fi ]) -- 2.11.4.GIT