[llvm-commits] [llvm] r117755 - in /llvm/trunk: Makefile.config.in autoconf/configure.ac configure tools/CMakeLists.txt tools/Makefile

[Dan Gohman]

On Oct 29, 2010, at 5:54 PM, Tobias Grosser wrote:
> 
> --- llvm/trunk/autoconf/configure.ac (original)
> +++ llvm/trunk/autoconf/configure.ac Fri Oct 29 19:54:26 2010
> @@ -121,6 +121,26 @@
>   fi
> done
> 
> +dnl Disable the build of polly, even if it is checked out into tools/polly.
> +AC_ARG_ENABLE(polly,
> +              AS_HELP_STRING([--enable-polly],
> +                             [Use polly if available (default is YES)]),,
> +                             enableval=default)
> +case "$enableval" in
> +  yes) AC_SUBST(ENABLE_POLLY,[1]) ;;
> +  no)  AC_SUBST(ENABLE_POLLY,[0]) ;;
> +  default) AC_SUBST(ENABLE_POLLY,[1]) ;;
> +  *) AC_MSG_ERROR([Invalid setting for --enable-polly. Use "yes" or "no"]) ;;
> +esac
> +
> +
> +dnl Check if polly is checked out into tools/polly and configure it if
> +dnl available.
> +if (test -d ${srcdir}/tools/polly) && (test $ENABLE_POLLY -eq 1) ; then
> +  AC_SUBST(LLVM_HAS_POLLY,1)
> +  AC_CONFIG_SUBDIRS([tools/polly])
> +fi

Hi Tobias and ether,

Instead of a configure-time check, can polly just use the OPTIONAL_PARALLEL_DIRS
mechanism?  This is what clang uses for what sounds like the same purpose; see
tools/Makefile.

Dan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101030/8ca35aa4/attachment.html>