RFR: 8350896: Integer/Long.compress gets wrong type from CompressBitsNode::Value [v2]

Jatin Bhateja jbhateja at openjdk.org
Thu Apr 10 06:59:28 UTC 2025


On Wed, 2 Apr 2025 09:50:42 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review comments resolutions
>
> @jatin-bhateja Thanks for the updates! I have a few more requests :)

Hi @eme64 , I have addressed and responded to your comments, please verify.

> test/hotspot/jtreg/compiler/c2/TestBitCompressValueTransform.java line 2:
> 
>> 1: /*
>> 2:  * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
> 
> Ah, I just noticed the test directory. I think we can put it in a more specific location.

There are operation specific tests under compiler/c2 let keep it this way.

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

PR Comment: https://git.openjdk.org/jdk/pull/23947#issuecomment-2791743206
PR Review Comment: https://git.openjdk.org/jdk/pull/23947#discussion_r2036638391


More information about the hotspot-compiler-dev mailing list