[Bug ada/33454] [4.3 Regression] Wrong system.ads for --with-cpu=default32

[bje at gcc dot gnu dot org]

--- Comment #4 from bje at gcc dot gnu dot org  2007-10-08 06:47 ---
This patch created more problems than it solved.  Reverting for the time being.


-- 

bje at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


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

[Bug ada/33454] [4.3 Regression] Wrong system.ads for --with-cpu=default32

[bje at gcc dot gnu dot org]

--- Comment #3 from bje at gcc dot gnu dot org  2007-10-08 06:46 ---
Subject: Bug 33454

Author: bje
Date: Mon Oct  8 06:45:56 2007
New Revision: 129121

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129121
Log:
PR ada/33454
Revert:
2007-08-31  Ben Elliston  <[EMAIL PROTECTED]>

* Makefile.in (LIBGNAT_TARGET_PAIRS): Use system-linux-ppc64.ads
when compiling for powerpc64-*-linux.
* system-linux-ppc64.ads: New file.

Removed:
trunk/gcc/ada/system-linux-ppc64.ads
Modified:
trunk/gcc/ada/ChangeLog
trunk/gcc/ada/Makefile.in


-- 


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

[Bug rtl-optimization/33669] [4.3 Regression] Revision 128957 miscompiles 481.wrf

[zadeck at naturalbridge dot com]

--- Comment #11 from zadeck at naturalbridge dot com  2007-10-08 03:53 
---
*** Bug 33662 has been marked as a duplicate of this bug. ***


-- 

zadeck at naturalbridge dot com changed:

   What|Removed |Added

 CC||kkojima at gcc dot gnu dot
   ||org


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

[Bug middle-end/33662] [4.3 Regression] Wrong register allocation on SH

[zadeck at naturalbridge dot com]

--- Comment #2 from zadeck at naturalbridge dot com  2007-10-08 03:53 
---


*** This bug has been marked as a duplicate of 33669 ***


-- 

zadeck at naturalbridge dot com changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||DUPLICATE


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

[Bug middle-end/33662] [4.3 Regression] Wrong register allocation on SH

[kkojima at gcc dot gnu dot org]

--- Comment #1 from kkojima at gcc dot gnu dot org  2007-10-08 02:22 ---
I've confirmed that the patch in the trail of pr33669 fixes this too.


-- 


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

[Bug target/33685] [4.3 regression] invalid rtl sharing found in the insn

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||pinskia at gcc dot gnu dot
   ||org
  Component|rtl-optimization|target
   Target Milestone|--- |4.3.0


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

[Bug rtl-optimization/33669] [4.3 Regression] Revision 128957 miscompiles 481.wrf

[zadeck at naturalbridge dot com]

--- Comment #10 from zadeck at naturalbridge dot com  2007-10-07 21:57 
---
Subject: Re:  [4.3 Regression]  Revision 128957
 miscompiles 481.wrf

This patch fixes pr33669.

The failure only happens if you have a block with 2 or more uses of a
multiword pseudo register that is local to this block and has been
allocated by local_alloc.  The uses must be in a particular form: the
last use must be a subreg use that only used some of the hard registers and
a previous non subreg use of the multiword register.

When all of this happens, the code did not properly expand this to a
whole multiregister when the second to last use is encountered in the
backwards scan.

I.e. a lot of things have to happen to get this to fail.

I have tested this patch on ia-64, x86-{64,32} and ppc-32.

Ok for commit?

Kenny

2007-10-07  Kenneth Zadeck <[EMAIL PROTECTED]>

PR middle-end/33669
* ra-conflict.c (record_one_conflict_between_regnos,
set_conflicts_for_earlyclobber, global_conflicts): Improved logging.
(global_conflicts): Removed incorrect check.

2007-10-07  Kenneth Zadeck <[EMAIL PROTECTED]>

PR middle-end/33669
* gcc.c-torture/execute/pr33669.c: New.


Index: ra-conflict.c
===
--- ra-conflict.c   (revision 129053)
+++ ra-conflict.c   (working copy)
@@ -196,7 +196,7 @@ record_one_conflict_between_regnos (enum
   int allocno2 = reg_allocno[r2];

   if (dump_file)
-fprintf (dump_file, "  rocbr adding %d<=>%d\n", r1, r2);
+fprintf (dump_file, "rocbr adding %d<=>%d\n", r1, r2);

   if (allocno1 >= 0 && allocno2 >= 0)
 set_conflict (allocno1, allocno2);
@@ -401,9 +401,6 @@ set_conflicts_for_earlyclobber (rtx insn
recog_data.operand[use +
1]);
}
}
-
-  if (dump_file) 
-fprintf (dump_file, "  finished early clobber conflicts.\n");
 }


@@ -984,7 +981,7 @@ global_conflicts (void)
allocnum, renumber);
}

- else if (!sparseset_bit_p (allocnos_live, allocnum))
+ else
{
  if (dump_file)
fprintf (dump_file, "dying pseudo\n");
@@ -1071,6 +1068,8 @@ global_conflicts (void)
 FIXME: We should consider either adding a new kind of
 clobber, or adding a flag to the clobber distinguish
 these two cases.  */
+ if (dump_file && VEC_length (df_ref_t, clobbers))
+   fprintf (dump_file, "  clobber conflicts\n");
  for (k = VEC_length (df_ref_t, clobbers) - 1; k >= 0; k--)
{
  struct df_ref *def = VEC_index (df_ref_t, clobbers, k);
@@ -1132,6 +1131,8 @@ global_conflicts (void)
  if (GET_CODE (PATTERN (insn)) == PARALLEL && multiple_sets
(insn))
{ 
  int j;
+ if (dump_file)
+   fprintf (dump_file, "  multiple sets\n");
  for (j = VEC_length (df_ref_t, dying_regs) - 1; j >= 0; j--)
{
  int used_in_output = 0;
Index: testsuite/gcc.c-torture/execute/pr33669.c
===
--- testsuite/gcc.c-torture/execute/pr33669.c   (revision 0)
+++ testsuite/gcc.c-torture/execute/pr33669.c   (revision 0)
@@ -0,0 +1,40 @@
+extern void abort (void);
+
+typedef struct foo_t
+{ 
+  unsigned int blksz;
+  unsigned int bf_cnt; 
+} foo_t;
+
+#define _RNDUP(x, unit)  x) + (unit) - 1) / (unit)) * (unit))
+#define _RNDDOWN(x, unit)  ((x) - ((x)%(unit)))
+
+long long
+foo (foo_t *const pxp,  long long offset, unsigned int extent)
+{
+  long long blkoffset = _RNDDOWN(offset, (long long )pxp->blksz);
+  unsigned int diff = (unsigned int)(offset - blkoffset);
+  unsigned int blkextent = _RNDUP(diff + extent, pxp->blksz);
+
+  if (pxp->blksz < blkextent)
+return -1LL;
+
+  if (pxp->bf_cnt > pxp->blksz)
+pxp->bf_cnt = pxp->blksz;
+
+  return blkoffset;
+}
+
+int
+main ()
+{
+  foo_t x;
+  long long xx;
+
+  x.blksz = 8192;
+  x.bf_cnt = 0;
+  xx = foo (&x, 0, 4096);
+  if (xx != 0LL)
+abort ();
+  return 0;
+}


-- 


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

[Bug libfortran/22423] Warnings when building libgfortran

[tobi at gcc dot gnu dot org]

--- Comment #13 from tobi at gcc dot gnu dot org  2007-10-07 21:48 ---
The current set of warnings is:

../../../libgfortran/runtime/backtrace.c:66: warning: 'local_strcasestr'
defined but not used
../../../libgfortran/runtime/environ.c:312: warning: 'init_choice' defined but
not used
../../../libgfortran/runtime/environ.c:339: warning: 'show_choice' defined but
not used
../../../libgfortran/runtime/main.c:179: warning: passing argument 1 of 'free'
discards qualifiers from pointer target type
../../../libgfortran/io/list_read.c:1518: warning: argument 'type' might be
clobbered by 'longjmp' or 'vfork'
../../../libgfortran/intrinsics/pack_generic.c:316: warning: passing argument 3
of 'pack_internal' from incompatible pointer type
../../../libgfortran/intrinsics/pack_generic.c:331: warning: passing argument 3
of 'pack_internal' from incompatible pointer type
../../../libgfortran/intrinsics/spread_generic.c:136: warning: format '%d'
expects type 'int', but argument 2 has type 'long int'
../../../libgfortran/intrinsics/spread_generic.c:147: warning: format '%d'
expects type 'int', but argument 2 has type 'long int'
../../../libgfortran/intrinsics/unpack_generic.c:201: warning: passing argument
3 of 'unpack_internal' from incompatible pointer type
../../../libgfortran/intrinsics/unpack_generic.c:217: warning: passing argument
3 of 'unpack_internal' from incompatible pointer type
../../../libgfortran/intrinsics/unpack_generic.c:233: warning: passing argument
3 of 'unpack_internal' from incompatible pointer type
../../../libgfortran/intrinsics/unpack_generic.c:253: warning: passing argument
3 of 'unpack_internal' from incompatible pointer type
../../../../libgfortran/runtime/backtrace.c:66: warning: 'local_strcasestr'
defined but not used
../../../../libgfortran/runtime/environ.c:312: warning: 'init_choice' defined
but not used
../../../../libgfortran/runtime/environ.c:339: warning: 'show_choice' defined
but not used
../../../../libgfortran/runtime/main.c:179: warning: passing argument 1 of
'free' discards qualifiers from pointer target type
../../../../libgfortran/io/list_read.c:1518: warning: argument 'type' might be
clobbered by 'longjmp' or 'vfork'
../../../../libgfortran/intrinsics/pack_generic.c:316: warning: passing
argument 3 of 'pack_internal' from incompatible pointer type
../../../../libgfortran/intrinsics/pack_generic.c:331: warning: passing
argument 3 of 'pack_internal' from incompatible pointer type
../../../../libgfortran/intrinsics/signal.c:201: warning: cast to pointer from
integer of different size
../../../../libgfortran/intrinsics/signal.c:208: warning: cast to pointer from
integer of different size
../../../../libgfortran/intrinsics/signal.c:229: warning: cast to pointer from
integer of different size
../../../../libgfortran/intrinsics/signal.c:236: warning: cast to pointer from
integer of different size
../../../../libgfortran/intrinsics/spread_generic.c:136: warning: format '%d'
expects type 'int', but argument 2 has type 'index_type'
../../../../libgfortran/intrinsics/spread_generic.c:147: warning: format '%d'
expects type 'int', but argument 2 has type 'index_type'
../../../../libgfortran/intrinsics/unpack_generic.c:201: warning: passing
argument 3 of 'unpack_internal' from incompatible pointer type
../../../../libgfortran/intrinsics/unpack_generic.c:217: warning: passing
argument 3 of 'unpack_internal' from incompatible pointer type
../../../../libgfortran/intrinsics/unpack_generic.c:233: warning: passing
argument 3 of 'unpack_internal' from incompatible pointer type
../../../../libgfortran/intrinsics/unpack_generic.c:253: warning: passing
argument 3 of 'unpack_internal' from incompatible pointer type


-- 


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

[Bug libfortran/33683] calculating lgamma instead of gamma

[tkoenig at gcc dot gnu dot org]

--- Comment #8 from tkoenig at gcc dot gnu dot org  2007-10-07 21:45 ---
Fixed.

Let's close this and watch for occurences of
gamma_5.f90 failing, then open a new bug report
if that happens.


-- 

tkoenig at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


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

[Bug libfortran/33683] calculating lgamma instead of gamma

[tkoenig at gcc dot gnu dot org]

--- Comment #7 from tkoenig at gcc dot gnu dot org  2007-10-07 21:36 ---
Subject: Bug 33683

Author: tkoenig
Date: Sun Oct  7 21:36:09 2007
New Revision: 129116

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129116
Log:
2007-10-07  Thomas Koenig  <[EMAIL PROTECTED]>

PR libfortran/33683
* mathbuiltins.def (GAMMA):  Change function name to
"tgamma" instad of "gamma".

2007-10-07  Thomas Koenig  <[EMAIL PROTECTED]>

PR libfortran/33683
* gfortran.dg/gamma_5.f90:  New test case


Added:
trunk/gcc/testsuite/gfortran.dg/gamma_5.f90
Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/mathbuiltins.def
trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug rtl-optimization/33685] [4.3 regression] invalid rtl sharing found in the insn

[bero at arklinux dot org]

--- Comment #1 from bero at arklinux dot org  2007-10-07 21:28 ---
Created an attachment (id=14317)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14317&action=view)
Preprocessed source, bzip2-ed


-- 


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

[Bug rtl-optimization/33685] New: [4.3 regression] invalid rtl sharing found in the insn

[bero at arklinux dot org]

gcc -DHAVE_CONFIG_H -I. -I.. -I../.. -DMAGIC=\"/usr/lib/rpm/magic\"
-DHAVE_LIBZ=1 -O2 -march=i586 -mtune=i686 -fomit-frame-pointer -fweb
-frename-registers -MT softmagic.lo -MD -MP -MF .deps/softmagic.Tpo -c
softmagic.c  -fPIC -DPIC -o .libs/softmagic.o
softmagic.c: In function 'cvt_16':
softmagic.c:553: error: invalid rtl sharing found in the insn
(insn 70 69 71 12 softmagic.c:552 (parallel [
(set (reg:HI 69)
(udiv:HI (reg:HI 71 [ .h ])
(subreg:HI (reg:SI 79 [ D.4365 ]) 0)))
(set (reg:HI 70)
(umod:HI (reg:HI 71 [ .h ])
(subreg:HI (reg:SI 79 [ D.4365 ]) 0)))
(use (reg:HI 72))
(clobber (reg:CC 17 flags))
]) 231 {*udivmodhi_noext} (nil))
softmagic.c:553: error: shared rtx
(subreg:HI (reg:SI 79 [ D.4365 ]) 0)
softmagic.c:553: internal compiler error: internal consistency failure


Happens at -O1 and higher.


-- 
   Summary: [4.3 regression] invalid rtl sharing found in the insn
   Product: gcc
   Version: 4.3.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: rtl-optimization
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: bero at arklinux dot org
 GCC build triplet: i586-pc-linux-gnu
  GCC host triplet: i586-pc-linux-gnu
GCC target triplet: i586-pc-linux-gnu


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

[Bug tree-optimization/33619] [4.1/4.2/4.3 Regression] TER breaks some inline-asm code (again)

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

   What|Removed |Added

   Priority|P3  |P1


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

[Bug c++/33616] [4.1/4.2/4.3 regression] "lvalue required as unary '&' operand" with template pmf

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

   What|Removed |Added

   Priority|P3  |P1


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

[Bug libfortran/33672] Additional runtime checks needed for namelist reads

[jvdelisle at gcc dot gnu dot org]

-- 

jvdelisle at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2007-10-07 20:07:36
   date||


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

[Bug fortran/33375] ICE (segfault) gfortran.dg/common_6.f90

[tobi at gcc dot gnu dot org]

--- Comment #4 from tobi at gcc dot gnu dot org  2007-10-07 20:03 ---
If we reject the common as syntactically wrong, it should never be resolved.


-- 

tobi at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||tobi at gcc dot gnu dot org


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

Unsere Empfehlung des Hauses persoenlich fuer gcc-bugs@gcc.gnu.org

[[EMAIL PROTECTED]]

Holen sie sich jetzt 2400 EUR in Cash ab

Und gehen Sie dabei einen sicheren und koeniglichen Weg !!! 

Alles mehrsprachig und einfach erklaert, starten Sie jetzt !!!
 
Für jeden erhaeltlich, ganz einfach: Anklicken, Registrieren, abholen !!!

WO ERHALTEN SIE SONST 2400 Euro in 12 Monaten aufgeteilt ?

http://www.2400-euro.eu

Exklusiv für Deutschland, Oesterreich & Schweiz !

Uebrigens - Günter Schlueter hat hiermit gestern und vorgestern 6100 Euro 
verdient bzw. gewonnen.

[Bug target/33120] Large module object files when declare arrays on Mac OSX

[tobi at gcc dot gnu dot org]

--- Comment #1 from tobi at gcc dot gnu dot org  2007-10-07 19:52 ---
Probably more of a target issue than a frontend issue.  Please push back to the
frontend with an explanation what the FE is doing wrong if the backend does the
right thing.


-- 

tobi at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||tobi at gcc dot gnu dot org
  Component|fortran |target
 GCC target triplet||i386-darwin


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

[Bug target/30243] [4.1/4.2/4.3 Regression][avr] signbit() causes an internal compiler error

[pinskia at gcc dot gnu dot org]

--- Comment #5 from pinskia at gcc dot gnu dot org  2007-10-07 19:51 ---
I think this patch is incorrect (there is no specific ordering of modes as far
as I know.  I think it should be using GET_MODE_SIZE instead.


-- 


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

[Bug c++/26698] [4.0/4.1/4.2/4.3 Regression] g++ accepts const-incorrect code due to conversion function

[patchapp at dberlin dot org]

--- Comment #12 from patchapp at dberlin dot org  2007-10-07 19:50 ---
Subject: Bug number PR c++/26698

A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00379.html


-- 


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

[Bug fortran/25095] Disallowed intrinsic in initialization statement

[tobi at gcc dot gnu dot org]

--- Comment #5 from tobi at gcc dot gnu dot org  2007-10-07 19:49 ---
(In reply to comment #4)
> E.g. the following is legal:

iff PRECISION(X) <= 350 :-)


-- 


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

[Bug target/30243] [4.1/4.2/4.3 Regression][avr] signbit() causes an internal compiler error

[eweddington at cso dot atmel dot com]

--- Comment #4 from eweddington at cso dot atmel dot com  2007-10-07 19:48 
---
Changing target milestone to 4.2.3. Hopefully the attached patch will be
committed by then.


-- 

eweddington at cso dot atmel dot com changed:

   What|Removed |Added

 CC||aesok at gcc dot gnu dot org
   Target Milestone|4.2.1   |4.2.3


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

[Bug target/30243] [4.1/4.2/4.3 Regression][avr] signbit() causes an internal compiler error

[eweddington at cso dot atmel dot com]

--- Comment #3 from eweddington at cso dot atmel dot com  2007-10-07 19:46 
---
Created an attachment (id=14316)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14316&action=view)
Patch by Anatoly Sokolov

Proposed patch by Anatoly Sokolov.


-- 


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

[Bug fortran/25095] Disallowed intrinsic in initialization statement

[tobi at gcc dot gnu dot org]

--- Comment #4 from tobi at gcc dot gnu dot org  2007-10-07 19:43 ---
E.g. the following is legal:
   INTEGER :: I(350)
   INTEGER, PARAMETER :: K(500) = (/ i, i=1,500 /)
   REAL :: X
   DATA I(4+K(PRECISION(X))) / 1 /
whereas
   DATA I(PRECISION(X)) / 1 /
isn't.


-- 

tobi at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||tobi at gcc dot gnu dot org


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

[Bug fortran/33664] crash on invalid program

[patchapp at dberlin dot org]

--- Comment #4 from patchapp at dberlin dot org  2007-10-07 19:15 ---
Subject: Bug number PR33664

A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00375.html


-- 


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

[Bug fortran/33542] gfortran does not detect ambigious specific names if they are the same as generic names

[patchapp at dberlin dot org]

--- Comment #8 from patchapp at dberlin dot org  2007-10-07 19:05 ---
Subject: Bug number PR33542

A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00372.html


-- 


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

[Bug libfortran/33683] calculating lgamma instead of gamma

[jvdelisle at gcc dot gnu dot org]

--- Comment #6 from jvdelisle at gcc dot gnu dot org  2007-10-07 18:43 
---
I have tested this on x86-64-linux.  It works here as well.  After reading the
man page myself, I think this is the answer and I would commit it as obvious.

The only question is what do other platforms do? AIX hp etc


-- 


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

[Bug other/32358] Acovea finds bugs in GCC 4.3.0 and problems with default optimizations levels

[patrik dot hagglund at bredband dot net]

--- Comment #7 from patrik dot hagglund at bredband dot net  2007-10-07 
17:56 ---
Created an attachment (id=14315)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14315&action=view)
example benchmark in libacovea-5.1.1


-- 


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

[Bug other/32358] Acovea finds bugs in GCC 4.3.0 and problems with default optimizations levels

[patrik dot hagglund at bredband dot net]

--- Comment #6 from patrik dot hagglund at bredband dot net  2007-10-07 
17:56 ---
Created an attachment (id=14314)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14314&action=view)
example benchmark in libacovea-5.1.1


-- 


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

[Bug libstdc++/33682] libstdc++ broken for !__GTHREAD_HAS_COND hosts

[dkwan at transmeta dot com]

--- Comment #2 from dkwan at transmeta dot com  2007-10-07 17:55 ---
Sorry about the breakage. I only tested the original patch on Linux. The
obvious patch should be fine.  I am not familiar with mingw-w64. Does it
provide a similar conditional variable as POSIX? If so, another way to fix this
is to do similar changes as in gthr-posix.h to make the new locking works for
mingw-w64 as well.


-- 


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

[Bug other/32358] Acovea finds bugs in GCC 4.3.0 and problems with default optimizations levels

[patrik dot hagglund at bredband dot net]

--- Comment #5 from patrik dot hagglund at bredband dot net  2007-10-07 
17:44 ---
I have found two bugs using my own (unfinished/unreleased) Acovea
replacement (available upon request). Both are for -fsee.

gcc (GCC) 4.3.0 20071005 (experimental)

--

gcc -std=gnu99 -O3 -fsee fftbench.c
fftbench.c: In function 'main':
fftbench.c:214: error: invalid rtl sharing found in the insn
(insn 682 681 261 27 fftbench.c:78 (set (reg/v:SI 126 [ k2.337 ])
(subreg:SI (reg:DI 313 [ k2.337 ]) 0)) -1 (nil))
fftbench.c:214: error: shared rtx
(subreg:SI (reg:DI 313 [ k2.337 ]) 0)
fftbench.c:214: internal compiler error: internal consistency failure

--

gcc -std=gnu99 -O3 -fsee -fno-regmove distbench.c
distbench.c: In function 'main':
distbench.c:140: internal compiler error: Segmentation fault


-- 

patrik dot hagglund at bredband dot net changed:

   What|Removed |Added

 CC||patrik dot hagglund at
   ||bredband dot net


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

[Bug fortran/22571] Reject derived types for dummy arguments declared in the subroutine unless they are SEQUENCE

[tobi at gcc dot gnu dot org]

--- Comment #8 from tobi at gcc dot gnu dot org  2007-10-07 17:16 ---
Something like the patch below fails unfortunately -- when resolving arglists
we still don't know enough about the callee (viz the checks of ptype, that are
already a hack on top of my original patch attempt) to determine what is legal.
 It's probably possible to add another scan over the argument list once the
procedure call has been resolved, but I haven't yet looked into this.  Also, it
doesn't really feel proper.

diff -r 8c9190af9173 gcc/fortran/resolve.c
--- a/gcc/fortran/resolve.c Sun Oct 07 11:45:15 2007 +
+++ b/gcc/fortran/resolve.c Sun Oct 07 19:11:46 2007 +0200
@@ -940,6 +940,31 @@ resolve_assumed_size_actual (gfc_expr *e
 }


+/* Given a derived type, returns true if the type is use-associated or
+   has the SEQUENCE or BIND(C) attributes and has no PRIVATE
+   componentes.  In these cases it may be used as dummy arguments in
+   calls without explicit interface.  */
+
+static bool
+interchangeable_type (gfc_typespec *ts)
+{
+  gfc_symbol *sym = ts->derived;
+
+  gcc_assert (ts->type == BT_DERIVED);
+
+  /* We don't have to recurse through potential derived type members,
+ because the type having either of use_assoc, is_bin_c or sequence
+ set requires this for the subtypes.  This requirement is checked
+ elsewhere.  Likewise, if the private_comp attribute is set in a
+ component, it is inherited by the type.  */
+  if (sym->ns != gfc_current_ns || sym->attr.use_assoc)
+return true;
+
+  return ((sym->attr.is_bind_c || sym->attr.sequence)
+ && !sym->attr.private_comp);
+}
+
+
 /* Resolve an actual argument list.  Most of the time, this is just
resolving the expressions in the list.
The exception is that we sometimes have to decide whether arguments
@@ -975,6 +1000,7 @@ resolve_actual_arglist (gfc_actual_argli
{
  if (gfc_resolve_expr (e) != SUCCESS)
return FAILURE;
+
  goto argument_list;
}

@@ -1114,6 +1140,15 @@ resolve_actual_arglist (gfc_actual_argli
return FAILURE;

 argument_list:
+  if ((ptype == PROC_EXTERNAL || ptype == PROC_UNKNOWN)
+ && e->ts.type == BT_DERIVED
+ && !interchangeable_type (&e->ts))
+   {
+ gfc_error ("Derived type '%s' cannot appear in argument list "
+"at %L", e->ts.derived->name, &e->where);
+ return FAILURE;
+   }
+ 
   /* Check argument list functions %VAL, %LOC and %REF.  There is
 nothing to do for %REF.  */
   if (arg->name && arg->name[0] == '%')


-- 

tobi at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||tobi at gcc dot gnu dot org


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

[Bug libfortran/33683] calculating lgamma instead of gamma

[tkoenig at gcc dot gnu dot org]

--- Comment #5 from tkoenig at gcc dot gnu dot org  2007-10-07 16:48 ---
This fixes things at least on my system; this also
passes regression-testing.

Index: mathbuiltins.def
===
--- mathbuiltins.def(revision 129050)
+++ mathbuiltins.def(working copy)
@@ -30,5 +30,5 @@ DEFINE_MATH_BUILTIN   (Y1,"y1", 
 DEFINE_MATH_BUILTIN   (YN,"yn", 2)
 DEFINE_MATH_BUILTIN   (ERF,   "erf",0)
 DEFINE_MATH_BUILTIN   (ERFC,  "erfc",   0)
-DEFINE_MATH_BUILTIN   (GAMMA, "gamma",  0)
+DEFINE_MATH_BUILTIN   (GAMMA, "tgamma",  0)
 DEFINE_MATH_BUILTIN   (LGAMMA,"lgamma", 0)


-- 

tkoenig at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |tkoenig at gcc dot gnu dot
   |dot org |org
 Status|NEW |ASSIGNED
   Last reconfirmed|2007-10-07 10:22:12 |2007-10-07 16:48:03
   date||


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

[Bug bootstrap/33676] libgfortran bootstrap failure: selected_int_kind.f90:22: Segmentation fault

[kargl at gcc dot gnu dot org]

--- Comment #3 from kargl at gcc dot gnu dot org  2007-10-07 16:39 ---
Sources retrieved with

svn merge -r HEAD:'{2007-10-04}' .

lead to the seg fault reported by Gerald.  So the
problem was introduced between 2007-10-03 and 
2007-10-04.

Also, note that the problem does not appear in
x86_64-*-freebsd.


-- 

kargl at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||kargl at gcc dot gnu dot org


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

[Bug tree-optimization/32921] [4.3 Regression] Revision 126326 causes 12% slowdown

[hjl at lucon dot org]

--- Comment #10 from hjl at lucon dot org  2007-10-07 15:19 ---
(In reply to comment #9)
> Gcc 4.3 revision 128885 is much slower than gcc 4.1 redhat revision 128771
> with -O2 -ffast-math on 64bit Linux/Core 2 Duo:
> 
> 437.leslie3d -26.2%
> 

The main difference between gcc 4.1 redhat and gcc 4.1 is the -mtune=generic
change. I believe gcc 4.3 will also be much slower than gcc 4.1 on Opteron.


-- 


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

[Bug tree-optimization/32921] [4.3 Regression] Revision 126326 causes 12% slowdown

[hjl at lucon dot org]

-- 

hjl at lucon dot org changed:

   What|Removed |Added

 CC||xuepeng dot guo at intel dot
   ||com
 Status|WAITING |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|2007-10-07 15:16:52 |2007-10-07 15:17:20
   date||


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

[Bug tree-optimization/32921] [4.3 Regression] Revision 126326 causes 12% slowdown

[hjl at lucon dot org]

--- Comment #9 from hjl at lucon dot org  2007-10-07 15:16 ---
Gcc 4.3 revision 128885 is much slower than gcc 4.1 redhat revision 128771
with -O2 -ffast-math on 64bit Linux/Core 2 Duo:

437.leslie3d -26.2%


-- 

hjl at lucon dot org changed:

   What|Removed |Added

 CC||Joey dot ye at intel dot com
   Last reconfirmed|2007-08-27 20:23:04 |2007-10-07 15:16:52
   date||


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

[Bug tree-optimization/33572] [4.3 Regression] wrong code with -O

[hjl at lucon dot org]

--- Comment #17 from hjl at lucon dot org  2007-10-07 14:39 ---
(In reply to comment #16)
> Subject: Bug 33572
> 
> Author: aoliva
> Date: Sat Oct  6 11:43:56 2007
> New Revision: 129051
> 
> URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129051
> Log:
> gcc/ChangeLog:
> PR tree-optimization/33572
> * tree-cfg.c (verify_stmts): Check for missing PHI defs.
> * tree-inline.c (update_ssa_across_eh_edges): Renamed to...
> (update_ssa_across_abnormal_edges): ... this.  Set slots in the
> return PHI node.
> (copy_edges_for_bb): Handle nonlocal label edges.
> (make_nonlocal_label_edges): Deleted.
> (optimize_inline_calls): Don't call it.
> gcc/testsuite/ChangeLog:
> PR tree-optimization/33572
> * g++.dg/torture/pr33572.C: New.
> 
> Added:
> trunk/gcc/testsuite/g++.dg/torture/pr33572.C
> Modified:
> trunk/gcc/ChangeLog
> trunk/gcc/testsuite/ChangeLog
> trunk/gcc/tree-cfg.c
> trunk/gcc/tree-inline.c
> 

g++.dg/torture/pr33572.C failed on Linux/ia32:

Executing on host:
/export/build/gnu/gcc/build-i686-linux/gcc/testsuite/g++/../../g++
-B/export/build/gnu/gcc/build-i686-linux/gcc/testsuite/g++/../../
/net/gnu-13/export/gnu/src/gcc/gcc/gcc/testsuite/g++.dg/torture/pr33572.C 
-nostdinc++
-I/export/build/gnu/gcc/build-i686-linux/i686-pc-linux-gnu/libstdc++-v3/include/i686-pc-linux-gnu
-I/export/build/gnu/gcc/build-i686-linux/i686-pc-linux-gnu/libstdc++-v3/include
-I/net/gnu-13/export/gnu/src/gcc/gcc/libstdc++-v3/libsupc++
-I/net/gnu-13/export/gnu/src/gcc/gcc/libstdc++-v3/include/backward
-I/net/gnu-13/export/gnu/src/gcc/gcc/libstdc++-v3/testsuite/util
-fmessage-length=0  -O0-fno-show-column  
-L/export/build/gnu/gcc/build-i686-linux/i686-pc-linux-gnu/./libstdc++-v3/src/.libs

-L/export/build/gnu/gcc/build-i686-linux/i686-pc-linux-gnu/./libstdc++-v3/src/.libs
-L/export/build/gnu/gcc/build-i686-linux/i686-pc-linux-gnu/./libiberty  -lm  
-o ./pr33572.exe(timeout = 300)
/tmp/ccSsvTAh.o: In function `vector::deallocate(int*)':^M
pr33572.C:(.text._ZN6vector10deallocateEPi[_ZN6vector10deallocateEPi]+0x24):
undefined reference to `__gnu_cxx::__pool::_M_reclaim_block(char*,
unsigned long)'^M
collect2: ld returned 1 exit status^M


-- 

hjl at lucon dot org changed:

   What|Removed |Added

 CC||hjl at lucon dot org


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

[Bug tree-optimization/33655] [4.3 Regression] ICE in bitfield_overlaps_p, at tree-sra.c:2901

[hjl at lucon dot org]

--- Comment #5 from hjl at lucon dot org  2007-10-07 14:33 ---
(In reply to comment #4)
> Subject: Bug 33655
> 
> Author: aoliva
> Date: Sat Oct  6 11:47:51 2007
> New Revision: 129052
> 
> URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129052
> Log:
> gcc/ChangeLog:
> PR tree-optimization/33655
> * tree-sra.c (bitfield_overlaps_p): Handle array and complex
> elements.
> gcc/testsuite/ChangeLog:
> PR tree-optimization/33655
> * gcc.dg/torture/pr33655.c: New.
> 
> Added:
> trunk/gcc/testsuite/gcc.dg/torture/pr33655.c
> Modified:
> trunk/gcc/ChangeLog
> trunk/gcc/testsuite/ChangeLog
> trunk/gcc/tree-sra.c
> 

This patch doesn't work on Linux/ia64. I got

Executing on host: /export/build/gnu/gcc/build-ia64-linux/gcc/xgcc
-B/export/build/gnu/gcc/build-ia64-linux/gcc/
/net/gnu-13/export/gnu/src/gcc/gcc/gcc/testsuite/gcc.dg/torture/pr33655.c   -O1
  -fno-show-column -S  -o pr33655.s(timeout = 300)
/net/gnu-13/export/gnu/src/gcc/gcc/gcc/testsuite/gcc.dg/torture/pr33655.c: In
function 'render_char':^M
/net/gnu-13/export/gnu/src/gcc/gcc/gcc/testsuite/gcc.dg/torture/pr33655.c:14:
internal compiler error: in size_binop, at fold-const.c:2036^M
Please submit a full bug report,^M
with preprocessed source if appropriate.^M
See  for instructions.^M


-- 

hjl at lucon dot org changed:

   What|Removed |Added

 CC||hjl at lucon dot org


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

[Bug fortran/20851] dummy argument may not appear in specification expression in elemental procedure

[patchapp at dberlin dot org]

--- Comment #5 from patchapp at dberlin dot org  2007-10-07 14:25 ---
Subject: Bug number PR20851

A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-10/msg00357.html


-- 


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

[Bug fortran/31608] wrong types in character array/scalar binop

[pault at gcc dot gnu dot org]

--- Comment #25 from pault at gcc dot gnu dot org  2007-10-07 12:49 ---
(In reply to comment #17)

> 
> the first line is correct, the second is not.
>

Richard, does this do it for you?

  {
char char.3[1:1];

char.3[1]{lb: 1 sz: 1} = (*(char[0:][1:1] *) atmp.0.data)[S.2][1]{lb: 1 sz:
1};
(*(char[0:][1:1] *) atmp.1.data)[S.2] = char.3;
  }


Paul


-- 


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

[Bug tree-optimization/33680] [4.3 Regression] ICE when compilling elbg.c from ffmpeg (vectorizer)

[irar at il dot ibm dot com]

--- Comment #7 from irar at il dot ibm dot com  2007-10-07 12:31 ---
(In reply to comment #3)

I get:
pr33680.c: In function גfג:
pr33680.c:1: error: expected an SSA_NAME object
pr33680.c:1: error: in statement
D.1618_93 = D.1556 /[ex] 4;
pr33680.c:1: internal compiler error: verify_ssa failed

The problem is that D.1556 is a VAR_DECL and not an SSA_NAME.

This stmt is created while gimplifying data-ref base in
vect_create_addr_base_for_vector_ref(). The expr is
(int[0:D.1553] *) newcentroid.1_22 + (long unsigned int) dim_4(D) * 8

 
sizes-gimplified type_1 BLK size 
unit size  
...

D.1618 = D.1556 /[ex] 4 is created, taking D.1556 as the unit size in
gimplify_compound_lval. And later D.1618 is replaced with an SSA_NAME
D.1618_93, since it's a lhs (in gimplify_modify_expr). 

> Vectorizer produces invalid Gimple SSA code:
> 
>   D.1769_169 = D.1599 /[ex] 4;
> 
> D.1599 should be renamed.
> 

Where should it be renamed? In gimplify_smth? 

Thanks,
Ira


-- 

irar at il dot ibm dot com changed:

   What|Removed |Added

 CC||irar at il dot ibm dot com


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

[Bug tree-optimization/33680] [4.3 Regression] ICE when compilling elbg.c from ffmpeg (vectorizer)

[pinskia at gcc dot gnu dot org]

--- Comment #6 from pinskia at gcc dot gnu dot org  2007-10-07 12:05 ---
*** Bug 33684 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||bero at arklinux dot org


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

[Bug middle-end/33684] [4.3 regression] internal compiler error: verify_ssa failed

[pinskia at gcc dot gnu dot org]

--- Comment #2 from pinskia at gcc dot gnu dot org  2007-10-07 12:05 ---
I just had reduced this bug before.

*** This bug has been marked as a duplicate of 33680 ***


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||DUPLICATE


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

[Bug libfortran/33683] calculating lgamma instead of gamma

[tobi at gcc dot gnu dot org]

--- Comment #4 from tobi at gcc dot gnu dot org  2007-10-07 12:01 ---
(In reply to comment #3)
> I think we need to use tgamma() where available.  If it isn't, we
> need a test to check whether gamma() is equivalent to tgamma() or
> to lgamma().
> 
> Yuck.

It may be simpler to calculate it ourselves, perhaps lifting code form glibc.


-- 

tobi at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||tobi at gcc dot gnu dot org


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

[Bug fortran/20851] dummy argument may not appear in specification expression in elemental procedure

[tobi at gcc dot gnu dot org]

--- Comment #4 from tobi at gcc dot gnu dot org  2007-10-07 11:46 ---
Fixed.


-- 

tobi at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.3.0


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

[Bug fortran/20851] dummy argument may not appear in specification expression in elemental procedure

[tobi at gcc dot gnu dot org]

--- Comment #3 from tobi at gcc dot gnu dot org  2007-10-07 11:45 ---
Subject: Bug 20851

Author: tobi
Date: Sun Oct  7 11:45:15 2007
New Revision: 129069

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129069
Log:
PR fortran/20851
fortran/
* expr.c (check_inquiry): Typo fix in error message.
(check_init_expr): Same * 3.
(check_restricted): Verify that no dummy arguments appear in
restricted expressions in ELEMENTAL procedures.
* resolve.c (resolve_fl_variable): Exchange order of checks to
avoid side-effect.
testsuite/
* initialization_1.f90: Fix dg-error annotations.
* initialization_14.f90: New.
* initialization_7.f90: Fix dg-error annotations.
* initialization_9.f90: Likewise.

Added:
trunk/gcc/testsuite/gfortran.dg/initialization_14.f90
Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/expr.c
trunk/gcc/fortran/resolve.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gfortran.dg/initialization_1.f90
trunk/gcc/testsuite/gfortran.dg/initialization_7.f90
trunk/gcc/testsuite/gfortran.dg/initialization_9.f90


-- 


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

[Bug rtl-optimization/33669] [4.3 Regression] Revision 128957 miscompiles 481.wrf

[zadeck at naturalbridge dot com]

-- 

zadeck at naturalbridge dot com changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |zadeck at naturalbridge dot
   |dot org |com
 Status|NEW |ASSIGNED
 GCC target triplet||linux/ia32
   Last reconfirmed|2007-10-07 09:41:07 |2007-10-07 11:36:14
   date||


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

[Bug middle-end/33684] [4.3 regression] internal compiler error: verify_ssa failed

[bero at arklinux dot org]

--- Comment #1 from bero at arklinux dot org  2007-10-07 10:24 ---
Created an attachment (id=14313)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14313&action=view)
Preprocessed source


-- 


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

[Bug middle-end/33684] New: [4.3 regression] internal compiler error: verify_ssa failed

[bero at arklinux dot org]

Compiling the attached preprocessed code at -O3 fails with

elbg.c: In function 'try_shift_candidate':
elbg.c:245: error: expected an SSA_NAME object
elbg.c:245: error: in statement
D.7733_702 = D.6104 /[ex] 4;
elbg.c:245: internal compiler error: verify_ssa failed
Please submit a full bug report,
with preprocessed source if appropriate.
See  for instructions.


-O2 is ok, -O3 on x86_32 is ok


-- 
   Summary: [4.3 regression] internal compiler error: verify_ssa
failed
   Product: gcc
   Version: 4.3.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: middle-end
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: bero at arklinux dot org
 GCC build triplet: x86_64-linux-gnu
  GCC host triplet: x86_64-linux-gnu
GCC target triplet: x86_64-linux-gnu


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

[Bug libfortran/33683] calculating lgamma instead of gamma

[tkoenig at gcc dot gnu dot org]

--- Comment #3 from tkoenig at gcc dot gnu dot org  2007-10-07 10:22 ---
(In reply to comment #2)
> I checked and the simplification routines work correctly, which means there is
> no real testsuite coverage for these functions. We should always include
> testcases comparing values calculated at runtime with constants coming from 
> our
> simplification routines.

I concur.  PR 22307 strikes again...

The cause of the problem is outlined in the gamma(3) man page
of my Linux distro:

DESCRIPTION
   For the definition of the Gamma function, see tgamma(3).

   *BSD version
   4.4BSD and FreeBSD libm have a gamma() function that computes the Gamma
   function, as one would expect.

   glibc version
   Glibc has a gamma() function that is equivalent to  lgamma()  and  com-
   putes  the  natural logarithm of the Gamma function.  (This is for com-
   patibility reasons only.  Don't use this function.)

CONFORMING TO
   4.2BSD.  Compatible with previous mistakes.

I think we need to use tgamma() where available.  If it isn't, we
need a test to check whether gamma() is equivalent to tgamma() or
to lgamma().

Yuck.


-- 

tkoenig at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2007-10-07 10:22:12
   date||


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

[Bug rtl-optimization/33669] [4.3 Regression] Revision 128957 miscompiles 481.wrf

[steven at gcc dot gnu dot org]

-- 

steven at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2007-10-07 09:41:07
   date||


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