Smooth Operator comments

Results 138 comments of


                                            Smooth Operator

Handle inner procs

Seems to be a compiler bug. ``` /home/adavidoff/git/Nim/compiler/semstmts.nim(1636) maybeAddResult /home/adavidoff/git/Nim/compiler/semstmts.nim(1490) addResult /home/adavidoff/git/Nim/lib/system/fatal.nim(53) sysFatal Error: unhandled exception: 'sym' is not accessible using discriminant 'kind' of type 'TNode' [FieldDefect] ``` Maybe we...

Handle inner procs

Stripping the `result` symbol didn't help, FWIW.

prefix leg names with original proc name

I don't have any great ideas on how to do this _trivially_ without cheating with a global or something, but here's what I came up with: We get more strict...

prefix leg names with original proc name

This is also a #190 problem.

Generic continuations

@zevv isn't going to wait around for generics, I guess.

Forking a continuation requires deepCopy()?

Not yet, and it's the most critical ~defect~ missing enhancement :smirk: as far as I'm concerned.

Forking a continuation requires deepCopy()?

This can be solved coincidentally with #190.

support CPS transformations on nnkLambda and non-top-level procs

```nim let foo = proc() {.cps:C.} = echo "foo" foo() ``` I don't see how we can support this as long as the compiler rejects the following: ```nim let foo...

CPS doesn't rewrite locals with proc types correctly

What made you decide this is a CPS bug?

Figure out a way to convert Continuation to Env only once per continuation

I think Nim has plenty of knobs and levers here and I don't really want to turn the checks off at all -- it just seems unnecessary and out of...