-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Complex numbers #3892
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.
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
scimind2460
wants to merge
36
commits into
rust-lang:master
Choose a base branch
from
scimind2460:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+162
−0
Open
Complex numbers #3892
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
ba634bb
Add complex numbers rfc
scimind2460 0f4d922
Change file name and pr to reflect current issue
scimind2460 863794c
Incorporate feedback into RFC
scimind2460 60465d0
Fix errors
scimind2460 66b4a8e
Remove impls and replace with stubs
scimind2460 e6a51ea
Fix polars
scimind2460 f63d8e6
Add more future possibilities and fix typos
scimind2460 8033da5
Add associated consts and collapse functions
scimind2460 d86a918
Apply suggestions from code review
scimind2460 4add4e1
Add fixes in type signature
scimind2460 5aceb3e
Apply suggestions from code review
scimind2460 94dece5
Clarified some details regarding `i`
scimind2460 0cf423a
Fix wording in complex numbers documentation
scimind2460 512a3be
Add possible workaround
scimind2460 ca8f9ba
Update rationale for Complex type in FFI context
scimind2460 23f1e67
Update drawbacks section for Complex type proposal
scimind2460 c055e1a
Improve clarity and rationale in complex numbers documentation
scimind2460 e438f35
Apply suggestions from code review
scimind2460 d2c671a
Update RFC on complex numbers support
scimind2460 3911071
Apply suggestions from code review
scimind2460 73f823f
Enhance rationale and examples for complex numbers API
scimind2460 e69a67f
Add suggestion
scimind2460 d1e41cb
Remove `this`
scimind2460 a1774ac
Refactor Complex struct and update methods
scimind2460 6851400
Apply suggestions from code review
scimind2460 d7429f8
Clarify reason for intrinsic calls
scimind2460 be1092e
Apply suggestion from @programmerjake
joshtriplett b28187b
Remove polar from the main body of the RFC
joshtriplett be027bb
Apply suggestions from code review
scimind2460 b376ab9
Update 3892-complex-numbers.md
scimind2460 410b497
Apply suggestions from code review
scimind2460 11a2bfa
Refine complex numbers documentation and remove redundancy
scimind2460 a99270d
Update text/3892-complex-numbers.md
scimind2460 5480bd4
Revise alternatives section for complex numbers
scimind2460 9e3bacd
Enhance discussion on complex number alternatives
scimind2460 d773668
Expand alternatives section for complex numbers
scimind2460 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
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,162 @@ | ||
| - Feature Name: `complex_numbers` | ||
| - Start Date: 2025-12-02 | ||
| - RFC PR: [rust-lang/rfcs#3892](https://github.com/rust-lang/rfcs/pull/3892) | ||
| - Rust Issue: [rust-lang/rust#0000](https://github.com/rust-lang/rust/issues/0000) | ||
|
|
||
| ## Summary | ||
| [summary]: #summary | ||
|
|
||
| FFI-compatible and calling-convention-compatible complex types are to be introduced into `core` to ensure synchronity with C primitives. | ||
|
|
||
| ## Motivation | ||
| [motivation]: #motivation | ||
|
|
||
| The C standard defines the _memory layout_ of a complex number, but not their _calling convention_. | ||
| This means crates like `num-complex` require workarounds to interface with FFI using `_Complex`, and cannot pass values directly. | ||
|
|
||
| In essence, this RFC makes code like this: | ||
| ```C | ||
| extern double _Complex computes_function(double _Complex x); | ||
| ``` | ||
| callable in Rust without indirection: | ||
| ```rust | ||
| extern "C" { | ||
| fn computes_function(x: Complex<f64>) -> Complex<f64>; | ||
| } | ||
| fn main() { | ||
| let returned_value = computes_function(Complex::<f64>::new(3.0, 4.0)) | ||
| } | ||
| ``` | ||
| using the standard library's FFI-compatible complex numbers. | ||
|
|
||
| ## Guide-level explanation | ||
| [guide-level-explanation]: #guide-level-explanation | ||
| `Complex<T>` numbers are in core::num and reexported in std::num, like `use core::num::Complex` or `use std::num::Complex` | ||
| `Complex<T>` numbers can be instantiated with any component type using `Complex::new(re, im)` where `re` and `im` are of the same type ( includes all numbers). | ||
scimind2460 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| ```rust | ||
| let x = Complex::new(3.0, 4.0); | ||
| ``` | ||
|
|
||
| Simple arithmetic is supported: | ||
|
|
||
| ```rust | ||
| let first = Complex::new(1.0, 2.0); | ||
| let second = Complex::new(3.0, 4.0); | ||
| let a = first + second; // 4 + 6i | ||
| let b = first - second; // -2 - 2i | ||
| let c = first * second; // -5 + 10i | ||
| let d = float_second / float_first; // 0.44 - 0.8i | ||
| ``` | ||
|
|
||
| ## Reference-level explanation | ||
| [reference-level-explanation]: #reference-level-explanation | ||
|
|
||
| The `core` crate will provide implementations for operator traits for possible component types. For now, complex operations like `Mul` and `Div` are defined specifically for each floating-point type, while `Add` and `Sub` are defined for any components that themselves implement `Add` and `Sub`. | ||
| Calls to some `libgcc` functions may also be needed, and will be emitted by the backend via compiler-builtins, specifically `__mulsc3`, `__muldc3`, `__divsc3` and `__divdc3` for the proper and complete implementation of these types. This is because if we implemented these operations generically, to implement any overflow checks, we would need to implement a trait that can generically call, say, a `max()` function and compare between values. `Add` and `Sub` do not have this problem (as they are relatively straightforward and do not hold any intermediate values) | ||
| They will have an internal representation similar to this (with public fields for real and imaginary parts): | ||
| ```rust | ||
| // in core::num::complex, which would be a private module holding complex types | ||
| #[lang = "complex"] // for calling convention. | ||
| #[repr(C)] | ||
| #[derive(Copy, Clone, PartialEq, Debug)] | ||
| pub struct Complex<T> {pub re: T, pub im: T}; | ||
| ``` | ||
| have a constructor | ||
| ```rust | ||
| impl Complex<T> { | ||
| fn new(re: T, im: T) -> Self; | ||
| } | ||
| ``` | ||
| and have arithmetic implementations similar to this: | ||
| ```rust | ||
| // `Add` and `Sub` work on individual components so can be used with any `T` | ||
| impl<T: Add> Add for Complex<T> { type Output = Self; /* ... */ } | ||
| impl<T: Sub> Sub for Complex<T> { type Output = Self; /* ... */ } | ||
|
|
||
| // `Mul` and `Div` may not work for all types in a generic way, so they are | ||
| // implemented only on concrete types. | ||
| impl Mul for Complex<f64> { type Output = Self; /* ... */ } | ||
| impl Div for Complex<f64> { type Output = Self; /* ... */ } | ||
| // Also f16, f32, and f128 | ||
| ``` | ||
| ## Drawbacks | ||
| [drawbacks]: #drawbacks | ||
scimind2460 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| The multiple emitted calls to `libgcc.so` (`__mulsc3` and the like) via compiler-builtins may cause a bit of overhead and may not be what the Rust lang team and compiler team want. | ||
|
|
||
| ## Rationale and alternatives | ||
scimind2460 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| [rationale-and-alternatives]: #rationale-and-alternatives | ||
|
|
||
| The rationale for this type is mostly FFI: C libraries that may be linked from Rust code currently cannot provide functions with direct struct implementations of Complex - they must be hidden under at least a layer of indirection. This is because of the undefined calling convention of complex numbers in C. For example: on powerpc64-linux-gnu, [returning double _Complex doesn't do the same thing as returning a struct with a field of type double[2].](https://gcc.godbolt.org/z/hh7zYcnK6) However, it is not always possible to write a C complex-valued function that wraps the first function in a pointer. Thus, FFI becomes a problem if such complex-valued functions are passed by value and not by reference. | ||
scimind2460 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| Additionally, this provides a unified API for complex numbers. Right now, many crates define their own complex types, making interoperability complicated, even though `num-complex` already exports its own type. (`rug::Complex` being an example) | ||
| You could theoretically do something like this: | ||
| ```c | ||
| double _Complex function(double _Complex value); | ||
| void wrapper_function(double _Complex* value, double _Complex* out) { | ||
| *out = function(*value); | ||
| } | ||
| ``` | ||
| for all functions you wish for. But this still needs to happen in C. | ||
|
|
||
| ### Alternatives: | ||
| - Don't do this: There are, obviously, millions of alternatives on crates.io, the foremost being `num-complex`. However, I believe that if we wish to support proper FFI with C, then a standard type that matches calling conventions with C complex numbers is an important feature of the language. Hence, I do not recommend this idea. | ||
| - Use a polar layout: Polar complex numbers are undoubtedly a more optimal solution for multiplying complexes. However, I believe that if we wish to have proper FFI with C, then complex number layout should be chosen in accordance with the layout that is used in the C standard, and that is the orthogonal layout. This is also the layout used by most other languages and other crates on crates.io. Additionally, the polar form suffers from many structual issues: it is not a "natural" form for expressing complex numbers in computers - you cannot express pi exactly, so you cannot use radians for angle units. Additionally, polar complex numbers do not have a unique representation for each number - it has an infinity of zeros with all possible angles. The third problem, and in my opinion the most fatal, is the complexity of addition: | ||
|
|
||
| $\left(r_1\angle\theta_1\right) + \left(r_2\angle\theta_2\right) = \left(\sqrt{r_1^2+r_2^2+2r_1r_2\cos(\theta_1-\theta_2)}\right)\angle(atan2({r_1\sin\theta_1+r_2\sin\theta_2},{r_1\cos\theta_1+r_2\cos\theta_2}))$ | ||
|
|
||
| which offsets any benefits that multiplication may bring. | ||
| - Non-generic primitive types: These are, obviously, the most obvious and practical solution. However, if we implemented lots of such types, then we would not be able to expand for `f16` and `f128` support without repeating the code already implemented. It would be extremely repetitive and tedious to document new types and their behavior, even if we used macros to generate implementations | ||
| - Only in `std::ffi`: Many suggestions have been given that `Complex` remains a type in only `std::ffi`. However, these miss a key point of the RFC: this addition is also about creating a unified interface for complex number support in std itself, and making it an FFI type would go against that. | ||
| ## Prior art | ||
| [prior-art]: #prior-art | ||
|
|
||
| FORTRAN, C, C++, Go, Perl and Python all have complex types implemented in the standard library or as a primitive. This clearly appears to be an important feature many languages have. | ||
scimind2460 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| For example, in Python: | ||
| ```py | ||
| complex_num = 1 + 2j | ||
| complex_second = 3 + 4j | ||
| print(complex_num * complex_second) | ||
| ``` | ||
| or in C: | ||
| ```c | ||
| float _Complex cmplx = 1 + 2*I; | ||
| float _Complex two_cmplx = 3 + 4*I; | ||
| printf("%.1f%+.1fi\n", creal(cmplx * two_cmplx), cimag(cmplx * two_cmplx)); | ||
| ``` | ||
| Even in Rust, it has been discussed two times in IRLO: | ||
| - [First discussion](https://internals.rust-lang.org/t/c-compatible-complex-types-using-traits/13757) | ||
| - [Second discussion](https://internals.rust-lang.org/t/standard-complex-number-in-std-library/23748) | ||
|
|
||
| Many crates, like `num-complex` also provide this feature, though it is not FFI-safe. | ||
| ## Unresolved questions | ||
| [unresolved-questions]: #unresolved-questions | ||
|
|
||
|
|
||
| ## Future possibilities | ||
| [future-possibilities]: #future-possibilities | ||
|
|
||
| - Maybe later on, we can think of adding a special custom suffix for complex numbers (`1+2j` for example), and using that as a simpler way of writing complex numbers if this RFC is accepted? This is very similar to how most languages implement complex numbers? Or perhaps we could consider a constant: | ||
| ```rust | ||
| impl<T: Float> Complex<T: Float> { | ||
| const I: T = Complex::new(T::zero(), T::one()); | ||
| } | ||
| ``` | ||
| where `zero` and `one` is implemented on the `Float` trait similar to `num_traits`? | ||
| Or maybe we could have a method on normal numbers: | ||
| ```rust | ||
| // for example | ||
| impl f32 { | ||
| fn i(self) -> Complex<f32> { | ||
| Complex::new(0, self) | ||
| } | ||
| } | ||
| ``` | ||
| that could help simplify the life of people who otherwise would have to keep writing `Complex::new()`? | ||
| - Should we support Imaginary eventually? This RFC doesn't cover it, but I think we can do this later in another RFC. | ||
| - Eventually we may support Gaussian integers (an extension of the real integers) which have a Euclidean division procedure with remainder. GCC has these, and we could theoretically eventually support these integers alongside GCC FFI. | ||
scimind2460 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| - We can also support f16 and f128 once methods for them are stabilised. | ||
| - We should also think about a `Display` implementation. Should we support something like `1 + 2i` or something else? Should we not make a `Display` impl at all, and just use re() and im() for the implementation? | ||
| - We should also consider adding aliases (like c32 and c64) for floating points once they are established, to allow for a shorthand syntax. | ||
| - Eventually, we should also consider adding polar conversions (e.g, `modulus` and `angle`) | ||
| - And also, we should consider adding complex trig functions (`csin`, `ccos`, etc.) that were deliberately left out of the MVP. | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.