public inbox for gcc-prs@sourceware.org
help / color / mirror / Atom feed
From: Glen Nakamura <glen@imodulo.com>
To: nobody@gcc.gnu.org
Cc: gcc-prs@gcc.gnu.org,
Subject: Re: optimization/6703: optimized memset fills incorrect number of bytes for void** parameter
Date: Mon, 20 May 2002 02:56:00 -0000	[thread overview]
Message-ID: <20020520095604.10414.qmail@sources.redhat.com> (raw)

The following reply was made to PR optimization/6703; it has been noted by GNATS.

From: Glen Nakamura <glen@imodulo.com>
To: gcc-gnats@gcc.gnu.org, gcc-prs@gcc.gnu.org,
	dbaron@fas.harvard.edu, gcc-bugs@gcc.gnu.org, nobody@gcc.gnu.org
Cc:  
Subject: Re: optimization/6703: optimized memset fills incorrect number of
 bytes for void** parameter
Date: Sun, 19 May 2002 23:48:23 -1000

 http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=6703
 This patch extracted from cvs diff -r 1.386 -r 1.387 seems to fix the problem:
 
 ChangeLog
 ===================================================================
 Tue Apr 23 14:24:25 CEST 2002  Jan Hubicka  <jh@suse.cz>
 
 	* i386.c (ix86_expand_clrstr): Fix typo.
 
 Index: i386.c
 ===================================================================
 RCS file: /cvsroot/gcc/gcc/gcc/config/i386/i386.c,v
 retrieving revision 1.368.2.9
 diff -c -3 -p -r1.368.2.9 i386.c
 *** i386.c	23 Apr 2002 08:11:22 -0000	1.368.2.9
 --- i386.c	20 May 2002 09:30:28 -0000
 *************** ix86_expand_clrstr (src, count_exp, alig
 *** 9420,9426 ****
    		 
 	 gen_rtx_SUBREG (SImode, zeroreg, 0)));
          if (TARGET_64BIT && (align <= 4 || count == 0))
    	{
 ! 	  rtx label = ix86_expand_aligntest (destreg, 2);
    	  emit_insn (gen_strsetsi (destreg,
    		 
 	   gen_rtx_SUBREG (SImode, zeroreg, 0)));
    	  emit_label (label);
 --- 9420,9426 ----
    		 
 	 gen_rtx_SUBREG (SImode, zeroreg, 0)));
          if (TARGET_64BIT && (align <= 4 || count == 0))
    	{
 ! 	  rtx label = ix86_expand_aligntest (countreg, 2);
    	  emit_insn (gen_strsetsi (destreg,
    		 
 	   gen_rtx_SUBREG (SImode, zeroreg, 0)));
    	  emit_label (label);
 *************** ix86_expand_clrstr (src, count_exp, alig
 *** 9431,9437 ****
    		 
 	 gen_rtx_SUBREG (HImode, zeroreg, 0)));
          if (align <= 2 || count == 0)
    	{
 ! 	  rtx label = ix86_expand_aligntest (destreg, 2);
    	  emit_insn (gen_strsethi (destreg,
    		 
 	   gen_rtx_SUBREG (HImode, zeroreg, 0)));
    	  emit_label (label);
 --- 9431,9437 ----
    		 
 	 gen_rtx_SUBREG (HImode, zeroreg, 0)));
          if (align <= 2 || count == 0)
    	{
 ! 	  rtx label = ix86_expand_aligntest (countreg, 2);
    	  emit_insn (gen_strsethi (destreg,
    		 
 	   gen_rtx_SUBREG (HImode, zeroreg, 0)));
    	  emit_label (label);
 *************** ix86_expand_clrstr (src, count_exp, alig
 *** 9442,9448 ****
    		 
 	 gen_rtx_SUBREG (QImode, zeroreg, 0)));
          if (align <= 1 || count == 0)
    	{
 ! 	  rtx label = ix86_expand_aligntest (destreg, 1);
    	  emit_insn (gen_strsetqi (destreg,
    		 
 	   gen_rtx_SUBREG (QImode, zeroreg, 0)));
    	  emit_label (label);
 --- 9442,9448 ----
    		 
 	 gen_rtx_SUBREG (QImode, zeroreg, 0)));
          if (align <= 1 || count == 0)
    	{
 ! 	  rtx label = ix86_expand_aligntest (countreg, 1);
    	  emit_insn (gen_strsetqi (destreg,
    		 
 	   gen_rtx_SUBREG (QImode, zeroreg, 0)));
    	  emit_label (label);
 


             reply	other threads:[~2002-05-20  9:56 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-20  2:56 Glen Nakamura [this message]
  -- strict thread matches above, loose matches on Subject: below --
2002-05-24 10:16 sayle
2002-05-20  3:06 Glen Nakamura
2002-05-17 10:56 dbaron

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20020520095604.10414.qmail@sources.redhat.com \
    --to=glen@imodulo.com \
    --cc=gcc-prs@gcc.gnu.org \
    --cc=nobody@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).