diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index 750f1905a04..de913961139 100644
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,7 @@
+2004-11-09  Tom Tromey  <tromey@redhat.com>
+
+	* include/jni.h: Added FIXME.
+
 2004-11-08  Tom Tromey  <tromey@redhat.com>
 
 	* java/util/GregorianCalendar.java: Removed unused imports.
diff --git a/libjava/include/jni.h b/libjava/include/jni.h
index 7765d3d66de..acffa97fd98 100644
--- a/libjava/include/jni.h
+++ b/libjava/include/jni.h
@@ -44,7 +44,9 @@ typedef struct _Jv_JavaVM JavaVM;
 # ifdef __GNUC__
 
 /* If we're using gcc, we can use a platform-independent scheme to get
-   the right integer types.  */
+   the right integer types.  FIXME: this is not always correct, for
+   instance on the c4x it will be wrong -- it depends on whether
+   QImode is 8 bits.  */
 typedef int    jbyte  __attribute__((__mode__(__QI__)));
 typedef int    jshort __attribute__((__mode__(__HI__)));
 typedef int    jint   __attribute__((__mode__(__SI__)));