(original) (raw)
Hi,
The Assembler and CodeBuffer classes supply CodeComment / block\_comment()
functionality, which does not work with stubs. The comments are
not printed with +PrintStubCode or +PrintInterpreter because the
comments are lost when the code is turned into a Stub, while
they are kept if the code is copied to a CodeBlob.
We fixed this in our SAP JVM, and contributed the change to the
ppc-aix-port some while ago, see
http://hg.openjdk.java.net/ppc-aix-port/jdk7u/hotspot/rev/d65d0876ab43.
I propose to add this fix to the OpenJDK mainline.
A webrev can be found here:
http://cr.openjdk.java.net/\~goetz/webrevs/webrev-comments\_in\_stubs/
Basically the change passes the codeBuffer to the Stub
constructor, and adapts the disassembler to print the
comments.
In the debug build the InterpreterCodelet Stub has a new field
holding the code comments.
I also added some ttyLocks and \\\\ns to beautify the output.
Could somebody please create a bug id for this issue and review the changes?
Thank you and best regards,
Goetz