Fix autoconf code to find LLVM tools.
authorGeoffrey Mainland <gmainlan@microsoft.com>
Mon, 25 Feb 2013 08:06:46 +0000 (08:06 +0000)
committerGeoffrey Mainland <gmainlan@microsoft.com>
Mon, 25 Feb 2013 08:06:46 +0000 (08:06 +0000)
The loop exit condition was testing ${LLC} instead of $1, which was
incorrect. While I'm here, quote the path being tested since it may contain
spaces (e.g. on Windows), and don't search paths that don't exist, which
eliminates un-useful error messages from find.

aclocal.m4

index b38418b..11c95ee 100644 (file)
@@ -1967,10 +1967,12 @@ AC_DEFUN([FIND_LLVM_PROG],[
         save_IFS=$IFS
         IFS=":;"
         for p in ${PATH}; do
-            $1=`${FindCmd} ${p} -type f -perm +111 -maxdepth 1 -regex '.*/$3-[[0-9]]\.[[0-9]]' -or -type l -perm +111 -maxdepth 1 -regex '.*/$3-[[0-9]]\.[[0-9]]' | ${SortCmd} -n | tail -1`
-            if test -n "${LLC}"; then
-                break
-            fi
+           if [ -d "${p}" ]; then
+                $1=`${FindCmd} "${p}" -type f -perm +111 -maxdepth 1 -regex '.*/$3-[[0-9]]\.[[0-9]]' -or -type l -perm +111 -maxdepth 1 -regex '.*/$3-[[0-9]]\.[[0-9]]' | ${SortCmd} -n | tail -1`
+                if test -n "$1"; then
+                    break
+                fi
+           fi
         done
         IFS=$save_IFS
     fi