[Bug target/30770] [4.3 regression] BOOT_CFLAGS=-O2 -g -mtune=nocona miscompiled the stage 3 compiler

2007-02-25 Thread ubizjak at gmail dot com 


--- Comment #8 from ubizjak at gmail dot com  2007-02-26 07:05 ---
Fixed in SVN.


-- 

ubizjak at gmail dot com changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30770

[Bug target/30770] [4.3 regression] BOOT_CFLAGS=-O2 -g -mtune=nocona miscompiled the stage 3 compiler

2007-02-24 Thread ubizjak at gmail dot com 


--- Comment #5 from ubizjak at gmail dot com  2007-02-24 19:09 ---
(In reply to comment #2)
 I have verified that revision 119252:
 
 http://gcc.gnu.org/ml/gcc-cvs/2006-11/msg00907.html
 breaks -mtune=nocona. Jan, can you take a look? Thanks.

Something is still wrong with unrolled_loop stringop handling.

Patch at http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01937.html and the
following patch that disables unrolled loop stringops (for nocona) bootstraps
OK for -mtune=nocona. This applies to -march=athlon64 as well.

Index: i386.c
===
--- i386.c  (revision 122286)
+++ i386.c  (working copy)
@@ -789,10 +789,10 @@
   COSTS_N_INSNS (44),  /* cost of FSQRT instruction.  */
   {{libcall, {{12, loop_1_byte}, {-1, rep_prefix_4_byte}}},
{libcall, {{32, loop}, {2, rep_prefix_8_byte},
- {10, unrolled_loop}, {-1, libcall,
+ /*{10, unrolled_loop},*/ {-1, libcall,
   {{libcall, {{6, loop_1_byte}, {48, loop}, {20480, rep_prefix_4_byte},
{-1, libcall}}},
-   {libcall, {{24, loop}, {64, unrolled_loop},
+   {libcall, {{24, loop}, /*{64, unrolled_loop},*/
  {8192, rep_prefix_8_byte}, {-1, libcall
 };



-- 

ubizjak at gmail dot com changed:

   What|Removed |Added

 CC||ubizjak at gmail dot com
 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2007-02-24 19:09:51
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30770

[Bug target/30770] [4.3 regression] BOOT_CFLAGS=-O2 -g -mtune=nocona miscompiled the stage 3 compiler

2007-02-24 Thread ubizjak at gmail dot com 


--- Comment #6 from ubizjak at gmail dot com  2007-02-24 22:54 ---
It was a typo in expand_movmem_epilogue() and expand_setmem_epilogue().
Following patch, fixes this bug and together with patch for PR target/30778
(http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01937.html) enables bootstrap for
all x86_64 targets.

2007-02-24  Uros Bizjak  [EMAIL PROTECTED]

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

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)
{


-- 

ubizjak at gmail dot com changed:

   What|Removed |Added

  BugsThisDependsOn||30778
 AssignedTo|unassigned at gcc dot gnu   |ubizjak at gmail dot com
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2007-02-24 19:09:51 |2007-02-24 22:54:35
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30770

[Bug target/30770] [4.3 regression] BOOT_CFLAGS=-O2 -g -mtune=nocona miscompiled the stage 3 compiler

2007-02-24 Thread uros at gcc dot gnu dot org 


--- Comment #7 from uros at gcc dot gnu dot org  2007-02-24 23:29 ---
Subject: Bug 30770

Author: uros
Date: Sat Feb 24 23:29:30 2007
New Revision: 122301

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=122301
Log:
PR target/30770
* config/i386/i386.md (expand_movmem_epilogue): Fix typo, mask
count argument with 0x10, not with 0x16.
(expand_setmem_epilogue): Ditto.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/i386.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30770

[Bug target/30770] [4.3 regression] BOOT_CFLAGS=-O2 -g -mtune=nocona miscompiled the stage 3 compiler

2007-02-23 Thread h dot mth at web dot de 


--- Comment #4 from h dot mth at web dot de  2007-02-23 23:29 ---
JFYI, '-O2 -march=athlon64' also fails.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30770

[Bug target/30770] [4.3 regression] BOOT_CFLAGS=-O2 -g -mtune=nocona miscompiled the stage 3 compiler

2007-02-15 Thread pinskia at gcc dot gnu dot org 


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

Summary|[4.3 regression]:   |[4.3 regression]
   |BOOT_CFLAGS=-O2 -g -   |BOOT_CFLAGS=-O2 -g -
   |mtune=nocona miscompiled   |mtune=nocona miscompiled
   |the stage 3 compiler|the stage 3 compiler
   Target Milestone|--- |4.3.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30770

[Bug target/30770] [4.3 regression]: BOOT_CFLAGS=-O2 -g -mtune=nocona miscompiled thes stage 3 compiler

2007-02-12 Thread belyshev at depni dot sinp dot msu dot ru 


--- Comment #3 from belyshev at depni dot sinp dot msu dot ru  2007-02-12 
21:19 ---
I believe this is the same bug as pr 30778.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30770