optimize SV creation funcs in sv.c
authorDaniel Dragan <bulk88@hotmail.com>
Sat, 4 Oct 2014 06:14:02 +0000 (02:14 -0400)
committerFather Chrysostomos <sprout@cpan.org>
Sat, 4 Oct 2014 16:38:43 +0000 (09:38 -0700)
commitff121dc6349d6a8a9f9fa598e3955c1d701e3f60
tree7fdf5a89aae217be60f81d40f1c37cb71dfbd929
parent2ff4512f4d92cb81d1db548ba0655eb9bfb081e3
optimize SV creation funcs in sv.c

In Perl_newSV, the sv_upgrade is redundant, except to protect against a
segv in blindly SV body derefing SvGROW (but not in sv_grow). sv_grow has
always upgraded a non-PV SV to PV. So don't it here. Since a new SV will
never have be a COW, have a SvLEN or a body, all of which SvGROW uses,
just call sv_grow. Less branching, and smaller code that way.

In Perl_newSV_type, give a hint to compiler that if a platform's
symbol visibility allows inlining, and newSV_type's arg is the base type
(currently SVt_NULL, maybe SVt_IV in future (see ML)), to possibly inline
new_SV into the caller and remove the sv_upgrade call. Also don't call
sv_upgrade if it isn't needed (SVt_NULL) in the public symbol version.

The redundant sv_upgrade then sv_grow goes to commit 79072805bf
"perl 5.0 alpha 2". VC 2003 -01 32 bit threaded machine code size in bytes
of 2 functions
Perl_newSV_type before 0x2f after 0x29
Perl_newSV  before 0x48 after 0x28
sv.c
sv.h