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