summaryrefslogtreecommitdiff
path: root/scope.h
diff options
context:
space:
mode:
authorFather Chrysostomos <sprout@cpan.org>2014-09-18 16:10:01 -0700
committerFather Chrysostomos <sprout@cpan.org>2014-09-18 18:20:50 -0700
commitff2a62e0c8bedade55bf86a22861a2fe06bb0a16 (patch)
treec049e3257b21cb049fbbda2a41886fcacd5eec5d /scope.h
parent4a8cd1baba77cafb21681f12b12c120bdd458938 (diff)
downloadperl-ff2a62e0c8bedade55bf86a22861a2fe06bb0a16.tar.gz
Skip no-common-vars optimisation for aliases
The ‘no common vars’ optimisation allows perl to copy the values straight from the rhs to the lhs in a list assignment. In ($a,$b) = ($c,$d), that means $c gets assigned to $a, then $d to $b. If the same variable occurs on both sides of the expression (($a,$b)=($b,$a)), then it is necessary to make temporary copies of the variables on the rhs, before assigning them to the left. If some variables have been aliased to others, then the common vars detection can be fooled: *x = *y; $x = 3; ($x, $z) = (1, $y); That assigns 1 to $x, and then goes to assign $y to $z, but $y is the same as $x, which has just been clobbered. So 1 gets assigned instead of 3. This commit solves this by recording in each typeglob whether the sca- lar is an alias of a scalar from elsewhere. If such a glob is encountered, then the entire expression is ‘tainted’ such that list assignments will assume there might be common vars.
Diffstat (limited to 'scope.h')
-rw-r--r--scope.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/scope.h b/scope.h
index 0dce9d6453..cad02cd881 100644
--- a/scope.h
+++ b/scope.h
@@ -16,12 +16,12 @@
#define SAVEt_CLEARPADRANGE 1
#define SAVEt_CLEARSV 2
#define SAVEt_REGCONTEXT 3
-/*** SPARE 4 ***/
-#define SAVEt_ARG0_MAX 4
+#define SAVEt_ARG0_MAX 3
/* one arg */
+#define SAVEt_GP_ALIASED_SV 4
#define SAVEt_BOOL 5
#define SAVEt_COMPILE_WARNINGS 6
#define SAVEt_COMPPAD 7