LVM2/daemons/clvmd clvmd.c

Alasdair G Kergon <[email protected]> Tue, 26 Oct 2010 11:46:19 +0100

Newsgroups	dev.linux.lists.lvm-devel
Message-ID	<[email protected]>

On Tue, Oct 26, 2010 at 09:57:04AM -0000, zkabelac at sourceware.org wrote:
> +#ifdef __clang__
>  		__builtin_unreachable();
> +#endif

Please do not scatter #ifdefs like that through the code.
If the standard /* NOTREACHED */ really doesn't work, then hide the
ifdef in a header file and use a #define to make it available precisely
where the compiler supports it, defining it to something empty
elsewhere.

Alasdair