Eliminate ENTER/LEAVE from sub calls

Every sub call is wrapped in an ENTER/LEAVE pair, which uses the next
free slot on the scope stack to save and then restore the old value of
PL_savestack_ix. Instead, store the old value in a new field in the
context structure, old_savestack_ix. This is quicker and simpler.

Not that we keep the ENTER/LEAVE for XS sub calls, as they don't push a
context frame, and so have nowhere else to remember PL_savestack_ix.

As a side-effect, this commit fixes a TODO test in t/op/sub.t,
which was related to dying while popping a context, then re-popping that
context. For the second pop, the scopestack has since been overwritten
and so too much was getting popped from the savestack. Since we no longer
use the scopestack, it's no longer an issue.

1 files changed, 0 insertions, 1 deletions

diff --git a/t/op/sub.t b/t/op/sub.t
index eaae3de3af..367f325e56 100644
--- a/t/op/sub.t
+++ b/t/op/sub.t
@@ -326,7 +326,6 @@ pass("RT #126845: stub with prototype, then definition with attribute");
     }
 
     f();
-    local $::TODO = "sub unwinding not safe yet";
     ::is($y, 7, "tie_exception");
 }