diff options
| author | Steffen Mueller <smueller@cpan.org> | 2014-11-30 17:26:38 +0100 |
|---|---|---|
| committer | Steffen Mueller <smueller@cpan.org> | 2014-12-01 08:36:36 +0100 |
| commit | 7667e7f116d93f938701c92dcd64886d64138761 (patch) | |
| tree | 51ec9b6b96dbcbfab5c9ed96f11e23f5cb44d252 /sv.h | |
| parent | a10d07f36dfb98f1bcda5f925e584162916f2c66 (diff) | |
| download | perl-7667e7f116d93f938701c92dcd64886d64138761.tar.gz | |
More branch prediction hints for sv_setsv_flags
Dave's cachegrind benchmark says "small win":
REF THIS
------ ------
Ir 105.35 105.91
Dr 104.45 105.42
Dw 105.42 105.17
COND 104.33 104.58
IND 107.04 106.76
COND_m 98.55 110.11
IND_m 110.09 111.08
Diffstat (limited to 'sv.h')
| -rw-r--r-- | sv.h | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -349,7 +349,7 @@ perform the upgrade if necessary. See C<svtype>. /* Sadly there are some parts of the core that have pointers to already-freed SV heads, and rely on being able to tell that they are now free. So mark them all by using a consistent macro. */ -#define SvIS_FREED(sv) ((sv)->sv_flags == SVTYPEMASK) +#define SvIS_FREED(sv) UNLIKELY(((sv)->sv_flags == SVTYPEMASK)) /* this is defined in this peculiar way to avoid compiler warnings. * See the <20121213131428.GD1842@iabyn.com> thread in p5p */ |