Rethink the generation rule for fmgroids.h macros.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 2 Nov 2020 16:57:28 +0000 (11:57 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 2 Nov 2020 16:57:28 +0000 (11:57 -0500)
commit8e1f37c07aafd4bb7aa6e1e1982010af11f8b5c7
tree93e0b3994b9a79e4f3342d2a4f74c3547ca7e3d8
parentfd2997565c6f66837440dd57f5e52b56aa964d14
Rethink the generation rule for fmgroids.h macros.

Traditionally, the names of fmgroids.h macros for pg_proc OIDs
have been constructed from the prosrc field.  But sometimes the
same C function underlies multiple pg_proc entries, forcing us
to make an arbitrary choice of which OID to reference; the other
entries are then not namable via fmgroids.h.  Moreover, we could
not have macros at all for pg_proc entries that aren't for
C-coded functions.

Instead, use the proname field, and append the proargtypes field
(replacing inter-argument spaces with underscores) if proname is
not unique.  Special-casing unique entries such as F_OIDEQ removes
the need to change a lot of code.  Indeed, I can only find two
places in the tree that need to be adjusted; while this changes
quite a few existing entries in fmgroids.h, few of them are
referenced from C code.

With this patch, all entries in pg_proc.dat have macros in fmgroids.h.

Discussion: https://postgr.es/m/472274.1604258384@sss.pgh.pa.us
src/backend/optimizer/util/clauses.c
src/backend/utils/Gen_fmgrtab.pl
src/backend/utils/adt/ruleutils.c