Move -ffast-math defense to float.c and remove the configure check. We had defenses against -ffast-math in timestamp-related files, which is a pretty obsolete place for them since we've not supported floating-point timestamps in a long time. Remove those and instead put one in float.c, which is still broken by using this switch. Add some commentary to put more color on why it's a bad idea.
Also remove the check from configure. That was just there to fail faster, but it doesn't really seem necessary anymore, and besides we have no corresponding check in meson.build. Author: Bertrand Drouvot <[email protected]> Suggested-by: Andres Freund <[email protected]> Suggested-by: Peter Eisentraut <[email protected]> Reviewed-by: Tom Lane <[email protected]> Discussion: https://postgr.es/m/abFXfKC8zR0Oclon%40ip-10-97-1-34.eu-west-3.compute.internal Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/82ff54377e557914d975d22c4847d98d0efa1dac Modified Files -------------- configure | 23 ----------------------- configure.ac | 7 ------- src/backend/utils/adt/date.c | 8 -------- src/backend/utils/adt/float.c | 17 +++++++++++++++++ src/backend/utils/adt/timestamp.c | 7 ------- src/interfaces/ecpg/pgtypeslib/interval.c | 4 ---- src/interfaces/ecpg/pgtypeslib/timestamp.c | 4 ---- 7 files changed, 17 insertions(+), 53 deletions(-)
