You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Tom Lane b853eb9718 Improve handling of ereport(ERROR) and elog(ERROR). ... In commit 71450d7fd6, we added code to inform suitably-intelligent compilers that ereport() doesn't return if the elevel is ERROR or higher. This patch extends that to elog(), and also fixes a double-evaluation hazard that the previous commit created in ereport(), as well as reducing the emitted code size. The elog() improvement requires the compiler to support __VA_ARGS__, which should be available in just about anything nowadays since it's required by C99. But our minimum language baseline is still C89, so add a configure test for that. The previous commit assumed that ereport's elevel could be evaluated twice, which isn't terribly safe --- there are already counterexamples in xlog.c. On compilers that have __builtin_constant_p, we can use that to protect the second test, since there's no possible optimization gain if the compiler doesn't know the value of elevel. Otherwise, use a local variable inside the macros to prevent double evaluation. The local-variable solution is inferior because (a) it leads to useless code being emitted when elevel isn't constant, and (b) it increases the optimization level needed for the compiler to recognize that subsequent code is unreachable. But it seems better than not teaching non-gcc compilers about unreachability at all. Lastly, if the compiler has __builtin_unreachable(), we can use that instead of abort(), resulting in a noticeable code savings since no function call is actually emitted. However, it seems wise to do this only in non-assert builds. In an assert build, continue to use abort(), so that the behavior will be predictable and debuggable if the "impossible" happens. These changes involve making the ereport and elog macros emit do-while statement blocks not just expressions, which forces small changes in a few call sites. Andres Freund, Tom Lane, Heikki Linnakangas		13 years ago
..
Makefile	Remove cvs keywords from all files.	15 years ago
ac_func_accept_argtypes.m4	Enable compiling with the mingw-w64 32 bit compiler.	14 years ago
acx_pthread.m4	Better support for thread-support flag detection with clang	15 years ago
c-compiler.m4	Improve handling of ereport(ERROR) and elog(ERROR).	13 years ago
c-library.m4	Put back AC_REQUIRE([AC_STRUCT_TM]).	13 years ago
config.guess	Update config.guess and config.sub	13 years ago
config.sub	Update config.guess and config.sub	13 years ago
docbook.m4	Fix configure's search for collateindex.pl.	13 years ago
general.m4	Remove useless whitespace at end of lines	15 years ago
install-sh	Fix install-strip on Mac OS X	13 years ago
libtool.m4	Rename AC_PROG_LD* macros to PGAC_PROG_LD*. This avoids clashes with the	21 years ago
missing	Remove cvs keywords from all files.	15 years ago
perl.m4	Detect Windows perl linkage parameters in configure script.	13 years ago
prep_buildtree	find -path is not portable, so use grep -v instead.	15 years ago
programs.m4	Fix install-strip on Mac OS X	13 years ago
python.m4	Fix plpython build on older versions of OS X.	13 years ago
tcl.m4	Remove cvs keywords from all files.	15 years ago