Re: Patch: IA-64 port

[Jeff Sturm <jsturm at sigma6 dot com>]

Tom Tromey wrote:
> I'm checking in the appended patch.  It implements a port of libgcj to
> the IA-64.  Hans Boehm wrote most of this; I just added some minor
> cleanups.
[...]

I don't understand the comment below in javaprims.h:

-  // these are things we will store in the constant 
+  // We use __LP64__ and not SIZEOF_VOID_P here because we want
+  // something that will be predefined by the compiler.  FIXME -- this
+  // definition probably shouldn't appear here anyway.
+#ifdef __LP64__
+  // We can safely put a long or a double in here without increasing
+  // the size of _Jv_Word; we take advantage of this in the
interpreter.
+  jlong l;
+  jdouble d;
+#endif
+

What compiler defines __LP64__?  IA64?  Alpha does not, so libgcj is
broken for now.

I realize that you're avoiding autoconf-generated defines in the
installed libgcj headers.  So instead we assume (SIZEOF_VOID_P == 8) iff
__LP64__ is defined, however we cannot rely on this.

I know it has been discussed before, but why not simply include config.h
in the installed headers?

--
Jeff Sturm
jsturm@sigma6.com