RFR (S): add CodeComments functionality to assember stubs
Vladimir Kozlov
vladimir.kozlov at oracle.com
Thu Sep 20 14:54:09 PDT 2012
I think it is good and I will push it if nobody objects.
Thanks,
Vladimir
Lindenmaier, Goetz wrote:
> 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 <file:///\\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
>
>
>
More information about the hotspot-compiler-dev
mailing list