Re: [PATCH] Use more ARRAY_SIZE.

2022-05-23 Thread Martin Liška
On 5/23/22 09:56, Iain Buclaw wrote: > Excerpts from Martin Liška's message of Mai 11, 2022 10:17 am: >> On 5/9/22 14:03, Richard Biener wrote: >>> On Thu, May 5, 2022 at 4:30 PM Martin Liška wrote: On 5/5/22 14:58, Iain Buclaw wrote: > This D front-end change doesn't look right to m

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-23 Thread Iain Buclaw via Gcc-patches
Excerpts from Martin Liška's message of Mai 11, 2022 10:17 am: > On 5/9/22 14:03, Richard Biener wrote: >> On Thu, May 5, 2022 at 4:30 PM Martin Liška wrote: >>> >>> On 5/5/22 14:58, Iain Buclaw wrote: This D front-end change doesn't look right to me, besides the slight >>> >>> Hello. >>> >>>

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-16 Thread Martin Liška
On 5/11/22 10:17, Martin Liška wrote: > On 5/9/22 14:03, Richard Biener wrote: >> On Thu, May 5, 2022 at 4:30 PM Martin Liška wrote: >>> >>> On 5/5/22 14:58, Iain Buclaw wrote: This D front-end change doesn't look right to me, besides the slight >>> >>> Hello. >>> >>> Sorry, I've re-read the

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-12 Thread Mikael Morin
Le 11/05/2022 à 10:17, Martin Liška a écrit : On 5/9/22 14:03, Richard Biener wrote: On Thu, May 5, 2022 at 4:30 PM Martin Liška wrote: On 5/5/22 14:58, Iain Buclaw wrote: This D front-end change doesn't look right to me, besides the slight Hello. Sorry, I've re-read the patch and fixed s

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-12 Thread Richard Biener via Gcc-patches
On Wed, May 11, 2022 at 10:01 PM David Malcolm via Gcc-patches wrote: > > On Wed, 2022-05-11 at 15:04 -0400, Eric Gallager wrote: > > On Wed, May 11, 2022 at 9:53 AM David Malcolm via Gcc-patches > > wrote: > > > > > > On Thu, 2022-05-05 at 14:16 +0200, Martin Liška wrote: > > > > Patch can boots

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-11 Thread David Malcolm via Gcc-patches
On Wed, 2022-05-11 at 15:04 -0400, Eric Gallager wrote: > On Wed, May 11, 2022 at 9:53 AM David Malcolm via Gcc-patches > wrote: > > > > On Thu, 2022-05-05 at 14:16 +0200, Martin Liška wrote: > > > Patch can bootstrap on x86_64-linux-gnu and survives regression > > > tests. > > > > > > Ready to

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-11 Thread Eric Gallager via Gcc-patches
On Wed, May 11, 2022 at 9:53 AM David Malcolm via Gcc-patches wrote: > > On Thu, 2022-05-05 at 14:16 +0200, Martin Liška wrote: > > Patch can bootstrap on x86_64-linux-gnu and survives regression > > tests. > > > > Ready to be installed? > > I looked over the changes to: > > gcc/analyzer/*.cc >

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-11 Thread David Malcolm via Gcc-patches
On Thu, 2022-05-05 at 14:16 +0200, Martin Liška wrote: > Patch can bootstrap on x86_64-linux-gnu and survives regression > tests. > > Ready to be installed? I looked over the changes to: gcc/analyzer/*.cc gcc/input.cc gcc/jit/*.cc and they look good to me. I assume that you enabled "jit"

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-11 Thread Martin Liška
On 5/9/22 14:03, Richard Biener wrote: > On Thu, May 5, 2022 at 4:30 PM Martin Liška wrote: >> >> On 5/5/22 14:58, Iain Buclaw wrote: >>> This D front-end change doesn't look right to me, besides the slight >> >> Hello. >> >> Sorry, I've re-read the patch and fixed some places where the macro usag

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-09 Thread Richard Biener via Gcc-patches
On Thu, May 5, 2022 at 4:30 PM Martin Liška wrote: > > On 5/5/22 14:58, Iain Buclaw wrote: > > This D front-end change doesn't look right to me, besides the slight > > Hello. > > Sorry, I've re-read the patch and fixed some places where the macro usage > was wrong. > > Patch can bootstrap on x86_6

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-09 Thread Arnaud Charlet via Gcc-patches
> > So I don't see how your patch can work, can you clarify? > > Sorry for that, fixed in the updated version. Assuming you're getting a successful Ada build, the Ada part is OK. > From 5ad8fe059d3419446647eadf8785c768b647d15b Mon Sep 17 00:00:00 2001 > From: Martin Liska > Date: Thu, 13 Jan 20

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-09 Thread Martin Liška
On 5/9/22 12:21, Arnaud Charlet wrote: > So I don't see how your patch can work, can you clarify? Sorry for that, fixed in the updated version. Cheers, MartinFrom 5ad8fe059d3419446647eadf8785c768b647d15b Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Thu, 13 Jan 2022 18:46:26 +0100 Subject: [

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-09 Thread Arnaud Charlet via Gcc-patches
> Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > gcc/ada/ChangeLog: > > * locales.c (iso_639_1_to_639_3): Use ARRAY_SIZE. > (language_name_to_639_3): Likewise. > (country_name_to_3166): Likewise. Can you

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-05 Thread Martin Liška
On 5/5/22 14:58, Iain Buclaw wrote: > This D front-end change doesn't look right to me, besides the slight Hello. Sorry, I've re-read the patch and fixed some places where the macro usage was wrong. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. MartinFrom 8d9630e411321c

Re: [PATCH] Use more ARRAY_SIZE.

2022-05-05 Thread Iain Buclaw via Gcc-patches
Excerpts from Martin Liška's message of Mai 5, 2022 2:16 pm: > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > gcc/d/ChangeLog: > > * longdouble.h: Use ARRAY_SIZE. > > diff --git a/gcc/d/longdouble.h b/gcc/d/longdoub