diff options
| author | Nicholas Clark <nick@ccl4.org> | 2014-03-01 17:40:05 +0100 |
|---|---|---|
| committer | Nicholas Clark <nick@ccl4.org> | 2014-03-01 17:40:05 +0100 |
| commit | 25983af42cdcf2dc1fea6717dac7aac441b6301d (patch) | |
| tree | 0a6de9811bcfe84d9a22b8a1cdf53f2fdba55834 /intrpvar.h | |
| parent | c79d007613fa174f6f5e1588ca5374f505fc44af (diff) | |
| download | perl-25983af42cdcf2dc1fea6717dac7aac441b6301d.tar.gz | |
pp_tms should use a local struct tms, instead of PL_timesbuf.
PL_timesbuf is effectively a vestige of Perl 1, and doesn't actually need to
be an interpreter variable. It will be removed early in v5.21.x, but it's a
good idea to refactor the code not to use it before then. A local struct tms
will be on the C stack, which will be in the CPU's L1 cache, whereas the
relevant part of the interpreter struct may well not be in the CPU cache at
all. Therefore this change might reduce cache pressure fractionally. A local
variable access should also be simpler machine code on most CPU architectures.
Diffstat (limited to 'intrpvar.h')
| -rw-r--r-- | intrpvar.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/intrpvar.h b/intrpvar.h index 2c1b73ee49..5d27ad5fb4 100644 --- a/intrpvar.h +++ b/intrpvar.h @@ -174,6 +174,7 @@ PERLVAR(I, statgv, GV *) PERLVARI(I, statname, SV *, NULL) #ifdef HAS_TIMES +/* Will be removed soon after v5.21.0. See RT #121351 */ PERLVAR(I, timesbuf, struct tms) #endif |