RFR: 8341587: [premain] Support Soft/Weak Reference in AOT cache [v2]

Chen Liang liach at openjdk.org
Mon Feb 24 22:25:05 UTC 2025


On Sat, 22 Feb 2025 02:11:09 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>>> Is this a new requirement, or did I perform a bad git operation somehow? I didn't have a problem with my last PR
>>> 
>>> <img alt="image" width="482" src="https://private-user-images.githubusercontent.com/54955201/414859591-56ce1654-aad7-4bc5-9b52-126d52f8423c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk5OTM3ODksIm5iZiI6MTczOTk5MzQ4OSwicGF0aCI6Ii81NDk1NTIwMS80MTQ4NTk1OTEtNTZjZTE2NTQtYWFkNy00YmM1LTliNTItMTI2ZDUyZjg0MjNjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE5VDE5MzEyOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTM0OTFiYjhmNTg4MjJkMTg0YWViZTBhZDRjZDVkNjg3ODczOGEzZDllMjI1NGFjM2I1ZTc5OTJjYWI4MThiOTkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.ftvCc3gzEVFqsB1VicQ-jNw-_7w1PKZDef8CUQHEFzc">
>> 
>> I just sync'd my premain with leyden/premain and merged the changes into my branch
>
> I tested on top of the latest premain repo and all Leyden tests passed. @macarte I think this PR is ready to be integrated.

@iklam How soon can this be upstreamed? I think I might be working on MH recently and I hope we can simplify the cache and avoid conflicts down the road by a quick upstream to mainline JDK.

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

PR Comment: https://git.openjdk.org/leyden/pull/41#issuecomment-2679796525


More information about the leyden-dev mailing list