This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Remove the non-inline function S_croak_memory_wrap from inline.h.
authorAndy Dougherty <doughera@lafayette.edu>
Wed, 27 Mar 2013 19:54:05 +0000 (15:54 -0400)
committerAndy Dougherty <doughera@lafayette.edu>
Thu, 28 Mar 2013 20:49:37 +0000 (16:49 -0400)
commit43387ee1abcd83c3c7586b7f7aa86e838d239aac
tree89766800af3f0f19490b84e6f9bb305ac8cc578b
parentcbe4d57f99858fe05f2e924ad9125b9b9e50504e
Remove the non-inline function S_croak_memory_wrap from inline.h.

This appears to resolve these three related tickets:

    [perl #116989] S_croak_memory_wrap breaks gcc warning flags detection
    [perl #117319] Can't include perl.h without linking to libperl
    [perl #117331] Time::HiRes::clock_gettime not implemented on Linux (regression?)

This patch changes S_croak_memory_wrap from a static (but not inline)
function into an ordinary exported function Perl_croak_memory_wrap.
This has the advantage of allowing programs (particuarly probes, such
as in cflags.SH and Time::HiRes) to include perl.h without linking
against libperl.  Since it is not a static function defined within each
compilation unit, the optimizer can no longer remove it when it's not
needed or inline it as needed.  This likely negates some of the savings
that motivated the original commit 380f764c1ead36fe3602184804292711.
However, calling the simpler function Perl_croak_memory_wrap() still
does take less set-up than the previous version, so it may still be a
slight win.  Specific cross-platform measurements are welcome.
embed.fnc
embed.h
handy.h
inline.h
proto.h
sv.c
util.c