> 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 one additional 
commit since the last revision:

  add java doc comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/26633/files
  - new: https://git.openjdk.org/jdk/pull/26633/files/99425bcb..170cf788

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=26633&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26633&range=00-01

  Stats: 70 lines in 1 file changed: 70 ins; 0 del; 0 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

Reply via email to