-
-
Notifications
You must be signed in to change notification settings - Fork 942
Closed
Labels
Milestone
Description
For #311, @subbuss wrote the following comment, which describes the work needed for this issue:
The reason -X+C doesn't have this bug is because I did not clean up the JIT to remove the
The bigger issue that this bug exposes is that "$!" is a proxy for the errorInfo value in ThreadContext and so, $! can be modified indirectly by directly calling setErrorInfo. Ideally, in later commits, we should do the following:
- clean up the JIT to get rid of the $! hack as done for the AST interp in 20632af
- audit all uses of setErrorInfo to see if they clobber $! accidentally anywhere else.
- unify the globalVariables.set("$!", foo) and setErrorInfo(foo) paths so that we use one or the other uniformly, if feasible.
Reactions are currently unavailable