clang fails to compile emacs in Microsoft Windows

clang fails to compile emacs in Microsoft Windows

[Biswapriyo Nath]

* compiler:
clang version 16.0.2
Target: x86_64-w64-windows-gnu
Thread model: posix

* compiler error:

../../src/sysdep.c:472:13: error: call to undeclared function
'waitpid'; ISO C99 and later do not support implicit function
declarations [-Wimplicit-function-declaration]
../../src/sysdep.c:518:43: error: use of undeclared identifier 'WNOHANG'

* Probable reason:

AC_HEADER_SYS_WAIT fails to check sys/wait.h file. It tests with
wait() function which is undefined in nt/inc/sys/wait.h file. Here is
the error message from config.log

configure:11452: checking for sys/wait.h that is POSIX.1 compatible
configure:11479: x86_64-w64-mingw32-cc -I ../nt/inc -c -g3 -O2
-gdwarf-2 -mtune=generic  -D__USE_MINGW_ANSI_STDIO=1 conftest.c >&5
conftest.c:70:3: error: call to undeclared function 'wait'; ISO C99
and later do not support implicit function declarations
[-Wimplicit-function-declaration]

I can provide further information if required. Thank you.

Re: clang fails to compile emacs in Microsoft Windows

[Eli Zaretskii]

> From: Biswapriyo Nath <nathbappai@gmail.com>
> Date: Fri, 12 May 2023 19:41:17 +0530
> 
> * compiler:
> clang version 16.0.2
> Target: x86_64-w64-windows-gnu
> Thread model: posix
> 
> * compiler error:
> 
> ../../src/sysdep.c:472:13: error: call to undeclared function
> 'waitpid'; ISO C99 and later do not support implicit function
> declarations [-Wimplicit-function-declaration]
> ../../src/sysdep.c:518:43: error: use of undeclared identifier 'WNOHANG'
> 
> * Probable reason:
> 
> AC_HEADER_SYS_WAIT fails to check sys/wait.h file. It tests with
> wait() function which is undefined in nt/inc/sys/wait.h file. Here is
> the error message from config.log
> 
> configure:11452: checking for sys/wait.h that is POSIX.1 compatible
> configure:11479: x86_64-w64-mingw32-cc -I ../nt/inc -c -g3 -O2
> -gdwarf-2 -mtune=generic  -D__USE_MINGW_ANSI_STDIO=1 conftest.c >&5
> conftest.c:70:3: error: call to undeclared function 'wait'; ISO C99
> and later do not support implicit function declarations
> [-Wimplicit-function-declaration]
> 
> I can provide further information if required. Thank you.

Thanks, but Clang is not supported for building Emacs on MS-Windows.
We only support MinGW GCC.  Unless someone steps up and takes upon
themselves the job of supporting the Clang build, this will remain so.
Supporting the MS-Windows build is hard and thankless job as it is;
adding another compiler on top of that is just too much.

Sorry.

Re: clang fails to compile emacs in Microsoft Windows

[Óscar Fuentes]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Biswapriyo Nath <nathbappai@gmail.com>
>> Date: Fri, 12 May 2023 19:41:17 +0530
>> 
>> * compiler:
>> clang version 16.0.2
>> Target: x86_64-w64-windows-gnu
>> Thread model: posix
>> 
>> * compiler error:
>> 
>> ../../src/sysdep.c:472:13: error: call to undeclared function
>> 'waitpid'; ISO C99 and later do not support implicit function
>> declarations [-Wimplicit-function-declaration]
>> ../../src/sysdep.c:518:43: error: use of undeclared identifier 'WNOHANG'
>> 
>> * Probable reason:
>> 
>> AC_HEADER_SYS_WAIT fails to check sys/wait.h file. It tests with
>> wait() function which is undefined in nt/inc/sys/wait.h file. Here is
>> the error message from config.log
>> 
>> configure:11452: checking for sys/wait.h that is POSIX.1 compatible
>> configure:11479: x86_64-w64-mingw32-cc -I ../nt/inc -c -g3 -O2
>> -gdwarf-2 -mtune=generic  -D__USE_MINGW_ANSI_STDIO=1 conftest.c >&5
>> conftest.c:70:3: error: call to undeclared function 'wait'; ISO C99
>> and later do not support implicit function declarations
>> [-Wimplicit-function-declaration]
>> 
>> I can provide further information if required. Thank you.
>
> Thanks, but Clang is not supported for building Emacs on MS-Windows.
> We only support MinGW GCC.  Unless someone steps up and takes upon
> themselves the job of supporting the Clang build, this will remain so.
> Supporting the MS-Windows build is hard and thankless job as it is;
> adding another compiler on top of that is just too much.
>
> Sorry.

I think that the problem reported by the OP was introduced recently
(and, by the looks of it, it is problematic not just because of Clang,
but as a C construct, because it seems to rely on an implicit function
declaration. Yes, I've seen the thread on gcc-devel, so no need to
explain things further).

Emacs 28.2 compiles with Clang on Windows with this simple patch:

diff -aur emacs-28.2-orig/configure.ac emacs-28.2/configure.ac
--- emacs-28.2-orig/configure.ac	2022-09-07 06:24:51.000000000 +0800
+++ emacs-28.2/configure.ac	2022-10-09 20:40:07.436552500 +0800
@@ -147,7 +147,7 @@
       fi
       cc_target=`$cc -v 2>&1 | sed -n 's/Target: //p'`
       case "$cc_target" in
-          *-*) host=$cc_target
+          *-*) host=$target_alias
 	      ;;
           "") AC_MSG_ERROR([Impossible to obtain $cc compiler target.
 Please explicitly provide --host.])
@@ -5765,8 +5765,8 @@
   mingw32)
    ## Is it any better under MinGW64 to relocate emacs into higher addresses?
    case "$canonical" in
-     x86_64-*-*) LD_SWITCH_SYSTEM_TEMACS="-Wl,-stack,0x00800000 -Wl,-heap,0x00100000 -Wl,-image-base,0x400000000 -Wl,-entry,__start -Wl,-Map,./temacs.map" ;;
-     *) LD_SWITCH_SYSTEM_TEMACS="-Wl,-stack,0x00800000 -Wl,-heap,0x00100000 -Wl,-image-base,0x01000000 -Wl,-entry,__start -Wl,-Map,./temacs.map" ;;
+     x86_64-*-*) LD_SWITCH_SYSTEM_TEMACS="-Wl,-stack,0x00800000 -Wl,-image-base,0x400000000 -Wl,-entry,__start -Wl,-Map,./temacs.map" ;;
+     *) LD_SWITCH_SYSTEM_TEMACS="-Wl,-stack,0x00800000 -Wl,-image-base,0x01000000 -Wl,-entry,__start -Wl,-Map,./temacs.map" ;;
    esac
    ## If they want unexec, disable Windows ASLR for the Emacs binary
    if test "$with_dumping" = "unexec"; then



and this one for UCRT in general:



--- emacs-28.1/nt/cmdproxy.c.orig	2022-06-23 19:24:10.845481100 +0200
+++ emacs-28.1/nt/cmdproxy.c	2022-06-23 19:24:19.271132900 +0200
@@ -35,6 +35,10 @@
 #include <string.h>  /* strlen */
 #include <ctype.h>   /* isspace, isalpha */
 
+#ifdef _UCRT
+#define _snprintf snprintf
+#endif
+
 /* We don't want to include stdio.h because we are already duplicating
    lots of it here */
 extern int _snprintf (char *buffer, size_t count, const char *format, ...);
--- emacs-28.1/src/sysdep.c.orig	2022-06-23 20:59:02.252891700 +0200
+++ emacs-28.1/src/sysdep.c	2022-06-23 21:05:31.998322600 +0200
@@ -2693,7 +2693,9 @@
   if (close_stream (stdout) != 0)
     {
+#ifndef _UCRT // XXX: close_stream(stdout) fails for some reason
       emacs_perror ("Write error to standard output");
       _exit (EXIT_FAILURE);
+#endif
     }
 
   /* Do not close stderr if addresses are being sanitized, as the
@@ -2702,7 +2704,11 @@
   if (err | (ADDRESS_SANITIZER
 	     ? fflush (stderr) != 0 || ferror (stderr)
 	     : close_stream (stderr) != 0))
+#ifndef _UCRT // XXX: close_stream(stderr) fails for some reason
     _exit (EXIT_FAILURE);
+#else
+    ;
+#endif
 }
 \f
 #ifndef DOS_NT


Those patches are kept in the MSYS2/Mingw-packages repository and we
provide clang-built Emacs 28.x binary packages.

I think that the OP was not requesting support from the Emacs
maintainers specifically, that's why he posted under -help and not under
-devel or -bugs.

Re: clang fails to compile emacs in Microsoft Windows

[Biswapriyo Nath]

It does not look like clang only issue. The check with
AC_HEADER_SYS_WAIT also fails in gcc but with a compiler warning as
following.

configure:11452: checking for sys/wait.h that is POSIX.1 compatible
configure:11479: x86_64-w64-mingw32-gcc -I ../nt/inc -c -g3 -O2
-gdwarf-2 -mtune=generic  -D__USE_MINGW_ANSI_STDIO=1 conftest.c >&5
conftest.c: In function 'main':
conftest.c:67:3: warning: implicit declaration of function 'wait'
[-Wimplicit-function-declaration]

Re: clang fails to compile emacs in Microsoft Windows

[Biswapriyo Nath]

On Fri, May 12, 2023 at 9:29 PM Biswapriyo Nath <nathbappai@gmail.com> wrote:
>
> It does not look like clang only issue. The check with
> AC_HEADER_SYS_WAIT also fails in gcc but with a compiler warning as
> following.

Mistake. I meant that the check succeed using gcc but with a compiler warning.

Re: clang fails to compile emacs in Microsoft Windows

[Eli Zaretskii]

> From: Óscar Fuentes <ofv@wanadoo.es>
> Date: Fri, 12 May 2023 17:47:25 +0200
> 
> Those patches are kept in the MSYS2/Mingw-packages repository and we
> provide clang-built Emacs 28.x binary packages.
> 
> I think that the OP was not requesting support from the Emacs
> maintainers specifically, that's why he posted under -help and not under
> -devel or -bugs.

If the MSYS2 project wants to maintain Clang patches, that's fine with
me.  But then these issues are best posted to the MSYS2 forum, not
here.

Re: clang fails to compile emacs in Microsoft Windows

[Eli Zaretskii]

> From: Biswapriyo Nath <nathbappai@gmail.com>
> Date: Fri, 12 May 2023 21:29:38 +0530
> Cc: ofv@wanadoo.es, eliz@gnu.org
> 
> It does not look like clang only issue. The check with
> AC_HEADER_SYS_WAIT also fails in gcc but with a compiler warning as
> following.
> 
> configure:11452: checking for sys/wait.h that is POSIX.1 compatible
> configure:11479: x86_64-w64-mingw32-gcc -I ../nt/inc -c -g3 -O2
> -gdwarf-2 -mtune=generic  -D__USE_MINGW_ANSI_STDIO=1 conftest.c >&5
> conftest.c: In function 'main':
> conftest.c:67:3: warning: implicit declaration of function 'wait'
> [-Wimplicit-function-declaration]

It doesn't matter, since the Windows build of Emacs doesn't call
'wait'.

Re: clang fails to compile emacs in Microsoft Windows

[Óscar Fuentes]

Eli Zaretskii <eliz@gnu.org> writes:

>> Those patches are kept in the MSYS2/Mingw-packages repository and we
>> provide clang-built Emacs 28.x binary packages.
>> 
>> I think that the OP was not requesting support from the Emacs
>> maintainers specifically, that's why he posted under -help and not under
>> -devel or -bugs.
>
> If the MSYS2 project wants to maintain Clang patches, that's fine with
> me.  But then these issues are best posted to the MSYS2 forum, not
> here.

Why? Here is the people that hack and build Emacs on a regular basis,
right?

Please note that the problem on the OP is not related to the patches.

Re: clang fails to compile emacs in Microsoft Windows

[Biswapriyo Nath]

Adding the wait() declaration in nt/inc/sys/wait.h fixes the issue.
May I send the patch here or in other mailing list?

Re: clang fails to compile emacs in Microsoft Windows

[Eli Zaretskii]

> From: Óscar Fuentes <ofv@wanadoo.es>
> Date: Fri, 12 May 2023 22:07:23 +0200
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > If the MSYS2 project wants to maintain Clang patches, that's fine with
> > me.  But then these issues are best posted to the MSYS2 forum, not
> > here.
> 
> Why? Here is the people that hack and build Emacs on a regular basis,
> right?
> 
> Please note that the problem on the OP is not related to the patches.

It is related to a configuration that isn't officially supported.

Re: clang fails to compile emacs in Microsoft Windows

[Eli Zaretskii]

> From: Biswapriyo Nath <nathbappai@gmail.com>
> Date: Sat, 13 May 2023 01:40:11 +0530
> Cc: Óscar Fuentes <ofv@wanadoo.es>, eliz@gnu.org
> 
> Adding the wait() declaration in nt/inc/sys/wait.h fixes the issue.

That's not the right solution for this issue, though, IMO.  Adding
that declaration is only TRT if you also add some implementation of
'wait' to one of the w32*.c files.

The problem here is that the configure script rejects the sys/wait.h
header we provide for MS-Windows because it deduces it isn't
Posix-compliant enough.  That is the problem to fix: convince the
configure script that our sys/wait.h is fine (which it is, since it
does everything the MS-Windows build needs from it).  The right means
for doing so is the nt/mingw-cfg.site file, where you need to add

  ac_cv_header_sys_wait_h=yes

This will cause this test to succeed unconditionally, which is what we
need.

Re: clang fails to compile emacs in Microsoft Windows

[Biswapriyo Nath]

Thank you for the suggestion. I should keep that patch in downstream
repository then.

Just for the record.
* The actual issue was reported here
https://github.com/msys2/MINGW-packages/issues/17117
* The patch may be added here
https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-emacs

Re: clang fails to compile emacs in Microsoft Windows

[Óscar Fuentes]

Eli Zaretskii <eliz@gnu.org> writes:

>> > If the MSYS2 project wants to maintain Clang patches, that's fine with
>> > me.  But then these issues are best posted to the MSYS2 forum, not
>> > here.
>> 
>> Why? Here is the people that hack and build Emacs on a regular basis,
>> right?
>> 
>> Please note that the problem on the OP is not related to the patches.
>
> It is related to a configuration that isn't officially supported.

Last time I looked the configure script had references to ancient
propietary compilers and systems that are not in use for decades. But
adding a trivial fix for a Free toolset on a Free software distribution
for the most widely used OS that *also improves support for gcc by
eliminating a warning* is not acceptable.

Re: clang fails to compile emacs in Microsoft Windows

[Eli Zaretskii]

> From: Óscar Fuentes <ofv@wanadoo.es>
> Date: Sat, 13 May 2023 14:39:16 +0200
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> > If the MSYS2 project wants to maintain Clang patches, that's fine with
> >> > me.  But then these issues are best posted to the MSYS2 forum, not
> >> > here.
> >> 
> >> Why? Here is the people that hack and build Emacs on a regular basis,
> >> right?
> >> 
> >> Please note that the problem on the OP is not related to the patches.
> >
> > It is related to a configuration that isn't officially supported.
> 
> Last time I looked the configure script had references to ancient
> propietary compilers and systems that are not in use for decades.

Which ones?

> But adding a trivial fix for a Free toolset on a Free software
> distribution for the most widely used OS that *also improves support
> for gcc by eliminating a warning* is not acceptable.

You have completely misunderstood and misinterpreted what I wrote.