Morten Ofstad
2004-Oct-25 08:41 UTC
[LLVMdev] Visual C patches - forgot to attach to last mail...
-------------- 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>
Brian Gaeke
2004-Oct-25 19:09 UTC
[LLVMdev] Visual C patches - forgot to attach to last mail...
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.