-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Teach rustc to do tail calls #217
Comments
Per Graydon, we need to do some more thinking about calling conventions before implementing this. LLVM requires the fastcc convention to implement tail calls, but it doesn't do quite what we want: graydon: brson: judging from comments in llvm-land, we may be in trouble. we might be able to compensate for what fastcc is doing today, but it's allowed to change its mind tomorrow. |
This is currently half-implemented due to some complications in the way LLVM treats tail calls. Rustc parses 'be' expressions and translates them as call+ret pairs, which is enough to get us to 'working' on simple, not-very-deep cases. Might be enough to bootstrap. There is apparently only one CC (fastcc) which supports guaranteed tail calls, and to adapt to it we need to change our ABI assumptions in several places (it turns into callee-restore when you enable the -tailcallopt flag). So even if we annotate our call+ret pairs with 'tail', as required, we can't actually tell LLVM to start performing that optimization yet. |
Turned out not to need more implementation than is shown here for self-hosting. Punting to next milestone. |
Anyone feel we're actually going to do this anymore? |
Doesn't seem like it's going to happen. |
What is the situation with LLVM and tail calls? If they can be made to reliably work without some invasive stuff like declaring the callee to be tail-called, we could define a limited set of things that can be passed to tail-calls (caller's own arguments, scalars), and error when something else is passed. Such tail calls might not be very useful in practice, though. |
Is it possible that looking at Haskell's GHC LLVM-backend might be helpful? http://hackage.haskell.org/trac/ghc/wiki/Commentary/Compiler/Backends/LLVM |
They only work with a callee ABI that is suboptimal in non-tail-call cases. So in a sense, yes, they require the callee to be declared-to-be-tail-called. We could implement this by, say, analyzing a crate and when we find a function that is tail called, either separately compiling it under the friendly-to-tail-call ABI, or compiling wrappers that switch ABI on entry, or something. Or alternatively we can switch every function everywhere to using the tail-call-friendly ABI. I'm not sure if we're currently doing that. We were for a while but we may have stopped. It's the LLVM "fastcall" ABI, which I don't think we're using anymore. In any case, it's a bit of a mess. |
We have sibling call optimization, now. Are we planning to try to do more? |
This continues to come up in conversation, and we've reserved |
I think Graydon's comment on the mailing list: https://mail.mozilla.org/pipermail/rust-dev/2013-April/003557.html puts a nail in this coffin. Reproduced here: On 10/04/2013 5:43 AM, Artella Coding wrote:
No, and it very likely won't. We have a longstanding bug on this: as well as a wiki page and several mailing list threads: https://github.com/mozilla/rust/wiki/Bikeshed-tailcall The summary of all this is:
I'm sorry to be saying all this, and it is with a heavy heart, but we -Graydon
|
It's maybe worth noting that Haskell commonly needs a static argument transformation for inlining, fusion, etc. http://stackoverflow.com/a/9660027/667457 Rust could support the static argument transformation by saying that functions defined inside another function should be elidgible for tailcall optimizations. Or simply warn if tailcalls between functions nested inside another fuction failed sibling optimization. Not sure the current situation. |
It is sad the tail recursion will not be implemented. Then I have a question: why creating a language syntax that seems like a functional language syntax if there lacks an essential feature of functional? |
I'm news in rust and I'm very sad. I try a tail recursive function and I stack overflow so fast. worst when I compile in release the behavior change. He just don't call my function. A friend said to me that the LLVM optimize to undefined value (LLVM know that is a tail recursion infinite ?!?). fn rec(i: i32) {
rec(i + 1)
}
fn main() {
println!("{}", rec(0));
} I agree with Boiethios, a language with functional features who doesn't implement tail call miss something. I write in C and C++, they don't guarantee tail call, but in fact they handle it very well. It's not gonna stop me to learn rust, now I know that rust doesn't handle it. I will code without so it's not a big problem. But I think that is a very good feature for a modern language. notice that fn rec(i: i32) {
println!("Hello");
rec(i + 1)
} stack overflow too in release mode of cargo |
We have plans to eventually implement guaranteed TCO, if possible. We even reserved a keyword for it, "become". Please check the RFCs repo. On Aug 3, 2016, 19:46 -0400, Antoine PLASKOWSKI [email protected], wrote:
|
The previous version used a bunch of manual string splitting and recursion and bad decisions. Recursion - To my knowledge, Rust doesn't support tail call optimization, so recursion can eventually blow the stack on long strings. rust-lang/rust#217 String Splitting - Given that Rust strings are UTF8, there's a non-zero chance that the algorithm may split on a non-character boundary if the code isn't careful. Development while creating the split was problematic at best and resulted in lots of issues involving the boundary. Bad decisions - I originally wrote the code in a way that bent Rust to my will rather than using what Rust provided.
Just a question: It is hypothetically possible to do call graph analysis and transform tail calls into loops, at least within a single crate, but it is computationally expensive at compile time and quite tricky to code. Was there any discussion of this possibility? That would still be an option now, without regard to the choice of calling convention. |
Add MAP_STACK and MAP_HUGETLB entries to MIPS so that the nix crate can be built
lbstanza's approach is to require annotation of functions to make them eligible for TCO (defn+ instead of defn). In rust, that would largely mitigate the extra compile time overhead to timthelion's suggestion, just so long as you aren't using tail calls literally everywhere lol. |
* update docs * cargo clean deletes previous docs * remove stdsimd from coresimd examples * use stdsimd instead of coresimd in core docs * add stdsimd as a dev-dependency of coresimd
Added support for an llvm 11 feature Fixed the Debug builder API to support the new arguments
* Move expected tests out of run-make tests. * Skip copyright check for `*expected` and `.gitignore` files. * Fix typos. * Readd code to add scripts directory to path. * Factor out duplicated code.
Rustc doesn't know how to do tail calls ('be' rather than 'ret') yet. It shouldn't be too hard to teach it how.
The text was updated successfully, but these errors were encountered: