Re: [PATCH v2] lib/posixtm.c: Fixe build error with clang-18

2024-01-16 Thread Khem Raj
On Tue, Jan 16, 2024 at 5:25 PM Paul Eggert wrote: > > Thanks, I installed the attached simpler (1-byte!) fix. Please give it a > try, as I lack easy access to clang 18. Ah cool!. This works for me too.

[PATCH v2] lib/posixtm.c: Fixe build error with clang-18

2024-01-16 Thread Khem Raj
e ckd_add(R, A, B) __builtin_add_overflow((A), (B), (R)) | ^~~ 1 error generated. * lib/posixtm.c (posixtime): Fix operand argument to checked integer operation. Signed-off-by: Khem Raj --- ChangeLog | 5 + lib/posixtm.c | 2 +- 2 files changed, 6

[PATCH] lib/posixtm.c: Fixe build error with clang-18

2024-01-15 Thread Khem Raj
* lib/posixtm.c (posixtime): Fix operand argument to checked integer operation. Signed-off-by: Khem Raj --- ChangeLog | 5 + lib/posixtm.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index e42d2c31ab..daca27a917 100644 --- a/ChangeLog

Re: [PATCH v2] Define alignof_slot using _Alignof when using C11 or newer

2023-01-15 Thread Khem Raj
On Sun, Jan 15, 2023 at 11:52 AM Paul Eggert wrote: > On 2023-01-14 17:51, Paul Eggert wrote: > > > We already have two copies of that stuff elsewhere (in lib/stdalign.in.h > > and m4/stdalign.m4), for reasons that currently escape me. > > To try to address that in an upward-compatible way, I ins

Re: [PATCH] Define alignof_slot using _Alignof when using C11 or newer

2023-01-14 Thread Khem Raj
On Sat, Jan 14, 2023 at 4:17 PM Bruno Haible wrote: > > Khem Raj wrote: > > [2] https://reviews.llvm.org/D133574 > > Thanks for the heads-up. > > > WG14 N2350 made very clear that it is an UB having type definitions > > within "offsetof" [1]. > >

[PATCH v2] Define alignof_slot using _Alignof when using C11 or newer

2023-01-14 Thread Khem Raj
_bug.cgi?id=52023 Signed-off-by: Khem Raj --- v1->v2: - Exclude gcc < 4.9 and clang < 8 even though they may have C11 support ChangeLog | 5 + lib/alignof.h | 10 +- 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index fb467a3c14..3

[PATCH] Define alignof_slot using _Alignof when using C11 or newer

2023-01-14 Thread Khem Raj
2] Fixes build when using -std >= gnu11 and using clang16+ [1] https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2350.htm [2] https://reviews.llvm.org/D133574 Signed-off-by: Khem Raj --- ChangeLog | 5 + lib/alignof.h | 2 ++ 2 files changed, 7 insertions(+) diff --git a/ChangeLog b/Cha

Re: [PATCH] sigsegv: Fix build on ppc/musl

2022-03-13 Thread Khem Raj
On Sun, Mar 13, 2022 at 9:14 AM Sam wrote: > > > > > On 13 Mar 2022, at 14:17, Bruno Haible wrote: > > > > Eric Blake wrote: > >> On Wed, Mar 09, 2022 at 11:37:14PM -0800, Khem Raj wrote: > >>> mcontext is not a standard layout so glibc and mus