On Fri, 6 Mar 2026 08:30:55 GMT, Marc Chevalier <[email protected]> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Apply suggestion from @stefank
>>   
>>   Co-authored-by: Stefan Karlsson <[email protected]>
>
> test/hotspot/jtreg/runtime/cds/appcds/cacheObject/ArchivedIntegerCacheTest.java
>  line 28:
> 
>> 26:  * @test
>> 27:  * @summary Test primitive box caches integrity in various scenarios 
>> (IntegerCache etc)
>> 28:  * @requires !java.enablePreview
> 
> When I run this test (locally or not), before or after merging fresh lworld 
> in this branch, I consistently get
> 
> Parse Exception: Syntax error in @requires expression: invalid name: 
> java.enablePreview
> 
> 
> I see this syntax exists in other files, which I don't understand, but I can 
> see it causes problems here. Is there something I'm missing?

Curious, VMProps.java unconditionally sets up this property. How are you 
running the test? I'll see if I can reproduce this issue.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2157#discussion_r2894730640

Reply via email to