git: f7a10a775259 - main - daemon: Prefer sys/cdefs.h __unreachable over the builtin

From: Warner Losh <imp_at_FreeBSD.org>
Date: Fri, 12 Apr 2024 22:42:47 UTC
The branch main has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=f7a10a7752592c0bec6e6ea377f3f6640e9ecfa5

commit f7a10a7752592c0bec6e6ea377f3f6640e9ecfa5
Author:     Collin Funk <collin.funk1@gmail.com>
AuthorDate: 2024-04-12 22:38:18 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2024-04-12 22:39:18 +0000

    daemon: Prefer sys/cdefs.h __unreachable over the builtin
    
    The __builtin_unreachable macro provided by Clang and GCC is a hint to
    the compiler used for optimization. The programs work fine even if the
    compiler doesn't support it. The sys/cdefs.h has had __unreachable for
    9 years (commit 732b31de5d9244bd1cc98192e09ee1881e9f55e9). It expands
    to the builtin if it is available. In the rare case that it is
    unsupported it expands to a null statement so compilation does not
    fail.
    
    Signed-off-by: Collin Funk <collin.funk1@gmail.com>
    Reviewed by: imp, freebsd@igalic.co
    Pull Request: https://github.com/freebsd/freebsd-src/pull/1117
---
 usr.sbin/daemon/daemon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/usr.sbin/daemon/daemon.c b/usr.sbin/daemon/daemon.c
index bce215af75d1..dd3315dadee4 100644
--- a/usr.sbin/daemon/daemon.c
+++ b/usr.sbin/daemon/daemon.c
@@ -275,7 +275,7 @@ main(int argc, char *argv[])
 			break;
 		case 'h':
 			usage(0);
-			__builtin_unreachable();
+			__unreachable();
 		default:
 			usage(1);
 		}