RFR (S): 8022494: Make compilation IDs sequential

Albert Noll albert.noll at oracle.com
Thu Oct 24 01:21:09 PDT 2013


Hi Aleksey,

thanks for looking at this.

On 24.10.2013 10:15, Aleksey Shipilev wrote:
> On 10/24/2013 12:01 PM, Albert Noll wrote:
>> Here is the updated webrev:
>> http://cr.openjdk.java.net/~anoll/8022494/webrev.03/
> Nice to see the locking gone.
>
> compileBroker.cpp:
>    * Is that considered correct that OSR and normal compilations are
> marked differently when running in debug mode, but not in release? I
> understand the comment before assign_compile_id, so this is more of the
> philosophical question.
Compilation IDs are only different if -XX:CICountOSR is set, which is
defaulted to false.
> sharedRuntime.cpp:
>    * Why do you need "2653   return;" in the method tail?
Thanks for spotting this. I missed it during the cleanup.

Best,
Albert
> Thanks,
> -Aleksey.



More information about the hotspot-compiler-dev mailing list