Rm Perl_sv_pv(byte)? from mathoms

These functions can be converted to call their plain macro equivalents
without changing their behavior.  I verified this by eyeballing their
implementations in mathoms.c and comparing that with what happens with
just the plain macros.

That means embed.fnc can be changed to generate the Perl_ forms
automatically, and the mathoms functions can be removed.

Author: khwilliamson

embed.fnc

@@ -3305,10 +3305,10 @@ Adp	|STRLEN |sv_pos_u2b_flags					\
 				|U32 flags
 Admp	|char * |sv_2pv 	|NN SV *sv				\
 				|NULLOK STRLEN *lp
-CMRbdp	|char * |sv_pv		|NN SV *sv
+Cdmp	|char * |sv_pv		|NN SV *sv
 Admp	|char * |sv_2pvbyte	|NN SV *sv				\
 				|NULLOK STRLEN * const lp
-CMRdp	|char * |sv_pvbyte	|NN SV *sv
+CRdmp	|char * |sv_pvbyte	|NN SV *sv
 Adp	|char * |sv_2pvbyte_flags					\
 				|NN SV *sv				\
 				|NULLOK STRLEN * const lp		\

embed.h

@@ -2319,6 +2319,8 @@
 #   define Perl_sv_force_normal(mTHX,a)         sv_force_normal(a)
 #   define Perl_sv_insert(mTHX,a,b,c,d,e)       sv_insert(a,b,c,d,e)
 #   define Perl_sv_mortalcopy(mTHX,a)           sv_mortalcopy(a)
+#   define Perl_sv_pv(mTHX,a)                   sv_pv(a)
+#   define Perl_sv_pvbyte(mTHX,a)               sv_pvbyte(a)
 #   define Perl_sv_pvn_force(mTHX,a,b)          sv_pvn_force(a,b)
 #   define Perl_sv_pvutf8(mTHX,a)               sv_pvutf8(a)
 #   define Perl_sv_setsv(mTHX,a,b)              sv_setsv(a,b)
@@ -2381,6 +2383,8 @@
 #   define Perl_sv_force_normal                 sv_force_normal
 #   define Perl_sv_insert                       sv_insert
 #   define Perl_sv_mortalcopy                   sv_mortalcopy
+#   define Perl_sv_pv                           sv_pv
+#   define Perl_sv_pvbyte                       sv_pvbyte
 #   define Perl_sv_pvn_force                    sv_pvn_force
 #   define Perl_sv_pvutf8                       sv_pvutf8
 #   define Perl_sv_setsv                        sv_setsv

mathoms.c

@@ -82,43 +82,6 @@ Perl_ref(pTHX_ OP *o, I32 type)
     return doref(o, type, TRUE);
 }
 
-/*
-=for apidoc_section $SV
-=for apidoc sv_pv
-
-Use the C<SvPV_nolen> macro instead
-
-=cut
-*/
-
-/* sv_pv() is now a macro using SvPV_nolen();
- * this function provided for binary compatibility only
- */
-
-char *
-Perl_sv_pv(pTHX_ SV *sv)
-{
-    PERL_ARGS_ASSERT_SV_PV;
-
-    if (SvPOK(sv))
-        return SvPVX(sv);
-
-    return sv_2pv(sv, NULL);
-}
-
-/* sv_pvbyte () is now a macro using Perl_sv_2pv_flags();
- * this function provided for binary compatibility only
- */
-
-char *
-Perl_sv_pvbyte(pTHX_ SV *sv)
-{
-    PERL_ARGS_ASSERT_SV_PVBYTE;
-
-    (void)sv_utf8_downgrade(sv, FALSE);
-    return sv_pv(sv);
-}
-
 #if defined(HUGE_VAL) || (defined(USE_LONG_DOUBLE) && defined(HUGE_VALL))
 /*
  * This hack is to force load of "huge" support from libm.a