On Tue, 24 Mar 2026 15:09:14 GMT, Chris Plummer <[email protected]> wrote:

>> Paul Hübner has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - Add tests to the problem list.
>>  - Revert "Constrain hasSA to non-preview configurations."
>>    
>>    This reverts commit 7bdfce0b642447580ad3da2c4d7fb3abd95a9a8c.
>
> test/hotspot/jtreg/ProblemList-enable-preview.txt line 50:
> 
>> 48: serviceability/sa/TestHeapDumpForInvokeDynamic.java                      
>>                         8377387 generic-all
>> 49: serviceability/sa/TestJmapCore.java                                      
>>                         8380779 generic-all
>> 50: serviceability/sa/TestJmapCoreMetaspace.java                             
>>                         8380779 generic-all
> 
> The following tests should use 8379925:
> 
> serviceability/sa/TestJmapCore.java
> serviceability/sa/TestJmapCoreMetaspace.java
> serviceability/sa/ClhsdbDumpheap.java
> 
> The other 4 are all separate issues and they don't require 
> -XX:-UseCompressedOops to reproduce. I was actually getting ready to do a 
> problem list update myself (already have the diffs in place) but was just 
> holding off until I could get bugs filed. If you want to file them all under 
> 8379925 for now that is ok. I might change them once I file new CRs, but you 
> should update the CR to not require -XX:-UseCompressedOops.

Actually the 8380769 CR is correct for TestHeapDumpForLargeArray.java. I 
misread it as 8380779

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2236#discussion_r2983062123

Reply via email to