Skip to content

ext/bcmath: Performance improvement bcsqrt() #18771

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

ext/bcmath: Performance improvement bcsqrt() #18771

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
57b91d6
local_num is not needed so it has been removed.
SakiTakamachi Jun 5, 2025
0b34dfd
Optimized and changed the initialization process to run a little later.
SakiTakamachi Jun 5, 2025
bb71577
create fast path and move old logic to standard path
SakiTakamachi Jun 7, 2025
975fcc4
Optimized standard path calculation
SakiTakamachi Jun 8, 2025
f6c50d8
Efficiently obtain the initial value for the standard path.
SakiTakamachi Jun 9, 2025
dd64d94
Omit low precision digits in calculations
SakiTakamachi Jun 10, 2025
d7d1d80
Added test cases
SakiTakamachi Jun 11, 2025
aef9c17
removed nearzero.c
SakiTakamachi Jun 11, 2025
7555b2f
Removed `bc_add_ex`, `bc_int2num`, and `bc_raise_bc_exponent` as they…
SakiTakamachi Jun 12, 2025
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Added test cases
  • Loading branch information
@SakiTakamachi
SakiTakamachi committed Jun 11, 2025
commit d7d1d803439873cb10fb1584a5d6896a2777b0ca
21 changes: 21 additions & 0 deletions ext/bcmath/tests/bcsqrt.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,20 @@ foreach ($scales as $scale) {
}
}

echo "\n";

$radicants = [
"0.005432987654321",
"0.0005432987654321",
"0.0000000000005432987654321",
"0.00000000000005432987654321",
"0.00000000000098765432109876543210987654321098765432109876543210987654321",
"0.000000000000098765432109876543210987654321098765432109876543210987654321",
];
foreach ($radicants as $radicant) {
echo bcsqrt($radicant, 20), "\n";
}

?>
--EXPECT--
0
Expand All @@ -52,3 +66,10 @@ foreach ($scales as $scale) {
0.3872983346
3.8729833462
1.0000000000

0.07370880309922960619
0.02330877013984435859
0.00000073708803099229
0.00000023308770139844
0.00000099380799005580
0.00000031426968054503