On Tue, 5 Aug 2025 01:23:17 GMT, Shaojin Wen <s...@openjdk.org> wrote:
> By adding the JVM startup parameters `-XX:+UnlockDiagnosticVMOptions > -XX:+PrintInlining` and analyzing the printed log information, and found that > the code size of the j.t.f.DateTimePrintContext::adjust method is 382, which > is greater than 325, causing inlining failure. > > > @ 7 java.time.format.DateTimePrintContext::adjust (382 bytes) failed to > inline: hot method too big > > > By splitting the code into `common/uncommon`, and moving the uncommon code > into adjust0, the adjust method is kept small and can be inlined by the C2 > optimizer. Do you intend to move this PR from Draft to Open? src/java.base/share/classes/java/time/format/DateTimePrintContext.java line 164: > 162: } > 163: > 164: private static TemporalAccessor adjust( Please fold the parameters into fewer (1) lines and briefly document the rationale for the internal method. ------------- PR Comment: https://git.openjdk.org/jdk/pull/26633#issuecomment-3168340810 PR Review Comment: https://git.openjdk.org/jdk/pull/26633#discussion_r2263021841
