RFR(M): 8054889 - Additional Compiler Diagnostic commands

David Chase david.r.chase at oracle.com
Fri Sep 5 15:40:59 UTC 2014


Nils,

on codeCache.hpp, I see a comment "// Dcmd"
And other places I see Dcmd used in class/method names, but
I have to know or guess that Dcmd is Diagnostic Command,
so maybe the comment should say "// Dcmd (Diagnostic commands)”

Picky, I know, but I’ve spent too much time lost in the sources trying to figure
out what was supposed to be going on.

Are we supposed to have @author tags on our (test) sources?
I thought I heard we were not, but that’s hardly definitive.

David

>>> Hi, 
>>> 
>>> This fix introduces three new diagnostic commands and corresponding tests. The commands are Compiler.queue for printing the contents of the compiler queue, Compiler.codelist for printing all nmethods in the codecache and Compiler.codecache for printing an informational code cache summary. 
>>> 
>>> Description here: 
>>> https://bugs.openjdk.java.net/browse/JDK-8054889 
>>> 
>>> Webrev: 
>>> http://cr.openjdk.java.net/~neliasso/8054889/webrev.02/ 
>>> 
>>> Thanks, 
>>> Nils Eliasson 
>> 
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140905/c71ca238/signature.asc>


More information about the hotspot-compiler-dev mailing list