Remove configure --disable-float4-byval
authorPeter Eisentraut <[email protected]>
Thu, 21 Nov 2019 17:00:07 +0000 (18:00 +0100)
committerPeter Eisentraut <[email protected]>
Thu, 21 Nov 2019 17:29:21 +0000 (18:29 +0100)
This build option was only useful to maintain compatibility for
version-0 functions, but those are no longer supported, so this option
can be removed.

float4 is now always pass-by-value; the pass-by-reference code path is
completely removed.

Discussion: https://www.postgresql.org/message-id/flat/f3e1e576-2749-bbd7-2d57-3f9dcf75255a@2ndquadrant.com

24 files changed:
configure
configure.in
doc/src/sgml/func.sgml
doc/src/sgml/installation.sgml
src/backend/access/index/indexam.c
src/backend/access/transam/xlog.c
src/backend/bootstrap/bootstrap.c
src/backend/catalog/genbki.pl
src/backend/commands/analyze.c
src/backend/utils/fmgr/dfmgr.c
src/backend/utils/fmgr/fmgr.c
src/backend/utils/misc/pg_controldata.c
src/bin/initdb/initdb.c
src/bin/pg_controldata/pg_controldata.c
src/bin/pg_resetwal/pg_resetwal.c
src/include/catalog/catversion.h
src/include/catalog/pg_control.h
src/include/catalog/pg_proc.dat
src/include/catalog/pg_type.dat
src/include/fmgr.h
src/include/pg_config.h.in
src/include/postgres.h
src/tools/msvc/Solution.pm
src/tools/msvc/config_default.pl

index 629fe8ecf03215091817a63ebc04d65871ee2a45..b06a95dabcae7ed3f45bef993a7773084d0ab000 100755 (executable)
--- a/configure
+++ b/configure
@@ -866,7 +866,6 @@ with_system_tzdata
 with_zlib
 with_gnu_ld
 enable_largefile
-enable_float4_byval
 enable_float8_byval
 '
       ac_precious_vars='build_alias
@@ -1525,7 +1524,6 @@ Optional Features:
   --enable-cassert        enable assertion checks (for debugging)
   --disable-thread-safety disable thread-safety in client libraries
   --disable-largefile     omit support for large files
-  --disable-float4-byval  disable float4 passed by value
   --disable-float8-byval  disable float8 passed by value
 
 Optional Packages:
@@ -16747,46 +16745,6 @@ _ACEOF
 
 
 
-# Decide whether float4 is passed by value: user-selectable, enabled by default
-{ $as_echo "$as_me:${as_lineno-$LINENO}: checking whether to build with float4 passed by value" >&5
-$as_echo_n "checking whether to build with float4 passed by value... " >&6; }
-
-
-# Check whether --enable-float4-byval was given.
-if test "${enable_float4_byval+set}" = set; then :
-  enableval=$enable_float4_byval;
-  case $enableval in
-    yes)
-
-$as_echo "#define USE_FLOAT4_BYVAL 1" >>confdefs.h
-
-               float4passbyval=true
-      ;;
-    no)
-      float4passbyval=false
-      ;;
-    *)
-      as_fn_error $? "no argument expected for --enable-float4-byval option" "$LINENO" 5
-      ;;
-  esac
-
-else
-  enable_float4_byval=yes
-
-$as_echo "#define USE_FLOAT4_BYVAL 1" >>confdefs.h
-
-               float4passbyval=true
-fi
-
-
-{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $enable_float4_byval" >&5
-$as_echo "$enable_float4_byval" >&6; }
-
-cat >>confdefs.h <<_ACEOF
-#define FLOAT4PASSBYVAL $float4passbyval
-_ACEOF
-
-
 # Decide whether float8 is passed by value.
 # Note: this setting also controls int8 and related types such as timestamp.
 # If sizeof(Datum) >= 8, this is user-selectable, enabled by default.
index 1ea1c465504427e2e51d4d87f99ea3d4f8c5ab18..56a177ba10299e42518a6df73725cef8b8e339d6 100644 (file)
@@ -1941,16 +1941,6 @@ AC_CHECK_SIZEOF([void *])
 AC_CHECK_SIZEOF([size_t])
 AC_CHECK_SIZEOF([long])
 
-# Decide whether float4 is passed by value: user-selectable, enabled by default
-AC_MSG_CHECKING([whether to build with float4 passed by value])
-PGAC_ARG_BOOL(enable, float4-byval, yes, [disable float4 passed by value],
-              [AC_DEFINE([USE_FLOAT4_BYVAL], 1,
-                         [Define to 1 if you want float4 values to be passed by value. (--enable-float4-byval)])
-               float4passbyval=true],
-              [float4passbyval=false])
-AC_MSG_RESULT([$enable_float4_byval])
-AC_DEFINE_UNQUOTED([FLOAT4PASSBYVAL], [$float4passbyval], [float4 values are passed by value if 'true', by reference if 'false'])
-
 # Decide whether float8 is passed by value.
 # Note: this setting also controls int8 and related types such as timestamp.
 # If sizeof(Datum) >= 8, this is user-selectable, enabled by default.
index e8082c05c9508a642cbc4e10e3f76e890b355e57..57a1539506d00755dfcd49252ffc7edd6e8d53f4 100644 (file)
@@ -19964,11 +19964,6 @@ SELECT collation for ('foo' COLLATE "de_DE");
        <entry><type>integer</type></entry>
       </row>
 
-      <row>
-       <entry><literal>float4_pass_by_value</literal></entry>
-       <entry><type>boolean</type></entry>
-      </row>
-
       <row>
        <entry><literal>float8_pass_by_value</literal></entry>
        <entry><type>boolean</type></entry>
index d8494e293bd8418e2d46f6ab88a7475974339646..b4d222295e18739fc692b4d9a4eddc14c64c9211 100644 (file)
@@ -1204,18 +1204,6 @@ build-postgresql:
        </listitem>
       </varlistentry>
 
-      <varlistentry>
-       <term><option>--disable-float4-byval</option></term>
-       <listitem>
-        <para>
-         Disable passing float4 values <quote>by value</quote>, causing them
-         to be passed <quote>by reference</quote> instead.  This option costs
-         performance, but may be needed for compatibility with very old
-         user-defined functions written in C.
-        </para>
-       </listitem>
-      </varlistentry>
-
       <varlistentry>
        <term><option>--disable-float8-byval</option></term>
        <listitem>
index 9dfa0ddfbb6052be623876ad3ab3807b7d71b6eb..4af418287d22c53fcb7e6d8f866d0a9de59b2833 100644 (file)
@@ -879,11 +879,6 @@ index_store_float8_orderby_distances(IndexScanDesc scan, Oid *orderByTypes,
        else if (orderByTypes[i] == FLOAT4OID)
        {
            /* convert distance function's result to ORDER BY type */
-#ifndef USE_FLOAT4_BYVAL
-           /* must free any old value to avoid memory leakage */
-           if (!scan->xs_orderbynulls[i])
-               pfree(DatumGetPointer(scan->xs_orderbyvals[i]));
-#endif
            if (distances && !distances[i].isnull)
            {
                scan->xs_orderbyvals[i] = Float4GetDatum((float4) distances[i].value);
index 3b766e66b9ecb27b9e8c944ec958486c933d87be..5f0ee50092cb7adc91aacea62b210f61b1072944 100644 (file)
@@ -4516,7 +4516,6 @@ WriteControlFile(void)
    ControlFile->toast_max_chunk_size = TOAST_MAX_CHUNK_SIZE;
    ControlFile->loblksize = LOBLKSIZE;
 
-   ControlFile->float4ByVal = FLOAT4PASSBYVAL;
    ControlFile->float8ByVal = FLOAT8PASSBYVAL;
 
    /* Contents are protected with a CRC */
@@ -4720,22 +4719,6 @@ ReadControlFile(void)
                           ControlFile->loblksize, (int) LOBLKSIZE),
                 errhint("It looks like you need to recompile or initdb.")));
 
-#ifdef USE_FLOAT4_BYVAL
-   if (ControlFile->float4ByVal != true)
-       ereport(FATAL,
-               (errmsg("database files are incompatible with server"),
-                errdetail("The database cluster was initialized without USE_FLOAT4_BYVAL"
-                          " but the server was compiled with USE_FLOAT4_BYVAL."),
-                errhint("It looks like you need to recompile or initdb.")));
-#else
-   if (ControlFile->float4ByVal != false)
-       ereport(FATAL,
-               (errmsg("database files are incompatible with server"),
-                errdetail("The database cluster was initialized with USE_FLOAT4_BYVAL"
-                          " but the server was compiled without USE_FLOAT4_BYVAL."),
-                errhint("It looks like you need to recompile or initdb.")));
-#endif
-
 #ifdef USE_FLOAT8_BYVAL
    if (ControlFile->float8ByVal != true)
        ereport(FATAL,
index 9238fbe98d7d1dfece034be04f09fa3da93a7d57..8ea033610d674460267a37a528bd7e6f27ab282c 100644 (file)
@@ -111,7 +111,7 @@ static const struct typinfo TypInfo[] = {
    F_INT2IN, F_INT2OUT},
    {"int4", INT4OID, 0, 4, true, 'i', 'p', InvalidOid,
    F_INT4IN, F_INT4OUT},
-   {"float4", FLOAT4OID, 0, 4, FLOAT4PASSBYVAL, 'i', 'p', InvalidOid,
+   {"float4", FLOAT4OID, 0, 4, true, 'i', 'p', InvalidOid,
    F_FLOAT4IN, F_FLOAT4OUT},
    {"name", NAMEOID, CHAROID, NAMEDATALEN, false, 'c', 'p', C_COLLATION_OID,
    F_NAMEIN, F_NAMEOUT},
index 201d12d358c646f927582795e05f82f4119d26d2..6eff045bd9a8251f117cbe4578aeadacb42a43d5 100644 (file)
@@ -890,7 +890,7 @@ sub morph_row_for_schemapg
        }
 
        # Expand booleans from 'f'/'t' to 'false'/'true'.
-       # Some values might be other macros (eg FLOAT4PASSBYVAL),
+       # Some values might be other macros (eg FLOAT8PASSBYVAL),
        # don't change.
        elsif ($atttype eq 'bool')
        {
index 7accb950eb1bda314c88fe9b2da8bd5c6a50b877..71372ceb161cd8a755a9f6dd62fa36cd0ad92dd8 100644 (file)
@@ -1496,7 +1496,7 @@ update_attstats(Oid relid, bool inh, int natts, VacAttrStats **vacattrstats)
                /* XXX knows more than it should about type float4: */
                arry = construct_array(numdatums, nnum,
                                       FLOAT4OID,
-                                      sizeof(float4), FLOAT4PASSBYVAL, 'i');
+                                      sizeof(float4), true, 'i');
                values[i++] = PointerGetDatum(arry);    /* stanumbersN */
            }
            else
index c8d2cef35fffed0c43b1fdad3c518ac7449a8896..37992ffd4482047d07a0a718ed76fb25b524c277 100644 (file)
@@ -365,15 +365,6 @@ incompatible_module_error(const char *libname,
                         magic_data.namedatalen,
                         module_magic_data->namedatalen);
    }
-   if (module_magic_data->float4byval != magic_data.float4byval)
-   {
-       if (details.len)
-           appendStringInfoChar(&details, '\n');
-       appendStringInfo(&details,
-                        _("Server has FLOAT4PASSBYVAL = %s, library has %s."),
-                        magic_data.float4byval ? "true" : "false",
-                        module_magic_data->float4byval ? "true" : "false");
-   }
    if (module_magic_data->float8byval != magic_data.float8byval)
    {
        if (details.len)
index 099ebd779ba26d2d2ee5f019d7d9f9e1505c0d82..2ce7a866c9050082113d488ba3b031a4605c1b08 100644 (file)
@@ -1683,7 +1683,7 @@ OidSendFunctionCall(Oid functionId, Datum val)
 /*-------------------------------------------------------------------------
  *     Support routines for standard maybe-pass-by-reference datatypes
  *
- * int8, float4, and float8 can be passed by value if Datum is wide enough.
+ * int8 and float8 can be passed by value if Datum is wide enough.
  * (For backwards-compatibility reasons, we allow pass-by-ref to be chosen
  * at compile time even if pass-by-val is possible.)
  *
@@ -1703,21 +1703,6 @@ Int64GetDatum(int64 X)
    *retval = X;
    return PointerGetDatum(retval);
 }
-#endif                         /* USE_FLOAT8_BYVAL */
-
-#ifndef USE_FLOAT4_BYVAL
-
-Datum
-Float4GetDatum(float4 X)
-{
-   float4     *retval = (float4 *) palloc(sizeof(float4));
-
-   *retval = X;
-   return PointerGetDatum(retval);
-}
-#endif
-
-#ifndef USE_FLOAT8_BYVAL
 
 Datum
 Float8GetDatum(float8 X)
@@ -1727,7 +1712,7 @@ Float8GetDatum(float8 X)
    *retval = X;
    return PointerGetDatum(retval);
 }
-#endif
+#endif                         /* USE_FLOAT8_BYVAL */
 
 
 /*-------------------------------------------------------------------------
index 3d0916fea7058619c79d5b5a9bff9db13c1f23ec..7007d6503f86592e06a049d44082690668a1c33b 100644 (file)
@@ -264,8 +264,8 @@ pg_control_recovery(PG_FUNCTION_ARGS)
 Datum
 pg_control_init(PG_FUNCTION_ARGS)
 {
-   Datum       values[12];
-   bool        nulls[12];
+   Datum       values[11];
+   bool        nulls[11];
    TupleDesc   tupdesc;
    HeapTuple   htup;
    ControlFileData *ControlFile;
@@ -294,11 +294,9 @@ pg_control_init(PG_FUNCTION_ARGS)
                       INT4OID, -1, 0);
    TupleDescInitEntry(tupdesc, (AttrNumber) 9, "large_object_chunk_size",
                       INT4OID, -1, 0);
-   TupleDescInitEntry(tupdesc, (AttrNumber) 10, "float4_pass_by_value",
+   TupleDescInitEntry(tupdesc, (AttrNumber) 10, "float8_pass_by_value",
                       BOOLOID, -1, 0);
-   TupleDescInitEntry(tupdesc, (AttrNumber) 11, "float8_pass_by_value",
-                      BOOLOID, -1, 0);
-   TupleDescInitEntry(tupdesc, (AttrNumber) 12, "data_page_checksum_version",
+   TupleDescInitEntry(tupdesc, (AttrNumber) 11, "data_page_checksum_version",
                       INT4OID, -1, 0);
    tupdesc = BlessTupleDesc(tupdesc);
 
@@ -335,15 +333,12 @@ pg_control_init(PG_FUNCTION_ARGS)
    values[8] = Int32GetDatum(ControlFile->loblksize);
    nulls[8] = false;
 
-   values[9] = BoolGetDatum(ControlFile->float4ByVal);
+   values[9] = BoolGetDatum(ControlFile->float8ByVal);
    nulls[9] = false;
 
-   values[10] = BoolGetDatum(ControlFile->float8ByVal);
+   values[10] = Int32GetDatum(ControlFile->data_checksum_version);
    nulls[10] = false;
 
-   values[11] = Int32GetDatum(ControlFile->data_checksum_version);
-   nulls[11] = false;
-
    htup = heap_form_tuple(tupdesc, values, nulls);
 
    PG_RETURN_DATUM(HeapTupleGetDatum(htup));
index 88a261d9bda104e682552c8c38978510c8717867..1f6d8939bec264d299071336572d4f389fbf9e73 100644 (file)
@@ -1399,9 +1399,6 @@ bootstrap_template1(void)
    bki_lines = replace_token(bki_lines, "ALIGNOF_POINTER",
                              (sizeof(Pointer) == 4) ? "i" : "d");
 
-   bki_lines = replace_token(bki_lines, "FLOAT4PASSBYVAL",
-                             FLOAT4PASSBYVAL ? "true" : "false");
-
    bki_lines = replace_token(bki_lines, "FLOAT8PASSBYVAL",
                              FLOAT8PASSBYVAL ? "true" : "false");
 
index b14767f8b608e4147f4e6311668d9d8a4f225914..19e21ab4912afecc9d83e24aa9468f0c97c05e59 100644 (file)
@@ -327,8 +327,6 @@ main(int argc, char *argv[])
    /* This is no longer configurable, but users may still expect to see it: */
    printf(_("Date/time type storage:               %s\n"),
           _("64-bit integers"));
-   printf(_("Float4 argument passing:              %s\n"),
-          (ControlFile->float4ByVal ? _("by value") : _("by reference")));
    printf(_("Float8 argument passing:              %s\n"),
           (ControlFile->float8ByVal ? _("by value") : _("by reference")));
    printf(_("Data page checksum version:           %u\n"),
index c4ee0168a918f598b80778e7f3e16f317ea4cfa8..2e286f6339d6385f28332c2cc562c9fd5b42ba89 100644 (file)
@@ -722,7 +722,6 @@ GuessControlValues(void)
    ControlFile.indexMaxKeys = INDEX_MAX_KEYS;
    ControlFile.toast_max_chunk_size = TOAST_MAX_CHUNK_SIZE;
    ControlFile.loblksize = LOBLKSIZE;
-   ControlFile.float4ByVal = FLOAT4PASSBYVAL;
    ControlFile.float8ByVal = FLOAT8PASSBYVAL;
 
    /*
@@ -801,8 +800,6 @@ PrintControlValues(bool guessed)
    /* This is no longer configurable, but users may still expect to see it: */
    printf(_("Date/time type storage:               %s\n"),
           _("64-bit integers"));
-   printf(_("Float4 argument passing:              %s\n"),
-          (ControlFile.float4ByVal ? _("by value") : _("by reference")));
    printf(_("Float8 argument passing:              %s\n"),
           (ControlFile.float8ByVal ? _("by value") : _("by reference")));
    printf(_("Data page checksum version:           %u\n"),
index 555384cd2bf655bea5fb6d6b47d7e9259cac5211..b6b380cfc24f89fe3d5c9b737f911dbec91ca21e 100644 (file)
@@ -53,6 +53,6 @@
  */
 
 /*                         yyyymmddN */
-#define CATALOG_VERSION_NO 201911211
+#define CATALOG_VERSION_NO 201911212
 
 #endif
index ff98d9e91a8b12cee111dd667680341d44aca8df..cf7d4485e9bc90fe6d297312e9225d3f7cb323a6 100644 (file)
@@ -22,7 +22,7 @@
 
 
 /* Version identifier for this pg_control format */
-#define PG_CONTROL_VERSION 1201
+#define PG_CONTROL_VERSION 1300
 
 /* Nonce key length, see below */
 #define MOCK_AUTH_NONCE_LEN        32
@@ -214,8 +214,6 @@ typedef struct ControlFileData
    uint32      toast_max_chunk_size;   /* chunk size in TOAST tables */
    uint32      loblksize;      /* chunk size in pg_largeobject */
 
-   /* flags indicating pass-by-value status of various types */
-   bool        float4ByVal;    /* float4 pass-by-value? */
    bool        float8ByVal;    /* float8, int8, etc pass-by-value? */
 
    /* Are data pages protected by checksums? Zero if no checksum version */
index fa0a2a10021f4be41b70b6f65ba421b36f1a8e05..d4fa6dd6491c104eef53054c93a05422629a5b83 100644 (file)
   descr => 'pg_controldata init state information as a function',
   proname => 'pg_control_init', provolatile => 'v', prorettype => 'record',
   proargtypes => '',
-  proallargtypes => '{int4,int4,int4,int4,int4,int4,int4,int4,int4,bool,bool,int4}',
-  proargmodes => '{o,o,o,o,o,o,o,o,o,o,o,o}',
-  proargnames => '{max_data_alignment,database_block_size,blocks_per_segment,wal_block_size,bytes_per_wal_segment,max_identifier_length,max_index_columns,max_toast_chunk_size,large_object_chunk_size,float4_pass_by_value,float8_pass_by_value,data_page_checksum_version}',
+  proallargtypes => '{int4,int4,int4,int4,int4,int4,int4,int4,int4,bool,int4}',
+  proargmodes => '{o,o,o,o,o,o,o,o,o,o,o}',
+  proargnames => '{max_data_alignment,database_block_size,blocks_per_segment,wal_block_size,bytes_per_wal_segment,max_identifier_length,max_index_columns,max_toast_chunk_size,large_object_chunk_size,float8_pass_by_value,data_page_checksum_version}',
   prosrc => 'pg_control_init' },
 
 # collation management functions
index be49e0011445da5c0b2de49eb50748fb43882ac6..d9b35af914afa69b06ba79571f3a3e5c9cc3caa4 100644 (file)
 
 { oid => '700', array_type_oid => '1021',
   descr => 'single-precision floating point number, 4-byte storage',
-  typname => 'float4', typlen => '4', typbyval => 'FLOAT4PASSBYVAL',
+  typname => 'float4', typlen => '4', typbyval => 't',
   typcategory => 'N', typinput => 'float4in', typoutput => 'float4out',
   typreceive => 'float4recv', typsend => 'float4send', typalign => 'i' },
 { oid => '701', array_type_oid => '1022',
index 29ae4674cc7e750a6690e28fe9d1450633d11682..724ee73bdedf6821214ee00ee6f3557516b61451 100644 (file)
@@ -446,7 +446,6 @@ typedef struct
    int         funcmaxargs;    /* FUNC_MAX_ARGS */
    int         indexmaxkeys;   /* INDEX_MAX_KEYS */
    int         namedatalen;    /* NAMEDATALEN */
-   int         float4byval;    /* FLOAT4PASSBYVAL */
    int         float8byval;    /* FLOAT8PASSBYVAL */
 } Pg_magic_struct;
 
@@ -458,7 +457,6 @@ typedef struct
    FUNC_MAX_ARGS, \
    INDEX_MAX_KEYS, \
    NAMEDATALEN, \
-   FLOAT4PASSBYVAL, \
    FLOAT8PASSBYVAL \
 }
 
index fc22526b7e399021dd5cd52bb667f09ae2875478..6f8549bc03b1f59d835e4eeddd67240e40c84a2e 100644 (file)
@@ -70,9 +70,6 @@
    MSVC and with C++ compilers. */
 #undef FLEXIBLE_ARRAY_MEMBER
 
-/* float4 values are passed by value if 'true', by reference if 'false' */
-#undef FLOAT4PASSBYVAL
-
 /* float8, int8, and related values are passed by value if 'true', by
    reference if 'false' */
 #undef FLOAT8PASSBYVAL
 /* Define to use /dev/urandom for random number generation */
 #undef USE_DEV_URANDOM
 
-/* Define to 1 if you want float4 values to be passed by value.
-   (--enable-float4-byval) */
-#undef USE_FLOAT4_BYVAL
-
 /* Define to 1 if you want float8, int8, etc values to be passed by value.
    (--enable-float8-byval) */
 #undef USE_FLOAT8_BYVAL
index 057a3413acd8a8ce44d75fbf0bf62cfeb9824bae..f5b7c52f8a8a248461e184452bc66ea259789530 100644 (file)
@@ -659,11 +659,7 @@ extern Datum Int64GetDatum(int64 X);
 /*
  * DatumGetFloat4
  *     Returns 4-byte floating point value of a datum.
- *
- * Note: this macro hides whether float4 is pass by value or by reference.
  */
-
-#ifdef USE_FLOAT4_BYVAL
 static inline float4
 DatumGetFloat4(Datum X)
 {
@@ -676,18 +672,11 @@ DatumGetFloat4(Datum X)
    myunion.value = DatumGetInt32(X);
    return myunion.retval;
 }
-#else
-#define DatumGetFloat4(X) (* ((float4 *) DatumGetPointer(X)))
-#endif
 
 /*
  * Float4GetDatum
  *     Returns datum representation for a 4-byte floating point number.
- *
- * Note: if float4 is pass by reference, this function returns a reference
- * to palloc'd space.
  */
-#ifdef USE_FLOAT4_BYVAL
 static inline Datum
 Float4GetDatum(float4 X)
 {
@@ -700,9 +689,6 @@ Float4GetDatum(float4 X)
    myunion.value = X;
    return Int32GetDatum(myunion.retval);
 }
-#else
-extern Datum Float4GetDatum(float4 X);
-#endif
 
 /*
  * DatumGetFloat8
@@ -757,10 +743,9 @@ extern Datum Float8GetDatum(float8 X);
 /*
  * Int64GetDatumFast
  * Float8GetDatumFast
- * Float4GetDatumFast
  *
  * These macros are intended to allow writing code that does not depend on
- * whether int64, float8, float4 are pass-by-reference types, while not
+ * whether int64 and float8 are pass-by-reference types, while not
  * sacrificing performance when they are.  The argument must be a variable
  * that will exist and have the same value for as long as the Datum is needed.
  * In the pass-by-ref case, the address of the variable is taken to use as
@@ -776,10 +761,4 @@ extern Datum Float8GetDatum(float8 X);
 #define Float8GetDatumFast(X) PointerGetDatum(&(X))
 #endif
 
-#ifdef USE_FLOAT4_BYVAL
-#define Float4GetDatumFast(X) Float4GetDatum(X)
-#else
-#define Float4GetDatumFast(X) PointerGetDatum(&(X))
-#endif
-
 #endif                         /* POSTGRES_H */
index 7f179f194be641bde846b470b27641ee45cdb2b6..e4ea62e80ba276ea34239263562a5a5490ab36e7 100644 (file)
@@ -31,8 +31,6 @@ sub _new
    $self->DeterminePlatform();
    my $bits = $self->{platform} eq 'Win32' ? 32 : 64;
 
-   $options->{float4byval} = 1
-     unless exists $options->{float4byval};
    $options->{float8byval} = ($bits == 64)
      unless exists $options->{float8byval};
    die "float8byval not permitted on 32 bit platforms"
@@ -209,15 +207,6 @@ sub GenerateFiles
        print $o "#define XLOG_BLCKSZ ",
          1024 * $self->{options}->{wal_blocksize}, "\n";
 
-       if ($self->{options}->{float4byval})
-       {
-           print $o "#define USE_FLOAT4_BYVAL 1\n";
-           print $o "#define FLOAT4PASSBYVAL true\n";
-       }
-       else
-       {
-           print $o "#define FLOAT4PASSBYVAL false\n";
-       }
        if ($self->{options}->{float8byval})
        {
            print $o "#define USE_FLOAT8_BYVAL 1\n";
index 043df4c5e871d5f21edde74e280806639df01e2a..62188c78e78f8ff9b685d59bf0ba613ad702ec9b 100644 (file)
@@ -4,7 +4,6 @@ use warnings;
 
 our $config = {
    asserts => 0,    # --enable-cassert
-        # float4byval=>1,         # --disable-float4-byval, on by default
 
    # float8byval=> $platformbits == 64, # --disable-float8-byval,
    # off by default on 32 bit platforms, on by default on 64 bit platforms