This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] movmem/setmem alignment code
- From: Jakub Jelinek <jakub at redhat dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Raksit Ashok <raksit at google dot com>, Ian Lance Taylor <iant at google dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, jh at suse dot cz
- Date: Fri, 2 Jan 2009 20:33:40 +0100
- Subject: Re: [PATCH] movmem/setmem alignment code
- References: <9718380a0807062335y20c7d182o2483bc774527e95d@mail.gmail.com> <m3d4low7fr.fsf@google.com> <9718380a0807081521h27de6250re26dedfcebedf1d5@mail.gmail.com> <6dc9ffc80901021101n1149d9cdu5b754962f603f5fa@mail.gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Jan 02, 2009 at 11:01:55AM -0800, H.J. Lu wrote:
> On Tue, Jul 8, 2008 at 2:21 PM, Raksit Ashok <raksit@google.com> wrote:
> > This was committed as revision 137646.
> > On Tue, Jul 8, 2008 at 11:35 AM, Ian Lance Taylor <iant@google.com> wrote:
> >> "Raksit Ashok" <raksit@google.com> writes:
> >>
> >>> 2008-07-07 Raksit Ashok <raksit@google.com>
> >>>
> >>> * expr.c (emit_block_move_via_movmem): Fix expected_align parameter.
> >>> (set_storage_via_setmem): Fix expected_align parameter.
> >>> * doc/invoke.texi (i386 and x86-64 Options): Add a couple of options
> >>> missing from the list.
> >>> * doc/md.texi (movmem): Explicitly state that expected alignment is
> >>> to be expressed in bytes.
> >>> (setmem): Explicitly state that expected alignment is to be expressed
> >>> in bytes.
> >>
> >> This is OK.
> It caused:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38708
That change is correct, so the bug is elsewhere.
Jakub