Re: Ping: [PATCH] testsuite: Fix struct size check [PR116155]

2024-08-13 Thread Qing Zhao
> On Aug 13, 2024, at 17:05, Dimitar Dimitrov wrote: > > On Tue, Aug 13, 2024 at 07:34:09PM +0200, Hans-Peter Nilsson wrote: >>> From: Sam James >>> Date: Tue, 13 Aug 2024 18:17:29 +0100 >> >>> Hans-Peter Nilsson writes: >>> I stumbled on this being a regression for cris-elf as well;

Re: Ping: [PATCH] testsuite: Fix struct size check [PR116155]

2024-08-13 Thread Dimitar Dimitrov
On Tue, Aug 13, 2024 at 07:34:09PM +0200, Hans-Peter Nilsson wrote: > > From: Sam James > > Date: Tue, 13 Aug 2024 18:17:29 +0100 > > > Hans-Peter Nilsson writes: > > > > > I stumbled on this being a regression for cris-elf as well; > > > the patch expectedly fixes the test-case for CRIS as wel

Re: Ping: [PATCH] testsuite: Fix struct size check [PR116155]

2024-08-13 Thread Sam James
Hans-Peter Nilsson writes: >> From: Sam James >> Date: Tue, 13 Aug 2024 18:17:29 +0100 > >> Hans-Peter Nilsson writes: >> >> > I stumbled on this being a regression for cris-elf as well; >> > the patch expectedly fixes the test-case for CRIS as well. >> > It's been a week since the patch was p

Re: Ping: [PATCH] testsuite: Fix struct size check [PR116155]

2024-08-13 Thread Hans-Peter Nilsson
> From: Sam James > Date: Tue, 13 Aug 2024 18:17:29 +0100 > Hans-Peter Nilsson writes: > > > I stumbled on this being a regression for cris-elf as well; > > the patch expectedly fixes the test-case for CRIS as well. > > It's been a week since the patch was posted and as I see no > > replies, I'

Re: Ping: [PATCH] testsuite: Fix struct size check [PR116155]

2024-08-13 Thread Sam James
Hans-Peter Nilsson writes: > I stumbled on this being a regression for cris-elf as well; > the patch expectedly fixes the test-case for CRIS as well. > It's been a week since the patch was posted and as I see no > replies, I'm pinging this in behalf of Dimitar. I can't formally approve it but I

Ping: [PATCH] testsuite: Fix struct size check [PR116155]

2024-08-13 Thread Hans-Peter Nilsson
I stumbled on this being a regression for cris-elf as well; the patch expectedly fixes the test-case for CRIS as well. It's been a week since the patch was posted and as I see no replies, I'm pinging this in behalf of Dimitar. > From: Dimitar Dimitrov > Date: Mon, 5 Aug 2024 21:29:35 +0300 > Th

[PATCH] testsuite: Fix struct size check [PR116155]

2024-08-05 Thread Dimitar Dimitrov
The size of "struct only_fam_2" is dependent on the alignment of the flexible array member "b", and not on the type of the preceding bit-fields. For most targets the two are equal. But on default_packed targets like pru-unknown-elf, the alignment of int is not equal to the size of int, so the tes