> 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.
Shaojin Wen has updated the pull request incrementally with two additional commits since the last revision: - code style - implNote before param, from @RogerRiggs ------------- Changes: - all: https://git.openjdk.org/jdk/pull/26633/files - new: https://git.openjdk.org/jdk/pull/26633/files/22ae4c83..2439770d Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=26633&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26633&range=02-03 Stats: 12 lines in 1 file changed: 2 ins; 7 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/26633.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/26633/head:pull/26633 PR: https://git.openjdk.org/jdk/pull/26633
