<i18n dev> RFR: 8319423: Improve Year.isLeap by checking divisibility by 16
Claes Redestad
redestad at openjdk.org
Fri Nov 3 21:01:45 UTC 2023
https://github.com/cassioneri/eaf suggest this code for leap year calculation:
public static boolean isLeap(long year) {
int d = year % 100 != 0 ? 4 : 16;
return (year & (d - 1)) == 0;
}
.. with a claim this would compile down to branchless, easily pipelined code.
This doesn't currently happen with C2. In the meantime I think we can improve the current code in `Year.isLeap` and `IsoChronology.isLeapYear` by leveraging the fact that the `% 100` check is only needed if `(year & 15) != 0`:
public static boolean isLeap(long year) {
return (year & 15) == 0 ? (year & 3) == 0 : (year & 3) == 0 && year % 100 != 0;
}
Mac M1:
Name Cnt Base Error Test Error Unit Change
LeapYearBench.isLeapYear 15 0,743 ± 0,009 0,994 ± 0,005 ops/us 1,34x (p = 0,000*)
LeapYearBench.isLeapYearChrono 15 0,748 ± 0,006 0,991 ± 0,003 ops/us 1,32x (p = 0,000*)
LeapYearBench.isLeapYearNS 15 0,558 ± 0,026 0,552 ± 0,033 ops/us 0,99x (p = 0,602 )
* = significant
Linux x64:
Name Cnt Base Error Test Error Unit Change
LeapYearBench.isLeapYear 15 0.534 ± 0.001 0.765 ± 0.004 ops/us 1.43x (p = 0.000*)
LeapYearBench.isLeapYearChrono 15 0.535 ± 0.000 0.753 ± 0.040 ops/us 1.41x (p = 0.000*)
LeapYearBench.isLeapYearNS 15 0.352 ± 0.000 0.351 ± 0.001 ops/us 1.00x (p = 0.000*)
* = significant
30% higher throughput on M1, 40% on x64. `isLeapYearNS` runs a variant of the code from https://github.com/cassioneri/eaf ported to java - perhaps the JIT can be improved to do whatever clang/gcc does here and achieve an even better speed-up.
Testing: so far only java/time/tck/java/time locally, will run a few tiers before filing an enhancement and opening the PR for review.
-------------
Commit messages:
- Add comment
- Delegate IsoChronology.isLeapYear to Year.isLeap (both specified to behave identically), add microbenchmark testing the variant that optimize well with gcc
- Merge branch 'master' into leapyear
- Leap year optimization inspired by Neri & Schneider
Changes: https://git.openjdk.org/jdk/pull/16491/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=16491&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8319423
Stats: 101 lines in 3 files changed: 99 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/16491.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16491/head:pull/16491
PR: https://git.openjdk.org/jdk/pull/16491
More information about the i18n-dev
mailing list