RFR(XS) : 8038240 : new WB API to get nmethod
john.r.rose at oracle.com
Thu Mar 27 00:54:48 UTC 2014
On Mar 25, 2014, at 3:03 PM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> Thanks for review, I support you suggestion, but as you said we are consistent w/ what we have, also we need this in 8u20, so we don't have enough time.
> I will file an RFE to change WB API and add version to the NMethod in accordance w/ your advice.
> Do you agree w/ such approach? If yes, can I consider you as a Reviewer?
Yes, that is a good approach. But, no, sorry: I didn't review your C code well enough to qualify as a Reviewer; I just looked at the API and traced the effect of the boolean. It looks like you are good to go with Vladimir's review and Morris's and my input.
More information about the hotspot-dev