RFR: 8319423: Improve Year.isLeap by checking divisibility by 16 [v2]
Claes Redestad
redestad at openjdk.org
Mon Nov 6 10:04:12 UTC 2023
On Mon, 6 Nov 2023 02:34:21 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:
> >
>
> I have filed [JDK-8319451](https://bugs.openjdk.org/projects/JDK/issues/JDK-8319451). I would suggest waiting for this bug to be resolved before proceeding with this PR.
Nice analysis!
While I'm sure we need to re-evaluate this enhancement after JDK-8319451 is resolved, I'm not a fan of blocking library enhancements on improvements to the runtime/compiler (as it's impossible to know up front if this is something we can fix in the next couple of days/weeks, or need to staff, plan and evaluate over a longer cycle). I included the Neri-Schneider variant in the microbenchmark here to make it easy to assess if an optimization such as JDK-8319451 would turn things around.
We should file an enhancement to re-visit the Granlund & Montgomery/Hacker's delight division. Improving this should benefit either variant, and might be needed together with JDK-8319451 for the `isLeapYearNS` test to win.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16491#issuecomment-1794464999
More information about the core-libs-dev
mailing list