On Mon, 2 Nov 2026 23:22:35 GMT, David Holmes <[email protected]> wrote:

>> Leonid Mesnik has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - Support reading intrinsic in AOT compiled code.
>>  - Updated test to take in account additional load instructions for
>>    integer.
>
> test/hotspot/jtreg/serviceability/jvmti/events/VMObjectAlloc/VMObjectAllocClone/VMObjectAllocCloneTest.java
>  line 27:
> 
>> 25:  * @test
>> 26:  * @bug 8396399
>> 27:  * @summary Test verifies how VMObjectAlloc works for clone() method.
> 
> The test doesn't actually ensure the intrinsic is used though.

Agree, I just reproduced the actual failures while fixing. 
Not sure if there is easy way to check that expected optimizations were called. 
So the test just rely on the optimization which are done with -Xcomp C1/C2.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/33684#discussion_r4451460588

Reply via email to