Re: C2 inlining failed because the String constructor is too large

2025-05-18 Thread Chen Liang
diff (when Hide Whitespace is chosen), I see the change does bring much more consistency. Chen From: core-libs-dev on behalf of wenshao Sent: Sunday, May 18, 2025 10:51 AM To: core-libs-dev Subject: C2 inlining failed because the String constructor is too large

Re: C2 inlining failed because the String constructor is too large

2025-05-18 Thread Remi Forax
It's perhaps easier to just add an annotation @ForceInlining. regards, Rémi > From: "wenshao" > To: "core-libs-dev" > Sent: Sunday, May 18, 2025 5:51:15 PM > Subject: C2 inlining failed because the String constructor is too large > Through JVM Opt

C2 inlining failed because the String constructor is too large

2025-05-18 Thread wenshao
Through JVM Option +PrintInlining, we found that String has a constructor codeSize of 852, which is too large. This caused failed to inline. The following is the output information of PrintInlining: ``` @ 9 java.lang.String:: (12 bytes) inline (hot) !m @ 1 java.nio.charset.Charset::defaultCharset