llvm dev - Oct 2004 - [LLVMdev] Re: Patch for missing rand48 on win32

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diff.txt
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20041025/d290c6e0/attachment.txt>

On Oct 25, 2004, at 3:41 AM, Morten Ofstad wrote:
> Index: lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp
> ==================================================================> RCS
file:
> /var/cvs/llvm/llvm/lib/ExecutionEngine/Interpreter/ 
> ExternalFunctions.cpp,v
> retrieving revision 1.78
> diff -u -r1.78 ExternalFunctions.cpp
> --- lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp	1 Sep 2004  
> 22:55:35 -0000	1.78
> +++ lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp	25 Oct 2004  
> 08:29:28 -0000
> @@ -213,6 +213,8 @@
>    return GV;
>  }
>
> +#ifdef HAVE_RAND48
> +
>  // double drand48()
>  GenericValue lle_X_drand48(FunctionType *M, const  
> vector<GenericValue> &Args) {
>    assert(Args.size() == 0);
> @@ -236,6 +238,16 @@
>    return GenericValue();
>  }
>
> +#endif
> [etc.]
Hi,

There's no HAVE_RAND48 symbol provided by autoconf. You'll have to add  
the appropriate check to autoconf/configure.ac before we can take this  
patch.

Thanks,

-Brian

> Hi,
> 
> There's no HAVE_RAND48 symbol provided by autoconf. You'll have to
add
> the appropriate check to autoconf/configure.ac before we can take this  
> patch.
Sorry I forgot to mention I didn't make any changes to the configure 
script. The problem is that I can't test these things since I'm not on a
Unix platform, and I'm not even using the configure/make build process. 
Chris' suggestion when I first mentioned this problem was to zap these 
rand48 things so I wasn't so worried about it... Can we apply the patch 
and wait for someone (who presumably needs these) to volunteer to fix 
the configure script?

m.