Uros Bizjak - [PATCH, x86_64]: Fix PR target/30770 (original) (raw)

This is the mail archive of the gcc-patches@gcc.gnu.orgmailing list for the GCC project.

• From: Uros Bizjak
• To: GCC Patches
• Cc: Jan Hubicka
• Date: Sat, 24 Feb 2007 23:57:11 +0100
• Subject: [PATCH, x86_64]: Fix PR target/30770
• Dkim-signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:user-agent:mime-version:to:cc:subject:content-type; b=CcwD4166lajdAumpI8Jk2LeCGgBJJ6lnnJYEvCqMzNtat4FSRsDGe7+9EjZQb+HWX2f0nHubfzOp7ZvdIm2jB2JvXxg6Z/Fiy/VcGMPixBuk7TID7yChdoyuMFuvdsV1VyAxt6Wz8c5XwVzkvXUZM850ZKLhRglf2uahGjTHzY0=

Hello!This patch fixes PR target/30770, bootstrap failure with BOOT_CFLAGS="-O2 -mtune=nocona".
There was a typo in expand_movmem_epilogue() and expand_setmem_epilogue(). The ChangeLog says everything...2007-02-24 Uros Bizjak ubizjak@gmail.com

   PR target/30770
   * config/i386/i386.md (expand_movmem_epilogue): Fix typo, mask
   count argument with 0x10, not with 0x16.
   (expand_setmem_epilogue): Ditto.

This patch, together with patch for PR target/30778 (http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01937.html) fixes bootstrap for all x86_64 targets.Patch was bootstrapped on x86_64 and regression tested for all default languages. Additionally, gcc was bootstrapped and regression tested (c only) with BOOT_CFLAGS="-O2 -mtune=nocona".Although this patch is trivial and obvious, etc..., I'll commit it (once approved) together with PR target/30778, as these two patches were regtested together. Also, both are needed to fix bootstrap.Uros.

Index: i386.c

--- i386.c (revision 122286) +++ i386.c (working copy) @@ -13522,7 +13549,7 @@ HOST_WIDE_INT countval = INTVAL (count); int offset = 0;

•  if ((countval & 0x16) && max_size > 16)

•  if ((countval & 0x10) && max_size > 16)

  { if (TARGET_64BIT) {

@@ -13675,7 +13702,7 @@ HOST_WIDE_INT countval = INTVAL (count); int offset = 0;

•  if ((countval & 0x16) && max_size > 16)

•  if ((countval & 0x10) && max_size > 16)

  { if (TARGET_64BIT) {