RFR: 8335358: [premain] Explore alternative ways to trigger the end of training run [v5]
Mat Carter
macarte at openjdk.org
Tue Sep 17 17:38:20 UTC 2024
On Tue, 17 Sep 2024 04:29:01 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> Mat Carter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> removed extra space left over from reverted change
>
> src/hotspot/share/opto/parse1.cpp line 1215:
>
>> 1213: }
>> 1214:
>> 1215: if (CDSConfig::is_dumping_preimage_static_archive_with_triggers() && method()->is_end_training_trigger()) {
>
> Do we come here also if `method()` is being inlined by C2?
It does, I tested inlining manually and proved that the method was inlined and still triggering, I'll see if I can add a test case so that if the inlining code path changes the test failure will direct us to add the appropriate code to the new code path
-------------
PR Review Comment: https://git.openjdk.org/leyden/pull/21#discussion_r1763632869
More information about the leyden-dev
mailing list