This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch ping (testsuite, config, middle-end/driver; some new some old)
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: rth at redhat dot com
- Cc: hans-peter dot nilsson at axis dot com, gcc-patches at gcc dot gnu dot org
- Date: Tue, 27 Dec 2005 12:59:59 +0100
- Subject: Re: Patch ping (testsuite, config, middle-end/driver; some new some old)
> Date: Fri, 23 Dec 2005 09:19:00 -0800
> From: Janis Johnson <janis187@us.ibm.com>
> On Fri, Dec 23, 2005 at 04:59:48AM +0100, Hans-Peter Nilsson wrote:
> > > > [RFA:] Fix Wpadded.c for default_packed targets (CRIS).
> > > > <URL:http://gcc.gnu.org/ml/gcc-patches/2005-10/msg01377.html>
> > > > (this is part middle-end, part testsuite)
> > No takers?
> The testsuite change is fine,
Thanks, both to you and Geoff!
> but it sounds as if what you really
> need is an answer to this question:
>
> > The 1.2 change, adding -fpack-struct=8, that RTH made, seems
> > like it was intended to provoke this warning for targets that
> > don't pad. But it doesn't work: -fpack-struct=N doesn't do
> > that; we'd need to change a minimum_field_alignment not
> > maximum_field_alignment. Or I'm missing something.
Yep, I should've presented a better request for clarification.
RTH?
As the patch is approved, I'll commit it in a week if no news.
brgds, H-P