RFR(XS) [8u20]: 8038240 : new WB API to get nmethod
igor.ignatyev at oracle.com
Thu Mar 27 17:25:27 UTC 2014
I'd like to backport this fix into 8u20. The original patch was applied
w/o any changes.
On 03/26/2014 01:45 AM, Vladimir Kozlov wrote:
> It is good.
> On 3/25/14 2:06 AM, Igor Ignatyev wrote:
>> Thanks for review, I've added a nmethod structure w/ code's array and
>> compilation level. Since to use NMethod class directly in WhiteBox
>> class, class should be added to bootclasspath, I decided to return the
>> structure as an object array instead.
>> I've replaced getMethodCompilationLevel in CompilerWhiteBoxTest and
>> planning to remove this method completely in a separate CR.
>> Also I've removed CLEAR_PENDING_EXCEPTION from CHECK_JNI_EXCEPTION macros
>> On 03/25/2014 01:11 AM, Vladimir Kozlov wrote:
>>> Can we pass a structure instead of code's array? We can add an
>>> additional information about nmethod into such structure later if
>>> needed. For now it would have just one pointer to jbyteArray array.
>> On 03/25/2014 02:23 AM, Morris Meyer wrote:
>>> Looks good. (Not a reviewer).
>>> On 3/24/14 10:24 AM, Igor Ignatyev wrote:
>>>> Hi all,
>>>> Please review the patch which introduces a new WhiteBox method --
>>>> getNMethod(). We need this method to write reliable tests for
>>>> JDK-8031321 : 'Support Intel bit manipulation instructions'.
>>>> webrev: http://cr.openjdk.java.net/~iignatyev/8038240/webrev.00/
>>>> jbs: https://bugs.openjdk.java.net/browse/JDK-8038240
>>>> testing: jprt
More information about the hotspot-dev