[commit: ghc] master: Find LLVM tools when version number at end (e.g., llc-3.0) (#7661) (64aaaa1)
Ian Lynagh
ian at well-typed.com
Fri Feb 15 19:03:40 CET 2013
On Wed, Feb 13, 2013 at 07:11:11PM -0800, David Terei wrote:
>
> +AC_DEFUN([FIND_LLVM_PROG],[
> + FP_ARG_WITH_PATH_GNU_PROG_OPTIONAL([$1], [$2], [$3])
> + if test "$1" == ""; then
> + GOOD_PATH=`echo $PATH | tr ':,;' ' '`
Hmm. I don't think we'll ever have a $PATH containing a path like
c:/..., but there could certainly be paths containing spaces.
> + $1=`${FindCmd} ${GOOD_PATH} -type f -perm +111 -maxdepth 1 -regex '.*/$3-[[0-9]]\.[[0-9]]' | ${SortCmd} -n | tail -1`
This feels rather unpleasant to me. Is it likely that someone will have
programs called llc-3.0 etc, but no llc? If not, I'd suggest that we
just detect llc and they can override which llc gets used with a
--with-llc flag if they want to.
In fact, i think we should do that regardless. After all, we don't try
searching for gcc-4.6 etc either.
Thanks
Ian
More information about the ghc-devs
mailing list