Work around IA64 tst-setcontext2.c compile failure

Work around IA64 tst-setcontext2.c compile failure

[Joseph Myers]

Building tests for IA64 runs into a build failure compiling
stdlib/tst-setcontext2.c:

tst-setcontext2.c: In function 'do_test':
tst-setcontext2.c:210:20: error: passing argument 1 of 'sigismember' from incompatible pointer type [-Werror=incompatible-pointer-types]
   if (sigismember (&oldctx.uc_sigmask, SIGUSR2) != 1)
                    ^

Indeed, the IA64 uc_sigmask as unsigned long rather than the larger
userspace sigset_t.  Fixing this might be hard; this patch works
around the build failure by making IA64 wrap the test with a version
that #defines sigismember to add a cast (I'd welcome a better approach
for fixing this).

Tested (compilation only) for ia64.

2016-11-09  Joseph Myers  <joseph@codesourcery.com>

	* sysdeps/unix/sysv/linux/ia64/tst-setcontext2.c: New file.

diff --git a/sysdeps/unix/sysv/linux/ia64/tst-setcontext2.c b/sysdeps/unix/sysv/linux/ia64/tst-setcontext2.c
new file mode 100644
index 0000000..7adc059
--- /dev/null
+++ b/sysdeps/unix/sysv/linux/ia64/tst-setcontext2.c
@@ -0,0 +1,24 @@
+/* Work around incorrect type of IA64 uc_sigmask.
+   Copyright (C) 2016 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <http://www.gnu.org/licenses/>.  */
+
+#include <signal.h>
+
+#undef sigismember
+#define sigismember(set, signo) sigismember ((const sigset_t *) (set), (signo))
+
+#include <stdlib/tst-setcontext2.c>

-- 
Joseph S. Myers
joseph@codesourcery.com

Ping Re: Work around IA64 tst-setcontext2.c compile failure

[Joseph Myers]

Ping.  This patch 
<https://sourceware.org/ml/libc-alpha/2016-11/msg00314.html> is pending 
review.

-- 
Joseph S. Myers
joseph@codesourcery.com

Ping^2 Re: Work around IA64 tst-setcontext2.c compile failure

[Joseph Myers]

Ping^2.  This patch 
<https://sourceware.org/ml/libc-alpha/2016-11/msg00314.html> is still 
pending review.

-- 
Joseph S. Myers
joseph@codesourcery.com

Ping^3 Re: Work around IA64 tst-setcontext2.c compile failure

[Joseph Myers]

Ping^3.  This patch 
<https://sourceware.org/ml/libc-alpha/2016-11/msg00314.html> is still 
pending review.

-- 
Joseph S. Myers
joseph@codesourcery.com

Re: Work around IA64 tst-setcontext2.c compile failure

[Florian Weimer]

On 11/09/2016 03:05 AM, Joseph Myers wrote:
> Tested (compilation only) for ia64.
>
> 2016-11-09  Joseph Myers  <joseph@codesourcery.com>
>
> 	* sysdeps/unix/sysv/linux/ia64/tst-setcontext2.c: New file.

Please feel encouraged to commit this.

Honestly, I don't really know what to do with these old architectures.

Thanks,
Florian