From 3196bed74979360646cbc8418db5c86c0527ced4 Mon Sep 17 00:00:00 2001 From: tromey Date: Thu, 29 Aug 2002 17:53:28 +0000 Subject: * include/jvm.h (struct _Jv_frame_info): New structure. * gnu/gcj/runtime/natNameFinder.cc: Include StringBuffer.h, java-interp.h. (lookupInterp): New method. (getAddrAsString): Use _Jv_frame_info. (dladdrLookup): Likewise. * gnu/gcj/runtime/NameFinder.java (lookup): Try to look up interpreted frame. (lookupInterp): Declare. * java/lang/natVMThrowable.cc: Include Thread.h, java-interp.h. (fillInStackTrace): Collect information on interpreted frames. Use _Jv_frame_info. * interpret.cc: Include Thread.h. (run): Create and push _Jv_MethodChain object. (_Jv_EndOfInterpreter): New global. * java/lang/Thread.java (interp_frame): New field. * include/java-interp.h (struct _Jv_MethodChain): New structure. Include NameFinder.h. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@56657 138bc75d-0d04-0410-961f-82ee72b054a4 --- libjava/gnu/gcj/runtime/NameFinder.java | 11 ++++++++- libjava/gnu/gcj/runtime/natNameFinder.cc | 40 +++++++++++++++++++++++++++----- 2 files changed, 44 insertions(+), 7 deletions(-) (limited to 'libjava/gnu/gcj') diff --git a/libjava/gnu/gcj/runtime/NameFinder.java b/libjava/gnu/gcj/runtime/NameFinder.java index 60f47ac64fe..f32ebe0f530 100644 --- a/libjava/gnu/gcj/runtime/NameFinder.java +++ b/libjava/gnu/gcj/runtime/NameFinder.java @@ -172,6 +172,12 @@ public class NameFinder */ native private String getAddrAsString(RawData addrs, int n); + /** + * If nth element of stack is an interpreted frame, return the + * element representing the method being interpreted. + */ + native private StackTraceElement lookupInterp(RawData addrs, int n); + /** * Creates the nth StackTraceElement from the given native stacktrace. */ @@ -179,11 +185,14 @@ public class NameFinder { StackTraceElement result; - result = dladdrLookup(addrs, n); + result = lookupInterp(addrs, n); + if (result == null) + result = dladdrLookup(addrs, n); if (result == null) { String name = null; String file = null; + String hex = getAddrAsString(addrs, n); if (addr2line != null) diff --git a/libjava/gnu/gcj/runtime/natNameFinder.cc b/libjava/gnu/gcj/runtime/natNameFinder.cc index 42cc164c324..29488eddda7 100644 --- a/libjava/gnu/gcj/runtime/natNameFinder.cc +++ b/libjava/gnu/gcj/runtime/natNameFinder.cc @@ -1,4 +1,4 @@ -// natNameFinder.cc - native helper methods for NameFiner.java +// natNameFinder.cc - native helper methods for NameFinder.java /* Copyright (C) 2002 Free Software Foundation, Inc @@ -19,6 +19,8 @@ details. */ #include #include #include +#include +#include #include @@ -29,15 +31,15 @@ details. */ java::lang::String* gnu::gcj::runtime::NameFinder::getExecutable (void) { - return JvNewStringLatin1 (_Jv_ThisExecutable ()); + return JvNewStringLatin1 (_Jv_ThisExecutable ()); } java::lang::String* gnu::gcj::runtime::NameFinder::getAddrAsString(RawData* addrs, jint n) { - void **p = (void **) addrs; + _Jv_frame_info *p = (_Jv_frame_info *) addrs; typedef unsigned word_t __attribute ((mode (word))); - word_t w = (word_t) p[n]; + word_t w = (word_t) p[n].addr; int digits = sizeof (void *) * 2; char hex[digits+5]; @@ -61,8 +63,8 @@ gnu::gcj::runtime::NameFinder::dladdrLookup(RawData* addrs, jint n) extern char **_Jv_argv; char name[1024]; char file_name[1024]; - void **stack = (void **) addrs; - void* p = stack[n]; + _Jv_frame_info *stack = (_Jv_frame_info *) addrs; + void* p = stack[n].addr; Dl_info dl_info; if (dladdr (p, &dl_info)) @@ -82,3 +84,29 @@ gnu::gcj::runtime::NameFinder::dladdrLookup(RawData* addrs, jint n) #endif return NULL; } + +java::lang::StackTraceElement * +gnu::gcj::runtime::NameFinder::lookupInterp(RawData* addrs, jint n) +{ +#ifdef INTERPRETER + _Jv_frame_info *stack = (_Jv_frame_info *) addrs; + if (stack[n].interp == NULL) + return NULL; + + _Jv_InterpMethod *meth + = reinterpret_cast<_Jv_InterpMethod *> (stack[n].interp); + // FIXME: demangle. + java::lang::StringBuffer *sb = new java::lang::StringBuffer(); + sb->append(_Jv_NewStringUtf8Const(meth->self->name)); + sb->append(_Jv_NewStringUtf8Const(meth->self->signature)); + // FIXME: source file name and line number can be found from + // bytecode debug information. But currently we don't keep that + // around. + // FIXME: is using the defining class correct here? + return new java::lang::StackTraceElement(NULL, -1, + meth->defining_class->getName(), + sb->toString(), false); +#else // INTERPRETER + return NULL; +#endif // INTERPRETER +} -- cgit v1.2.3