llvm dev - Sep 2004 - [LLVMdev] Type uint64_t required but not found

Hi John,

configure still exits, when checking for uint64_t. I've attached a patch, 
that properly will fix it. Either uint64_t or u_int64_t will succeed:

Index: configure.ac
==================================================================RCS file:
/var/cvs/llvm/llvm/autoconf/configure.ac,v
retrieving revision 1.106
diff -u -r1.106 configure.ac
--- configure.ac	2 Sep 2004 18:44:44 -0000	1.106
+++ configure.ac	2 Sep 2004 20:13:17 -0000
@@ -315,7 +315,7 @@
AC_TYPE_PID_T
AC_TYPE_SIZE_T
AC_CHECK_TYPES([int64_t],,AC_MSG_ERROR([Type int64_t required but not 
found]))
-AC_CHECK_TYPES([uint64_t],,AC_MSG_ERROR([Type uint64_t required but not 
found]))
+AC_CHECK_TYPES([uint64_t])
AC_CHECK_TYPES([u_int64_t])
AC_HEADER_TIME
AC_STRUCT_TM

_________________________________________________________________
F� alle de nye og sjove ikoner med MSN Messenger http://messenger.msn.dk/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diff.txt
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20040902/e1cd35c5/attachment.txt>

Henrik Bach wrote:
} Hi John,
} 
} configure still exits, when checking for uint64_t. I've attached a patch, 
} that properly will fix it. Either uint64_t or u_int64_t will succeed:
} 
} Index: configure.ac
} ==================================================================} RCS file:
/var/cvs/llvm/llvm/autoconf/configure.ac,v
} retrieving revision 1.106
} diff -u -r1.106 configure.ac
} --- configure.ac	2 Sep 2004 18:44:44 -0000	1.106
} +++ configure.ac	2 Sep 2004 20:13:17 -0000
} @@ -315,7 +315,7 @@
} AC_TYPE_PID_T
} AC_TYPE_SIZE_T
} AC_CHECK_TYPES([int64_t],,AC_MSG_ERROR([Type int64_t required but not 
} found]))
} -AC_CHECK_TYPES([uint64_t],,AC_MSG_ERROR([Type uint64_t required but not 
} found]))
} +AC_CHECK_TYPES([uint64_t])
} AC_CHECK_TYPES([u_int64_t])
} AC_HEADER_TIME
} AC_STRUCT_TM
} 
Maybe instead of just assuming one will be there you can do something
like this:


AC_CHECK_TYPES([uint64_t],,
               AC_CHECK_TYPES([u_int64_t],,
                              AC_MSG_ERROR([Type uint64_t or u_int64_t ...])))

I know I just jumped into the middle of this discussion, but at least
this will error out if something drastic happens. :)

-bw

-- 
||  "If wishes and buts were clusters of nuts, we'd all have a bowl of
||   granola!" - Mr. Jellineck

Bill/Henrik,

I've committed Bill's suggestion which is a little safer.

Reid.

Bill Wendling wrote:
> Henrik Bach wrote:
> } Hi John,
> } 
> } configure still exits, when checking for uint64_t. I've attached a
patch,
> } that properly will fix it. Either uint64_t or u_int64_t will succeed:
> } 
> } Index: configure.ac
> } ==================================================================> }
RCS file: /var/cvs/llvm/llvm/autoconf/configure.ac,v
> } retrieving revision 1.106
> } diff -u -r1.106 configure.ac
> } --- configure.ac	2 Sep 2004 18:44:44 -0000	1.106
> } +++ configure.ac	2 Sep 2004 20:13:17 -0000
> } @@ -315,7 +315,7 @@
> } AC_TYPE_PID_T
> } AC_TYPE_SIZE_T
> } AC_CHECK_TYPES([int64_t],,AC_MSG_ERROR([Type int64_t required but not 
> } found]))
> } -AC_CHECK_TYPES([uint64_t],,AC_MSG_ERROR([Type uint64_t required but not 
> } found]))
> } +AC_CHECK_TYPES([uint64_t])
> } AC_CHECK_TYPES([u_int64_t])
> } AC_HEADER_TIME
> } AC_STRUCT_TM
> } 
> Maybe instead of just assuming one will be there you can do something
> like this:
> 
> 
> AC_CHECK_TYPES([uint64_t],,
>                AC_CHECK_TYPES([u_int64_t],,
>                               AC_MSG_ERROR([Type uint64_t or u_int64_t
...])))
> 
> I know I just jumped into the middle of this discussion, but at least
> this will error out if something drastic happens. :)
> 
> -bw
>