# HG changeset patch # User Thomas Wuerthinger # Date 1361243332 28800 # Node ID 94ea9a864fc68ace69f672b57c7169ddfce1194f # Parent 5aa05e6f9dee3ada18b7aa392fd78186d419acaf Remove usages of VmIds.toString. diff -r 5aa05e6f9dee -r 94ea9a864fc6 src/share/vm/graal/graalCompiler.cpp --- a/src/share/vm/graal/graalCompiler.cpp Mon Feb 18 19:02:14 2013 -0800 +++ b/src/share/vm/graal/graalCompiler.cpp Mon Feb 18 19:08:52 2013 -0800 @@ -188,7 +188,7 @@ } Handle GraalCompiler::get_JavaType(Symbol* klass_name, TRAPS) { - return VMToCompiler::createUnresolvedJavaType(VmIds::toString(klass_name, THREAD), THREAD); + return VMToCompiler::createUnresolvedJavaType(java_lang_String::create_from_symbol(klass_name, THREAD), THREAD); } Handle GraalCompiler::get_JavaTypeFromSignature(Symbol* signature, KlassHandle loading_klass, TRAPS) { @@ -255,12 +255,12 @@ } Handle GraalCompiler::get_JavaType(KlassHandle klass, TRAPS) { - Handle name = VmIds::toString(klass->name(), THREAD); + Handle name = java_lang_String::create_from_symbol(klass->name(), THREAD); return createHotSpotResolvedObjectType(klass, name, CHECK_NULL); } Handle GraalCompiler::get_JavaField(int offset, int flags, Symbol* field_name, Handle field_holder, Handle field_type, TRAPS) { - Handle name = VmIds::toString(field_name, CHECK_NULL); + Handle name = java_lang_String::create_from_symbol(field_name, CHECK_NULL); return VMToCompiler::createJavaField(field_holder, name, field_type, offset, flags, false, CHECK_NULL); } diff -r 5aa05e6f9dee -r 94ea9a864fc6 src/share/vm/graal/graalCompilerToVM.cpp --- a/src/share/vm/graal/graalCompilerToVM.cpp Mon Feb 18 19:02:14 2013 -0800 +++ b/src/share/vm/graal/graalCompilerToVM.cpp Mon Feb 18 19:08:52 2013 -0800 @@ -120,7 +120,7 @@ C2V_VMENTRY(jstring, getSignature, (JNIEnv *env, jobject, jlong metaspace_method)) Method* method = asMethod(metaspace_method); assert(method != NULL && method->signature() != NULL, "signature required"); - return VmIds::toString(method->signature(), THREAD); + return (jstring)JNIHandles::make_local(java_lang_String::create_from_symbol(method->signature(), THREAD)()); C2V_END C2V_VMENTRY(jobjectArray, initializeExceptionHandlers, (JNIEnv *, jobject, jlong metaspace_method, jobjectArray java_handlers)) @@ -271,7 +271,7 @@ C2V_VMENTRY(void, initializeMethod,(JNIEnv *, jobject, jlong metaspace_method, jobject hotspot_method)) methodHandle method = asMethod(metaspace_method); - Handle name = VmIds::toString(method->name(), CHECK); + Handle name = java_lang_String::create_from_symbol(method->name(), CHECK); InstanceKlass::cast(HotSpotResolvedJavaMethod::klass())->initialize(CHECK); HotSpotResolvedJavaMethod::set_name(hotspot_method, name()); HotSpotResolvedJavaMethod::set_codeSize(hotspot_method, method->code_size()); @@ -432,8 +432,8 @@ return JNIHandles::make_local(THREAD, VMToCompiler::createResolvedJavaMethod(holder, method(), THREAD)); } else { // Get the method's name and signature. - Handle name = VmIds::toString(cp->name_ref_at(index), CHECK_NULL); - Handle signature = VmIds::toString(cp->signature_ref_at(index), CHECK_NULL); + Handle name = java_lang_String::create_from_symbol(cp->name_ref_at(index), CHECK_NULL); + Handle signature = java_lang_String::create_from_symbol(cp->signature_ref_at(index), CHECK_NULL); int holder_index = cp->klass_ref_index_at(index); Handle type = GraalCompiler::get_JavaType(cp, holder_index, cp->pool_holder(), CHECK_NULL); return JNIHandles::make_local(THREAD, VMToCompiler::createUnresolvedJavaMethod(name, signature, type, THREAD)); @@ -551,7 +551,7 @@ Handle type = GraalCompiler::get_JavaTypeFromSignature(fs.signature(), k, Thread::current()); int flags = fs.access_flags().as_int(); bool internal = fs.access_flags().is_internal(); - Handle name = VmIds::toString(fs.name(), Thread::current()); + Handle name = java_lang_String::create_from_symbol(fs.name(), Thread::current()); Handle field = VMToCompiler::createJavaField(JNIHandles::resolve(klass), name, type, fs.offset(), flags, internal, Thread::current()); fields.append(field()); } diff -r 5aa05e6f9dee -r 94ea9a864fc6 src/share/vm/graal/graalVmIds.hpp --- a/src/share/vm/graal/graalVmIds.hpp Mon Feb 18 19:02:14 2013 -0800 +++ b/src/share/vm/graal/graalVmIds.hpp Mon Feb 18 19:08:52 2013 -0800 @@ -39,9 +39,6 @@ // Returns the stub address with the given vmId taken from a java.lang.Long static address getStub(oop id); - // Helper function to convert a symbol to a java.lang.String object - template static T toString(Symbol* symbol, TRAPS); - // Helper function to convert a java.lang.String object to a symbol (this will return NULL if the symbol doesn't exist in the system) static Symbol* toSymbol(jstring string); @@ -53,22 +50,6 @@ return (address)(getBoxedLong(obj)); } -template <> inline Handle VmIds::toString(Symbol* symbol, TRAPS) { - return java_lang_String::create_from_symbol(symbol, THREAD); -} - -template <> inline oop VmIds::toString(Symbol* symbol, TRAPS) { - return toString(symbol, THREAD)(); -} - -template <> inline jstring VmIds::toString(Symbol* symbol, TRAPS) { - return (jstring)JNIHandles::make_local(toString(symbol, THREAD)); -} - -template <> inline jobject VmIds::toString(Symbol* symbol, TRAPS) { - return JNIHandles::make_local(toString(symbol, THREAD)); -} - inline Symbol* VmIds::toSymbol(jstring string) { return java_lang_String::as_symbol(JNIHandles::resolve(string), Thread::current()); }