This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Re: [Patch 2/2] Make java compiled resources public.
Tom Tromey wrote:
>>>>>> "David" == David Daney <ddaney@avtrex.com> writes:
>>>>>>
>
> David> OK to commit?
>
> There are some whitespace problems with this patch, but otherwise it
> is ok. Thanks.
>
This is an alternative version that I think may be better. From IRC:
(11:47:20 AM) iant_work: daney: I suppose the alternative scheme would
be to use a single string, and also use an encoding for '/'
(11:47:31 AM) iant_work: is there an advantage to breaking it up?
(11:47:39 AM) iant_work: I don't care all that much myself
(11:49:18 AM) daney: I had considered that. Internally to the java
runtime the breaking of the names on '/' signify parts of a path, but as
far as the name is concerned, it probably does not matter.
(11:49:59 AM) iant_work: daney: I don't care all that much; the mangling
will be simpler if you use a single string; but if separate strings is
better for some reason, it's OK with me
(11:50:45 AM) daney: The mangling would be shorter with a single string.
I will change it.
The new version has the form: _ZGr##escaped_name, where ## is the length
of the escaped name. I think the code (in jc1 as well as the
un-mangler) is cleaner and the mangled names are shorter (or the same
length).
This new version also bootstrapped on x86_64 with no failures in libjava.
OK to commit, or do you think the previous version is preferable?
2008-01-22 David Daney <ddaney@avtrex.com>
* class.c (hide) Rename to...
(java_hide_decl) ... this throughout, and make public.
* resource.c (Jr_count): Remove.
(compile_resource_data): Call java_mangle_resource_name to generate
decl name. Make resource decl public and hidden.
* mangle.c (java_mangle_resource_name): New function.
* java-tree.h (java_hide_decl, java_mangle_resource_name): Declare
functions.
Index: class.c
===================================================================
--- class.c (revision 131725)
+++ class.c (working copy)
@@ -742,8 +742,8 @@ build_java_method_type (tree fntype, tre
return fntype;
}
-static void
-hide (tree decl ATTRIBUTE_UNUSED)
+void
+java_hide_decl (tree decl ATTRIBUTE_UNUSED)
{
#ifdef HAVE_GAS_HIDDEN
DECL_VISIBILITY (decl) = VISIBILITY_HIDDEN;
@@ -872,7 +872,7 @@ add_field (tree class, tree name, tree f
/* Hide everything that shouldn't be visible outside a DSO. */
if (flag_indirect_classes
|| (FIELD_PRIVATE (field)))
- hide (field);
+ java_hide_decl (field);
/* Considered external unless we are compiling it into this
object file. */
DECL_EXTERNAL (field) = (is_compiled_class (class) != 2);
@@ -1031,7 +1031,7 @@ build_static_class_ref (tree type)
{
TREE_PUBLIC (decl) = 1;
if (CLASS_PRIVATE (TYPE_NAME (type)))
- hide (decl);
+ java_hide_decl (decl);
}
DECL_IGNORED_P (decl) = 1;
DECL_ARTIFICIAL (decl) = 1;
@@ -1071,7 +1071,7 @@ build_classdollar_field (tree type)
TREE_CONSTANT (decl) = 1;
TREE_READONLY (decl) = 1;
TREE_PUBLIC (decl) = 1;
- hide (decl);
+ java_hide_decl (decl);
DECL_IGNORED_P (decl) = 1;
DECL_ARTIFICIAL (decl) = 1;
MAYBE_CREATE_VAR_LANG_DECL_SPECIFIC (decl);
@@ -1760,7 +1760,7 @@ make_class_data (tree type)
/* The only dispatch table exported from a DSO is the dispatch
table for java.lang.Class. */
if (DECL_NAME (type_decl) != id_class)
- hide (dtable_decl);
+ java_hide_decl (dtable_decl);
if (! flag_indirect_classes)
rest_of_decl_compilation (dtable_decl, 1, 0);
/* Maybe we're compiling Class as the first class. If so, set
@@ -2613,7 +2613,7 @@ layout_class_method (tree this_class, tr
|| (METHOD_PRIVATE (method_decl) && METHOD_STATIC (method_decl)
&& ! METHOD_NATIVE (method_decl)
&& ! special_method_p (method_decl)))
- hide (method_decl);
+ java_hide_decl (method_decl);
/* Considered external unless it is being compiled into this object
file, or it was already flagged as external. */
Index: resource.c
===================================================================
--- resource.c (revision 131725)
+++ resource.c (working copy)
@@ -51,14 +51,10 @@ The Free Software Foundation is independ
/* A list of all the resources files. */
static GTY(()) tree resources = NULL;
-/* Count of all the resources compiled in this invocation. */
-static int Jr_count = 0;
-
void
compile_resource_data (const char *name, const char *buffer, int length)
{
tree rtype, field = NULL_TREE, data_type, rinit, data, decl;
- char buf[60];
data_type = build_prim_array_type (unsigned_byte_type_node,
strlen (name) + length);
@@ -79,11 +75,10 @@ compile_resource_data (const char *name,
TREE_CONSTANT (rinit) = 1;
TREE_INVARIANT (rinit) = 1;
- /* Generate a unique-enough identifier. */
- sprintf (buf, "_Jr%d", ++Jr_count);
-
- decl = build_decl (VAR_DECL, get_identifier (buf), rtype);
+ decl = build_decl (VAR_DECL, java_mangle_resource_name (name), rtype);
TREE_STATIC (decl) = 1;
+ TREE_PUBLIC (decl) = 1;
+ java_hide_decl (decl);
DECL_ARTIFICIAL (decl) = 1;
DECL_IGNORED_P (decl) = 1;
TREE_READONLY (decl) = 1;
Index: mangle.c
===================================================================
--- mangle.c (revision 131725)
+++ mangle.c (working copy)
@@ -796,6 +796,56 @@ compression_table_add (tree type)
TREE_VEC_ELT (compression_table, compression_next++) = type;
}
+/* Mangle an embedded resource file name. "_ZGr" is the prefix, The
+ length and then the name itself of the resource name is appended
+ while escaping '$', '.', and '/' to: "$$", "$_", and "$S". */
+
+tree
+java_mangle_resource_name (const char *name)
+{
+ int len = strlen (name);
+ char *n = (char *) alloca (2 * len + 1);
+ char *d;
+ const unsigned char *w1 = (unsigned char *) name;
+ const unsigned char *w2;
+ const unsigned char *limit = w1 + len;
+
+ d = n;
+
+ init_mangling ();
+ MANGLE_RAW_STRING ("Gr");
+
+ while (w1 < limit)
+ {
+ int ch;
+ w2 = w1;
+ ch = UTF8_GET (w1, limit);
+ gcc_assert (ch > 0);
+ switch (ch)
+ {
+ case '$':
+ *d++ = '$';
+ *d++ = '$';
+ break;
+ case '.':
+ *d++ = '$';
+ *d++ = '_';
+ break;
+ case '/':
+ *d++ = '$';
+ *d++ = 'S';
+ break;
+ default:
+ memcpy (d, w2, w1 - w2);
+ d += w1 - w2;
+ break;
+ }
+ }
+ append_gpp_mangled_name (n, d - n);
+
+ return finish_mangling ();
+}
+
/* Mangling initialization routine. */
static void
Index: java-tree.h
===================================================================
--- java-tree.h (revision 131725)
+++ java-tree.h (working copy)
@@ -1026,6 +1026,7 @@ extern tree parse_signature (struct JCF
extern tree add_field (tree, tree, tree, int);
extern tree add_method (tree, int, tree, tree);
extern tree add_method_1 (tree, int, tree, tree);
+extern void java_hide_decl (tree);
extern tree make_class (void);
extern tree push_class (tree, tree);
extern tree unmangle_classname (const char *name, int name_length);
@@ -1205,6 +1206,7 @@ extern void java_check_methods (tree);
extern void java_mangle_decl (tree);
extern tree java_mangle_class_field (struct obstack *, tree);
extern tree java_mangle_vtable (struct obstack *, tree);
+extern tree java_mangle_resource_name (const char *);
extern void append_gpp_mangled_name (const char *, int);
extern void add_predefined_file (tree);