RFR (S): 8022494: Make compilation IDs sequential

[Albert Noll]

Hi all,

please review this small patch.

jbs: https://jbs.oracle.com/bugs/browse/JDK-8022494
webrev: http://cr.openjdk.java.net/~anoll/8022494/webrev.00/ 
<http://cr.openjdk.java.net/%7Eanoll/8022494/webrev.00/>

Problem: Compilation IDs can be non-sequential, if

AdapterHandlerLibrary::create_native_wrapper(methodHandle method, int compile_id) {}

returns prior to issuing the compilation, i.e., that happens if
method->code() is not NULL.

Solution: Move the check if method is already compiled (method->code() 
!= NULL) prior
to incrementing the compilation ID. The current patch does NOT guarantee 
sequential compilation
IDs if AdapterHandlerLibrary::buffer_blob() returns null. Please let me 
know if this is an issue that should
be considered an a revision of the patch.
The patch furthermore contains minor code clean-ups.

Many thanks in advance,
Albert

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20130807/7815c265/attachment.html