[LLVMdev] RFC: configure.ac

[Bill Wendling]

Hi all,

I got this warning during configuration:

/Volumes/SandBox/Clean/llvmCore.roots/llvmCore~obj/src/llvm/configure:
line 4186: ,: command not found

It looks like this is the culprit:

case $ac_cv_c_bigendian in
  yes)
    ENDIAN=big
,

(note the ',')

This looks like it could possibly fix the problem. I don't have an
autoconf that won't mess things up, though.

Comments?

-bw

Index: autoconf/configure.ac
===================================================================
--- autoconf/configure.ac	(revision 44957)
+++ autoconf/configure.ac	(working copy)
@@ -227,9 +227,9 @@
 AC_SUBST(ARCH,$llvm_cv_target_arch)

 dnl Check for the endianness of the target
-AC_C_BIGENDIAN([AC_SUBST([ENDIAN],[big]),
+AC_C_BIGENDIAN([AC_SUBST(ENDIAN,[big]),
                 AC_DEFINE([MSB_FIRST], [1], [Define if this target is
big endian])],
-               [AC_SUBST([ENDIAN],[little]),
+               [AC_SUBST(ENDIAN,[little]),
                 AC_DEFINE([LSB_FIRST], [1], [Define if this target is
little endian])])

 dnl Check for build platform executable suffix if we're crosscompiling