Commit c3fe73e
Extend
#55886 accidentally created a new function
`Base.MathConstants.rationalize` instead of extending
`Base.rationalize`, which is the reason why `Base.rationalize(Int, π)`
isn’t constant-folded in Julia 1.10 and 1.11:
```
julia> @Btime rationalize(Int,π);
1.837 ns (0 allocations: 0 bytes) # v1.9: constant-folded
88.416 μs (412 allocations: 15.00 KiB) # v1.10: not constant-folded
```
This PR fixes that. It should probably be backported to 1.10 and 1.11.Base.rationalize instead of defining new function (#56793)1 parent b208ee9 commit c3fe73e
1 file changed
+1
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
32 | | - | |
| 32 | + | |
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| |||
0 commit comments