RFR: 8349457: [leyden] Workaround Leyden problems with -XX:-TieredCompilation [v4]
Aleksey Shipilev
shade at openjdk.org
Mon Feb 10 16:49:22 UTC 2025
On Mon, 10 Feb 2025 16:43:03 GMT, Ashutosh Mehra <asmehra at openjdk.org> wrote:
>> I looked at both `.cds` and `.cds.code` files, and they are of similar size with `-XX:-TieredCompilation` whether we have this runtime check or not. I suspect this is because with `-XX:-TieredCompilation` we _do not have_ C1 compilations at all, so there is nothing to precompile. This makes the code change redundant.
>
>> I looked at both .cds and .cds.code files, and they are of similar size with -XX:-TieredCompilation whether we have this runtime check or not. I suspect this is because with -XX:-TieredCompilation we do not have C1 compilations at all, so there is nothing to precompile. This makes the code change redundant.
>
> hmm ... I just tried your original patch with [quarkus-getting-started](https://github.com/openjdk/leyden/tree/premain/test/hotspot/jtreg/premain/quarkus-getting-started) test and see the size of `.cds.code` goes down from `4816316` to `1459860` and `.cds` came down from `38363136` to `36966400`
Do you see the same kind of improvement with new version of the patch and `-XX:-TieredCompilation`?
-------------
PR Review Comment: https://git.openjdk.org/leyden/pull/32#discussion_r1949504813
More information about the leyden-dev
mailing list