lib: Don't use special code for i_unreached() with STATIC_CHECKER

This was done originally to help scan-build, but this is no longer
necessary. Also actually running code where __builtin_unreachable() was
reached produced (very weird) undefined behavior.

Reverts e2e9ea6da9f3db5fd7fe467db79232d20d03832f

diff --git a/src/lib/macros.h b/src/lib/macros.h
index c66e26ef0bc252680e4fd035e4c1fcc5a3fb92bd..cecd56913a081d02cbd8cbf83aa2284ca6b450a6 100644 (file)
--- a/src/lib/macros.h
+++ b/src/lib/macros.h
@@ -234,12 +234,8 @@
 
 #endif
 
-#ifndef STATIC_CHECKER
-#  define i_unreached() \
+#define i_unreached() \
        i_panic("file %s: line %d: unreached", __FILE__, __LINE__)
-#else
-#  define i_unreached() __builtin_unreachable()
-#endif
 
 /* Convenience macros to test the versions of dovecot. */
 #if defined DOVECOT_VERSION_MAJOR && defined DOVECOT_VERSION_MINOR