[Bug c/51391] Differences between setting Winline in command-line and through pragma GCC diagnostic

2011-12-06 Thread pnewell at cs dot cmu.edu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51391

--- Comment #17 from pnewell at cs dot cmu.edu 2011-12-06 08:00:42 UTC ---
I am a bit afraid to do this, but I think I need to do it for my own education.

I saw the comment by Richard (#16) and have to admit that I would like a
translation into something I might understand. Its sorta like being with a
lawyer and going "Can you please speak English?". I understand that this is
internal conversation, but as the reporter of the bug I would really like to
comment on how gcc wishes to deal with it. Users may not get the big picture,
but I think anyone who reports a bug really wants to understand what the
process is.

I really appreciate (in advance) your understanding,
Thanks,
Paul


[Bug c++/51430] [4.7 Regression] ICE with array as static const member

2011-12-06 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51430

Jakub Jelinek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2011-12-06
 AssignedTo|unassigned at gcc dot   |jakub at gcc dot gnu.org
   |gnu.org |
 Ever Confirmed|0   |1

--- Comment #1 from Jakub Jelinek  2011-12-06 
08:19:56 UTC ---
Created attachment 26001
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26001
gcc47-pr51430.patch

Caused by http://gcc.gnu.org/viewcvs?view=revision&revision=174317

Untested fix.


[Bug gcov-profile/51113] [4.7 regression] rev. 181105 causes Firefox profiledbuild failure

2011-12-06 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51113

--- Comment #13 from Jakub Jelinek  2011-12-06 
08:32:02 UTC ---
This bug is about the regression that it caused, so if that regression is
fixed, this should be closed.  Whether you file a new bug (or is it enhancement
PR?) or keep it in your had is up to you I'd say, whatever you prefer.


[Bug middle-end/32667] block copy with exact overlap is expanded as memcpy

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32667

Richard Guenther  changed:

   What|Removed |Added

Summary|builtin operator= generates |block copy with exact
   |memcpy with overlapping |overlap is expanded as
   |memory regions  |memcpy
  Known to fail||4.7.0

--- Comment #8 from Richard Guenther  2011-12-06 
08:44:46 UTC ---
This is a generic middle-end issue (at best).  There is at least one duplicate
of this bug (can't find it with a quick search).


[Bug middle-end/50335] ICE in psct_dynamic_dim, at graphite-poly.h:659

2011-12-06 Thread mkuvyrkov at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50335

Maxim Kuvyrkov  changed:

   What|Removed |Added

 CC||mkuvyrkov at gcc dot
   ||gnu.org, spop at gcc dot
   ||gnu.org

--- Comment #7 from Maxim Kuvyrkov  2011-12-06 
08:50:37 UTC ---
Sebastian,

Putting this PR on your horizon.  This is an easily reproducible ICE in
graphite-poly.h.

Let us know if you are swamped and won't be able to look into this.

Thank you.


[Bug gcov-profile/51113] [4.7 regression] rev. 181105 causes Firefox profiledbuild failure

2011-12-06 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51113

Markus Trippelsdorf  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #14 from Markus Trippelsdorf  
2011-12-06 08:52:50 UTC ---
OK. Lets close this one then.


[Bug fortran/51434] ICE with scalar init of an array parameter, used in DT default init with transfer

2011-12-06 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51434

Tobias Burnus  changed:

   What|Removed |Added

   Keywords||ice-on-valid-code
 CC||burnus at gcc dot gnu.org
Summary|internal compiler error |ICE with scalar init of an
   ||array parameter, used in DT
   ||default init with transfer

--- Comment #4 from Tobias Burnus  2011-12-06 
08:53:24 UTC ---
Reduced test case. The TYPE is crucial - if "t" is just an initialization to a
module variable, no ICE occurs.

character, parameter :: s(5) = 'a'
type b
  character :: t(5) = transfer('abc  ', s)  ! << ICE
end type
!type(b) :: x
!print *, x%t   ! shall print "abc  "
!print *, s ! prints successfully "a"
end

==14250== Invalid read of size 8
==14250==at 0x51A7CD: gfc_match_data_decl() (decl.c:1589)
==14250==by 0x55F429: _ZL10match_wordPKcPF5matchvEP5locus.part.3
(parse.c:65)
==14250==by 0x55FC39: decode_statement() (parse.c:283)

That's the line:
  else if (mpz_cmp (c->ts.u.cl->length->value.integer,
c->initializer->ts.u.cl->length->value.integer))
with
  c->initializer->ts.u.cl == NULL


[Bug tree-optimization/51363] [4.7 Regression] ICE: in gimple_rhs_has_side_effects, at gimple.c:2530 with -O -fno-tree-ccp

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51363

--- Comment #2 from Richard Guenther  2011-12-06 
09:02:23 UTC ---
Author: rguenth
Date: Tue Dec  6 09:02:17 2011
New Revision: 182041

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182041
Log:
2011-12-06  Richard Guenther  

PR tree-optimization/51363
* gimple.c (gimple_rhs_has_side_effects): Remove.
* gimple.h (gimple_rhs_has_side_effects): Likewise.
* tree-ssa-dom.c (optimize_stmt): Simplify conditional.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/gimple.c
trunk/gcc/gimple.h
trunk/gcc/tree-ssa-dom.c


[Bug tree-optimization/51363] [4.7 Regression] ICE: in gimple_rhs_has_side_effects, at gimple.c:2530 with -O -fno-tree-ccp

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51363

Richard Guenther  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #3 from Richard Guenther  2011-12-06 
09:02:35 UTC ---
Fixed.


[Bug lto/51432] [4.6 regression] ICE in -flto -std=c++0x -g with cross-compiler

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51432

Richard Guenther  changed:

   What|Removed |Added

   Keywords||lto
 Blocks||47819
   Target Milestone|--- |4.6.3


[Bug lto/48094] ld: warning: section has unexpectedly large size errors in objc/obj-c++ lto

2011-12-06 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48094

--- Comment #14 from Iain Sandoe  2011-12-06 09:11:56 
UTC ---
(In reply to comment #13)
> (In reply to comment #12)
> > I guess, ideally, the ObjC meta-data should be re-created after LTO has done
> > its magic -- but that's def. not a stage 3 type job ...
> 
> Ideally from what point of view? Certainly not that of one where there is
> proper separation between front ends and the middle end. There are no language
> specific post optimization passes. From the point of view of the LTO front 
> end,
> whatever the ObjC-family front ends have handed over is language independent.

... I do see the point about separation - and I wasn't intending to imply that
there was lang-specific optimization.
 (what I meant to say is that, if the meta-data are created for the optimized
code, then they are already - by definition - optimized themselves).

I would not propose a lang-specific ME pass ... however, I might propose a pass
that deals with one category of construct (which only happens to be generated
by one FE).   Surely we do this already - the ME just deals with constructs -
some of which are only ever generated by Fortran or Ada or C++ ?? 

 [[ probably displaying my considerable lack of knowledge about the ME here ]]

===

The metadata are really a property of the target runtime (thus, they change
depending on the target runtime, for the same ObjC source).

So, at the least, they are in a grey area.  (IMO) The front-end would ideally
be completely target-runtime agnostic - at least until lowering to gimple (the
method calls are constructed differently, for example).

so, for the same ME data, one should - in principle - be able to generate the
correct metadata for the target  ...

... however, clearly, this does not belong in the back end either (since that
would imply duplication that is not needed) ... :-(

-- so perhaps we are left with needing to communicate something between the two
... in the same manner that we "tell" the back end how to express things that
have alignment or visibility attributes attached to them in the FE ... 

just musing about possible solutions .. nothing that feels particularly "Right"
as it stands ... .

> But those two L_OBJC_ImageInfo variables have a number appended. Is that
> supposed to be so? Or is this LTO declaration merging at work and maybe you
> want to avoid that?
> (Maybe I'm talking completely non-sense, but something like that came up for
> PR47259 also...)

I think that LTO is behaving entirely correctly now (the original example in
the bug has been fixed).

N input files containing a local variable (even if it has the same name in
each) should produce N instances of that in the output - which is what is
happening.

I would guess have no way to mark a variable as "local, but shared" because -
probably outside of this case there is no ld that could make sense of it. 

 I don't know if it will currently break the NeXT ObjC runtime if we re-specify
the section as coalesced and mark the vars as DECL_ONE_ONLY.. but I suspect it
will break at some point.


[Bug libgomp/50977] non-deterministic failure in cactusADM using openmp

2011-12-06 Thread razya at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50977

razya at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||FIXED

--- Comment #4 from razya at gcc dot gnu.org 2011-12-06 09:25:10 UTC ---
libgomp patches by Alan Modra resolved the failures.

http://gcc.gnu.org/ml/gcc-patches/2011-11/msg02387.html
http://gcc.gnu.org/ml/gcc-patches/2011-11/msg02455.html
http://gcc.gnu.org/ml/gcc-patches/2011-11/msg02458.html

Closing this PR.
Thanks,
Razya


[Bug fortran/51435] New: Bad association status after null() of derived type component

2011-12-06 Thread darmar.xxl at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51435

 Bug #: 51435
   Summary: Bad association status after null() of derived type
component
Classification: Unclassified
   Product: gcc
   Version: 4.6.1
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: fortran
AssignedTo: unassig...@gcc.gnu.org
ReportedBy: darmar@gmail.com


Created attachment 26002
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26002
test code

Here is the code:
!---
module arr_m
type arr_t
real(8), dimension(:), allocatable :: rsk
! if allocatable changed to pointer, all is OK
end type
end module arr_m
!*
module list_m
use arr_m
implicit none

type my_list
type(arr_t), pointer :: head => null()
!!!integer, pointer :: tail => null() !if this is uncommented, all
is OK.
end type my_list
end module list_m
!***
module worker_mod
use list_m
implicit none

type data_all_t
type(my_list) :: my_data
end type data_all_t
contains
subroutine do_job()
type(data_all_t) :: dum

if (associated(dum%my_data%head)) then
print *, 'do_job my_data%head is associated' 
!
! gfortran goes here.
! WRONG. head was nullified.
!
else
print *, 'do_job my_data%head is NOT associated' ! OK
end if

end subroutine

end module
!***
program hello
use worker_mod
implicit none

call do_job()

end program
!***


gfortran reports that dum%my_data%head is associated in the sub "do_job", while
"head" component was nullified using null() in the variable declaration line.

I tested this with gfortran 4.6 and 4.7 on Linux and Windows (produces wrong
results). Version 4.5 outputs correct result. (ifort is OK).


[Bug middle-end/51436] New: [4.7 Regression] ICE(segfault) in gimplify_and_update_call_from_tree gimple-fold.c:603

2011-12-06 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51436

 Bug #: 51436
   Summary: [4.7 Regression] ICE(segfault) in
gimplify_and_update_call_from_tree gimple-fold.c:603
Classification: Unclassified
   Product: gcc
   Version: 4.7.0
Status: UNCONFIRMED
  Keywords: ice-on-valid-code
  Severity: normal
  Priority: P3
 Component: middle-end
AssignedTo: unassig...@gcc.gnu.org
ReportedBy: bur...@gcc.gnu.org
  Host: x86_64-unknown-linux-gnu


Created attachment 26003
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26003
Reduced test case (test.ii, 5 KiB)

The attached test case compiles with g++ 4.6 but with the current 4.7.0
20111206 it fails with:

$ g++ test52.ii
test52.ii: In static member function ‘static Dest
v8::internal::BitCastHelper::cast(Source*) [with Dest =
v8::internal::String**; Source = v8::internal::String*]’:
test52.ii:116:16: internal compiler error: Segmentation fault


Program received signal SIGSEGV, Segmentation fault.
gimplify_and_update_call_from_tree (si_p=0x7fffd8b0, expr=)
at gcc/gimple-fold.c:603
603   if (TREE_CODE (vdef) == SSA_NAME)
(gdb) p vdef
$1 = (tree_node *) 0x0


(gdb) bt
#0  gimplify_and_update_call_from_tree (si_p=0x7fffd8b0, expr=)
at gcc/gimple-fold.c:603
#1  0x008e704d in gimple_fold_call (inplace=,
gsi=)
at gcc/gimple-fold.c:1123
#2  fold_stmt_1 (gsi=0x7fffd8b0, inplace=false) at gcc/gimple-fold.c:1187
#3  0x00a96b41 in fold_marked_statements (first=5,
statements=0x173bbc0)
at gcc/tree-inline.c:4131
#4  0x00a9f1f7 in optimize_inline_calls (fn=0x2cf86b00)
at gcc/tree-inline.c:4275
#5  0x00f03223 in early_inliner () at gcc/ipa-inline.c:1968
#6  0x009af115 in execute_one_pass (pass=0x15bf400) at
gcc/passes.c:2074
#7  0x009af4b5 in execute_pass_list (pass=0x15bf400) at
gcc/passes.c:2129
#8  0x009ae684 in do_per_function_toporder (callback=0x9af4a0
, data=0x15bbbc0)
at gcc/passes.c:1616


[Bug libstdc++/51183] pair piecewise_construct_t constructor copies

2011-12-06 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51183

Paolo Carlini  changed:

   What|Removed |Added

 CC|paolo.carlini at oracle dot |
   |com |
 AssignedTo|unassigned at gcc dot   |paolo.carlini at oracle dot
   |gnu.org |com
   Target Milestone|--- |4.7.0

--- Comment #12 from Paolo Carlini  2011-12-06 
09:55:45 UTC ---
I'm taking care of this.


[Bug libstdc++/51183] pair piecewise_construct_t constructor copies

2011-12-06 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51183

Paolo Carlini  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED


[Bug tree-optimization/50904] [4.7 regression] pessimization when -fno-protect-parens is enabled by -Ofast

2011-12-06 Thread venkataramanan.kumar.gnu at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50904

--- Comment #49 from Venkataramanan Kumar  2011-12-06 09:59:39 UTC ---
I am planning to test the patch on polyhedron benchmarks. 
Then I test it on CPU2006 SPEC.


[Bug target/51354] [4.7 Regression] ICE in maybe_record_trace_start

2011-12-06 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51354

Jakub Jelinek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2011-12-06
 AssignedTo|unassigned at gcc dot   |jakub at gcc dot gnu.org
   |gnu.org |
 Ever Confirmed|0   |1

--- Comment #1 from Jakub Jelinek  2011-12-06 
10:02:15 UTC ---
Created attachment 26004
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26004
gcc47-pr51354.patch

Untested fix.


[Bug c++/51427] Less-than-useful error message when union/struct tags conflict

2011-12-06 Thread dodji at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51427

--- Comment #3 from Dodji Seketeli  2011-12-06 
10:06:55 UTC ---
Author: dodji
Date: Tue Dec  6 10:06:49 2011
New Revision: 182043

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182043
Log:
PR c++/51427 - Better diagnostic when union/struct tags conflicts

gcc/cp/

PR c++/51427
* parser.c (cp_parser_check_class_key): Add note about earlier
declaration.

gcc/testsuite/

PR c++/51427
* g++.dg/diagnostic/wrong-tag-1.C

Added:
trunk/gcc/testsuite/g++.dg/diagnostic/wrong-tag-1.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/parser.c
trunk/gcc/testsuite/ChangeLog


[Bug fortran/51435] [4.6/4.7 Regression] Bad association status after null() of derived type component

2011-12-06 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51435

Tobias Burnus  changed:

   What|Removed |Added

   Keywords||wrong-code
 CC||burnus at gcc dot gnu.org
   Target Milestone|--- |4.6.3
Summary|Bad association status  |[4.6/4.7 Regression] Bad
   |after null() of derived |association status after
   |type component  |null() of derived type
   ||component

--- Comment #1 from Tobias Burnus  2011-12-06 
10:08:32 UTC ---
With GCC 4.3, 4.4 and 4.5, I get:
  do_job my_data%head is NOT associated
while GCC 4.6 prints
 do_job my_data%head is associated

With GCC 4.7 I get the correct result ("NOT associated") but it shows the same
valgrind warning as GCC 4.6. (GCC 4.5 shows no valgrind warning.) Thus, either
the issue is fixed in 4.7 - and 4.6 and 4.7 have additional issues. Or it works
in 4.7 only by chance.

==19540== Conditional jump or move depends on uninitialised value(s)
==19540==at 0x400776: __worker_mod_MOD_do_job (hjhjff.f90:29)
==19540==by 0x400849: MAIN__ (hjhjff.f90:47)
==19540==by 0x40087F: main (hjhjff.f90:44)

Line 29 is:
if (associated(dum%my_data%head)) then


[Bug middle-end/48087] [4.5/4.6/4.7 Regression] -Wall -Werror adds warnings over and above those generated by -Wall

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48087

--- Comment #20 from Richard Guenther  2011-12-06 
10:09:20 UTC ---
Looking again, I think that for warnings promoted to errors we should _not_
increase errorcount, but we should instead have a new global flag that
tells us whether there were any "real" errors, and check that in the
place that terminates compilation without producing assembler output.


[Bug c++/51427] Less-than-useful error message when union/struct tags conflict

2011-12-06 Thread dodji at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51427

Dodji Seketeli  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #4 from Dodji Seketeli  2011-12-06 
10:10:44 UTC ---
Fixed in trunk (4.7).


[Bug libstdc++/51183] pair piecewise_construct_t constructor copies

2011-12-06 Thread chris at bubblescope dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51183

--- Comment #13 from Chris Jefferson  2011-12-06 
10:13:56 UTC ---
You can if you like, but if you haven't started yet, I plan on having a patch
ready in about... 2 hours?


[Bug tree-optimization/48193] [4.7 Regression] ICE: verify_ssa failed: type mismatch between an SSA_NAME and its symbol with -flto --param partial-inlining-entry-probability=101

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48193

Richard Guenther  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #3 from Richard Guenther  2011-12-06 
10:14:29 UTC ---
Fixed.


[Bug middle-end/48666] [4.6/4.7 Regression] -Warray-bounds doesn't trigger anymore for simple case

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48666

Richard Guenther  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||INVALID

--- Comment #4 from Richard Guenther  2011-12-06 
10:26:54 UTC ---
By design:

  /* Accesses to trailing arrays via pointers may access storage
 beyond the types array bounds.  */
...
  /* If this is the last field in a struct type or a field in a
 union type do not warn.  */
  if (!next)
return;

both val32Tab2 and val32Tab3 might be allocated larger and used
as trailing array.  We'd warn during GCC bootstrap for our RTX union
for example.


[Bug middle-end/51436] [4.7 Regression] ICE(segfault) in gimplify_and_update_call_from_tree gimple-fold.c:603

2011-12-06 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51436

Markus Trippelsdorf  changed:

   What|Removed |Added

 CC||markus at trippelsdorf dot
   ||de

--- Comment #1 from Markus Trippelsdorf  
2011-12-06 10:31:20 UTC ---
Slightly smaller testcase:

typedef long unsigned size_t;
extern "C" void *memcpy (void *, __const void *, size_t);
template < class Dest, class Source > struct BitCastHelper {
static Dest cast (const Source & source) __attribute__ ((always_inline)) {
Dest dest;
memcpy (0, &source, sizeof dest);
}
};
template < class Dest, class Source > Dest BitCast (Source)
{
BitCastHelper < Dest, Source >::cast (0);
}

class MaybeObject
{
};
class Object:MaybeObject
{
public:
static Object *cast (Object *) {
}
};
class HeapObject:public Object
{
};
class String:public HeapObject
{
};
class ExternalString:public String
{
};
class ExternalTwoByteString:public ExternalString
{
};

template < typename T > class Handle
{
public:
Handle () {
}
T *operator* () const;
template < class S > static Handle < T > cast (Handle < S > that) {
T::cast (*that);
}
T **location_;
};

template < typename T > T * Handle < T >::operator* () const
{
*BitCast < T ** >(location_);
}

void
TestCharacterStream ()
{
Handle < String > uc16_string;
Handle < ExternalTwoByteString >::cast (uc16_string);
}


[Bug libstdc++/51183] pair piecewise_construct_t constructor copies

2011-12-06 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51183

--- Comment #14 from Paolo Carlini  2011-12-06 
10:30:42 UTC ---
I'm attaching what I already tested and was going to commit. If you like,
please work on top of it and produce a combined new patch. Like, if you think
another testcase is necessary or, if the assembly supports you, you believe
non-const ref passing is actually useful.

I'd like to resolve this issue today and move on. Thanks!


[Bug libstdc++/51183] pair piecewise_construct_t constructor copies

2011-12-06 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51183

--- Comment #15 from Paolo Carlini  2011-12-06 
10:31:30 UTC ---
Created attachment 26005
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26005
Tested patch


[Bug tree-optimization/48766] [4.4/4.5/4.6/4.7 Regression] Infinite recursion in fold_binary_loc()

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48766

Richard Guenther  changed:

   What|Removed |Added

 CC||jsm28 at gcc dot gnu.org

--- Comment #5 from Richard Guenther  2011-12-06 
10:33:11 UTC ---
(In reply to comment #3)
> The combination -fwrapv -ftrapv is not particularly meaningful; it ought 
> to act exactly the same as -ftrapv (i.e. -ftrapv should override any 
> previous -fwrapv, and vice versa; -fwrapv -fno-trapv should mean -fwrapv 
> and -ftrapv -fno-wrapv should mean -ftrapv, as at present).

I suppose the new Negative() .opt file annotation cannot cover this?

Internally we probably should have a single enum that enumerates all
valid integer overflow behaviors (what about the weak -f[no-]strict-overflow)?


[Bug fortran/51435] [4.6/4.7 Regression] Bad association status after null() of derived type component

2011-12-06 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51435

--- Comment #2 from Tobias Burnus  2011-12-06 
10:33:57 UTC ---
Lightly tested patch:

--- a/gcc/fortran/expr.c
+++ b/gcc/fortran/expr.c
@@ -3733,17 +3733,14 @@ gfc_has_default_initializer (gfc_symbol *der)
 if (c->ts.type == BT_DERIVED)
   {
 if (!c->attr.pointer
 && gfc_has_default_initializer (c->ts.u.derived))
  return true;
   }
-else
-  {
-if (c->initializer)
- return true;
-  }
+if (c->initializer)
+  return true;

   return false;
 }

 /* Get an expression for a default initializer.  */


[Bug tree-optimization/49772] [4.7 Regression] ICE: in ipa_pta_execute, at tree-ssa-structalias.c:6790 with -fipa-pta

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49772

Richard Guenther  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #7 from Richard Guenther  2011-12-06 
10:37:53 UTC ---
We no longer ICE.


[Bug middle-end/50426] [4.7 Regression] gfortran -O1 ICE in estimate_function_body_sizes

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50426

Richard Guenther  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #5 from Richard Guenther  2011-12-06 
10:44:51 UTC ---
1977gcc_assert (!gimple_call_cannot_inline_p (stmt));

has been removed.


[Bug lto/50601] [4.7 Regression] New LTO failures

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50601

Richard Guenther  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|unassigned at gcc dot   |rguenth at gcc dot gnu.org
   |gnu.org |

--- Comment #4 from Richard Guenther  2011-12-06 
10:56:50 UTC ---
FAIL: g++.dg/lto/20100302 cp_lto_20100302_0.o-cp_lto_20100302_1.o link, -flto
-fabi-version=2 (internal compiler error)

is

lto1: internal compiler error: in make_decl_rtl, at varasm.c:1141

an ICE during ltrans stage:

1136  /* A weak alias has TREE_PUBLIC set but not the other bits.  */
1137  gcc_assert (TREE_CODE (decl) != VAR_DECL
1138  || TREE_STATIC (decl)
1139  || TREE_PUBLIC (decl)
1140  || DECL_EXTERNAL (decl)
1141  || DECL_REGISTER (decl));

and the decl is the VAR_DECL _ZN1AIDv4_fE1tE (A::t).

This ABI forward-compatibility alias the C++ frontend generates does
not honor the constraints that assert asserts:

(gdb) call debug_tree (decl)
 
unit size 
align 32 symtab 0 alias set -1 canonical type 0x75a31e70
precision 32
pointer_to_this >
V4SF
size 
unit size 
align 128 symtab 0 alias set -1 canonical type 0x75b5f930 nunits 4
pointer_to_this >
addressable used ignored V4SF file
/space/rguenther/src/svn/trunk/gcc/testsuite/g++.dg/lto/20100302_0.C line 9 col
19 size  unit size 
align 128>

we create RTL for this decl because do_assemble_alias does

5452  /* We must force creation of DECL_RTL for debug info generation, even
though
5453 we don't use it here.  */
5454  make_decl_rtl (decl);

but the alias has DECL_IGNORED set, so we can fix this by not creating
RTL for DECL_IGNORED decls (hopefully).


[Bug lto/50601] [4.7 Regression] New LTO failures

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50601

--- Comment #5 from Richard Guenther  2011-12-06 
11:03:29 UTC ---
Which shifts the error to

/space/rguenther/src/svn/trunk/gcc/testsuite/g++.dg/lto/20100302_1.C: In
function 'main':^M
/space/rguenther/src/svn/trunk/gcc/testsuite/g++.dg/lto/20100302_1.C:5:5:
error: invalid argument to gimple call^M
&_ZN1AIDv4_fE1tE^M
/space/rguenther/src/svn/trunk/gcc/testsuite/g++.dg/lto/20100302_1.C:5:5:
error: constant not recomputed when ADDR_EXPR changed^M
&_ZN1AIDv4_fE1tE^M

...


[Bug c/51437] New: GCC should warn on the use of reserved identifier/macro names

2011-12-06 Thread vanboxem.ruben at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51437

 Bug #: 51437
   Summary: GCC should warn on the use of reserved
identifier/macro names
Classification: Unclassified
   Product: gcc
   Version: unknown
Status: UNCONFIRMED
  Severity: enhancement
  Priority: P3
 Component: c
AssignedTo: unassig...@gcc.gnu.org
ReportedBy: vanboxem.ru...@gmail.com


For C and C++, Clang should warn when user code uses identifiers or macros that
are implementation-reserved.

For C(99), this would mean names starting with an underscore followed by
another unerscore or capitalized letter.

For C++ a single underscore is also reserved for the global namespace.

I could have messed the above rules up, but these are the cause of a lot of
unexpected bugs, and a warning would be super-easy to implement.


[Bug lto/50601] [4.7 Regression] New LTO failures

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50601

--- Comment #6 from Richard Guenther  2011-12-06 
11:34:41 UTC ---
Because the decl isn't considered decl_address_invariant_p () - of course,
it is neither TREE_STATIC nor DECL_EXTERNAL ...

It _is_ both TREE_PUBLIC and TREE_STATIC when created by the C++ frontend
(and during compile stage).

We get

2
cp_lto_20100302_0.o 4
164 6cc97232062f75e0 PREVAILING_DEF_IRONLY _Z1fPU8__vectorf
177 6cc97232062f75e0 PREVAILING_DEF_IRONLY _Z1fPDv4_f
180 6cc97232062f75e0 PREVAILING_DEF_IRONLY _ZN1AIDv4_fE1tE
182 6cc97232062f75e0 PREVAILING_DEF_IRONLY _ZN1AIU8__vectorfE1tE
cp_lto_20100302_1.o 3
164 5d25700523736b18 PREVAILING_DEF main
168 5d25700523736b18 RESOLVED_IR _Z1fPDv4_f
179 5d25700523736b18 RESOLVED_IR _ZN1AIDv4_fE1tE

which is then mangled via cgraph_make_decl_local during the IPA
function and variable visibility pass.  This removes the TREE_PUBLIC
flag, the TREE_STATIC flag remains at WPA stage.  Note that the
parent, non-alias decl stays TREE_PUBLIC.

The no longer TREE_PUBLIC alias is then mangled at LTRANS stage, and
the TREE_STATIC flag is cleared in:

input_varpool_node (file_data=0x75b61000, ib=0x19ec4f0)
at /space/rguenther/src/svn/trunk/gcc/lto-cgraph.c:1083
1083  if (node->finalized)
(gdb) l
1078  if (node->in_other_partition)
1079{
1080  DECL_EXTERNAL (node->decl) = 1;
1081  TREE_STATIC (node->decl) = 0;
1082}

but DECL_EXTERNAL is in turn cleared again by

assemble_alias (decl=0x75a2c140, target=0x75b5af00)
at /space/rguenther/src/svn/trunk/gcc/varasm.c:5850
5850  if (TREE_CODE (decl) == FUNCTION_DECL)
(gdb) l
5845 note that the symbol is in fact locally defined.  */
5846  if (! is_weakref)
5847DECL_EXTERNAL (decl) = 0;

that, of course, looks bogus in the light of LTO partitioning.  Doing

  /* A quirk of the initial implementation of aliases required that the user
 add "extern" to all of them.  Which is silly, but now historical.  Do
 note that the symbol is in fact locally defined.  */
  if (! is_weakref
  && (TREE_CODE (decl) == FUNCTION_DECL
  || !varpool_node (decl)->in_other_partition))
DECL_EXTERNAL (decl) = 0;

instead "fixes" this bug.


[Bug c/51437] GCC should warn on the use of reserved identifier/macro names

2011-12-06 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51437

--- Comment #1 from Jonathan Wakely  2011-12-06 
11:37:39 UTC ---
(In reply to comment #0)
> For C and C++, Clang

Ahem!

> For C++ a single underscore is also reserved for the global namespace.

That comes from C, not C++:
"All identifiers that begin with an underscore are always reserved for use as
identifiers with file scope in both the ordinary and tag name spaces."


[Bug tree-optimization/51245] [4.7 Regression] ICE in vn_reference_insert_pieces, at tree-ssa-sccvn.c:1983

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51245

--- Comment #3 from Richard Guenther  2011-12-06 
11:41:04 UTC ---
Author: rguenth
Date: Tue Dec  6 11:41:00 2011
New Revision: 182044

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182044
Log:
2011-12-06  Richard Guenther  

PR tree-optimization/51245
* tree-ssa-sccvn.c (vn_reference_lookup_or_insert_constant_for_pieces):
New function.
(vn_reference_lookup_3): Use it.  Properly valueize all refs
we create.

* gcc.dg/torture/pr51245.c: New testcase.

Added:
trunk/gcc/testsuite/gcc.dg/torture/pr51245.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-ssa-sccvn.c


[Bug target/34501] The vector cost model does not seem suited for Intel Core2Duo

2011-12-06 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34501

Dominique d'Humieres  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #6 from Dominique d'Humieres  2011-12-06 
11:45:39 UTC ---
Although I don't know if the cost model is perfectly tuned for the Intel
Core2Duo, the particular instance of this PR has been fixed since a long time
(see pr34265 and pr50904), on trunk at r182043, I now get on a slightly faster
proc (2.5Ghz vs. 2.1Ghz):

[macbook] lin/test% time gfc -O3 -ffast-math -funroll-loops induct.f90
7.969u 0.101s 0:08.07 99.8%0+0k 0+40io 0pf+0w
[macbook] lin/test% time a.out > /dev/null
13.062u 0.026s 0:13.09 99.9%0+0k 0+0io 0pf+0w
[macbook] lin/test% time gfc -O3 -ffast-math -funroll-loops induct.f90 --param
min-vect-loop-bound=2
7.965u 0.110s 0:08.08 99.8%0+0k 0+23io 0pf+0w
[macbook] lin/test% time a.out > /dev/null
13.063u 0.027s 0:13.09 99.9%0+0k 0+0io 0pf+0w

So I am closing the PR as fixed. Thanks for all the work leading to a nice
speed-up.


[Bug c/51437] GCC should warn on the use of reserved identifier/macro names

2011-12-06 Thread vanboxem.ruben at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51437

--- Comment #2 from Ruben Van Boxem  
2011-12-06 11:50:18 UTC ---
(In reply to comment #1)
> (In reply to comment #0)
> > For C and C++, Clang
> 
> Ahem!
> 
> > For C++ a single underscore is also reserved for the global namespace.
> 
> That comes from C, not C++:
> "All identifiers that begin with an underscore are always reserved for use as
> identifiers with file scope in both the ordinary and tag name spaces."

Fair enough. I warned you, I only quickly glanced at both standard documens :)


[Bug tree-optimization/50622] [4.7 Regression] ICE: verify_gimple failed for std::complex

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50622

Richard Guenther  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #11 from Richard Guenther  2011-12-06 
11:55:52 UTC ---
Fixed.


[Bug tree-optimization/50802] [4.7 Regression] FAIL: gcc.c-torture/execute/arith-rand-ll.c execution at -O2 and -Os

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50802

Richard Guenther  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #20 from Richard Guenther  2011-12-06 
12:04:38 UTC ---
Fixed.


[Bug tree-optimization/50729] [4.7 Regression] Silent code gen fault: Value range propagation seems to propagate values across narrowing/widening

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50729

Richard Guenther  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED

--- Comment #10 from Richard Guenther  2011-12-06 
12:04:45 UTC ---
Fixed.


[Bug tree-optimization/51245] [4.7 Regression] ICE in vn_reference_insert_pieces, at tree-ssa-sccvn.c:1983

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51245

Richard Guenther  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #4 from Richard Guenther  2011-12-06 
12:06:01 UTC ---
Fixed.


[Bug middle-end/51436] [4.7 Regression] ICE(segfault) in gimplify_and_update_call_from_tree gimple-fold.c:603

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51436

Richard Guenther  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2011-12-06
 AssignedTo|unassigned at gcc dot   |rguenth at gcc dot gnu.org
   |gnu.org |
 Ever Confirmed|0   |1

--- Comment #2 from Richard Guenther  2011-12-06 
12:07:53 UTC ---
Mine.


[Bug middle-end/51436] [4.7 Regression] ICE(segfault) in gimplify_and_update_call_from_tree gimple-fold.c:603

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51436

--- Comment #3 from Richard Guenther  2011-12-06 
12:17:42 UTC ---
Author: rguenth
Date: Tue Dec  6 12:17:31 2011
New Revision: 182045

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182045
Log:
2011-12-06  Richard Guenther  

PR middle-end/51436
* gimple-fold.c (gimplify_and_update_call_from_tree): Guard
vdef check for the fact we do not have virtual operands when
not optimizing.

* g++.dg/torture/pr51436.C: New testcase.

Added:
trunk/gcc/testsuite/g++.dg/torture/pr51436.C
Modified:
trunk/gcc/ChangeLog
trunk/gcc/gimple-fold.c
trunk/gcc/testsuite/ChangeLog


[Bug middle-end/51436] [4.7 Regression] ICE(segfault) in gimplify_and_update_call_from_tree gimple-fold.c:603

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51436

Richard Guenther  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.7.0

--- Comment #4 from Richard Guenther  2011-12-06 
12:18:03 UTC ---
Fixed.


[Bug middle-end/51144] r181279 possibly miscompilation of genmddeps

2011-12-06 Thread krebbel at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51144

Andreas Krebbel  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #7 from Andreas Krebbel  2011-12-06 
12:29:55 UTC ---
Fixed per above comment.


[Bug c++/43674] Delegating constructors are not supported

2011-12-06 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43674

Paolo Carlini  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.7.0

--- Comment #4 from Paolo Carlini  2011-12-06 
12:36:02 UTC ---
Done with this patch:

2011-12-05  Ville Voutilainen  
Pedro Lamarão  

Implement C++11 delegating constructors.


[Bug middle-end/50823] [4.7 Regression] ICE in inline_small_functions, at ipa-inline.c:1407

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50823

--- Comment #13 from Richard Guenther  2011-12-06 
13:03:40 UTC ---
This fixes it:

Index: gcc/ipa-inline.c
===
--- gcc/ipa-inline.c(revision 182044)
+++ gcc/ipa-inline.c(working copy)
@@ -808,7 +808,6 @@ edge_badness (struct cgraph_edge *edge,
   else if (flag_guess_branch_prob)
 {
   int div = edge->frequency * (1<<10) / CGRAPH_FREQ_MAX;
-  int growth_for_all;

   div = MAX (div, 1);
   gcc_checking_assert (edge->frequency <= CGRAPH_FREQ_MAX);
@@ -846,14 +845,12 @@ edge_badness (struct cgraph_edge *edge,
  if (dump)
fprintf (dump_file, "Badness overflow\n");
}
-  growth_for_all = estimate_growth (callee);
-  badness += growth_for_all;
   if (dump)
{
  fprintf (dump_file,
-  "  %i: guessed profile. frequency %f, overall growth
%i,"
+  "  %i: guessed profile. frequency %f,"
   " benefit %f%%, divisor %i\n",
-  (int) badness, (double)edge->frequency / CGRAPH_FREQ_BASE,
growth_for_all,
+  (int) badness, (double)edge->frequency / CGRAPH_FREQ_BASE,
   relative_time_benefit (callee_info, edge, time_growth) * 100
/ 256.0, div);
}
 }

I'm going to bootstrap, test and apply this given that Honza seems to have
no interest in fixing his own regressions.  [and this particular patch
was on my agenda for quite some time]


[Bug fortran/51435] [4.6/4.7 Regression] Bad association status after null() of derived type component

2011-12-06 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51435

Jakub Jelinek  changed:

   What|Removed |Added

   Priority|P3  |P4
 CC||jakub at gcc dot gnu.org


[Bug middle-end/48666] [4.6/4.7 Regression] -Warray-bounds doesn't trigger anymore for simple case

2011-12-06 Thread franz.sirl-kernel at lauterbach dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48666

--- Comment #5 from franz.sirl-kernel at lauterbach dot com 2011-12-06 13:09:25 
UTC ---
(In reply to comment #4)

I see. Does it make sense to file an enhancement request for something like
-Warray-bounds=strict ? In our case the code was really wrong and the warning
was helpful.


[Bug middle-end/48666] [4.6/4.7 Regression] -Warray-bounds doesn't trigger anymore for simple case

2011-12-06 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48666

Manuel López-Ibáñez  changed:

   What|Removed |Added

 CC||manu at gcc dot gnu.org

--- Comment #6 from Manuel López-Ibáñez  2011-12-06 
13:25:11 UTC ---
Clang warns:

pr48666.c:11:3: warning: array index of '4' indexes past the end of an array
(that contains 3 elements) [-Warray-bounds]
  x->val32Tab3[4] = 42;
  ^~
pr48666.c:5:3: note: array 'val32Tab3' declared here
  int val32Tab3[3];
  ^
1 warning generated.

It seems they took the opposite view:

http://llvm.org/bugs/show_bug.cgi?id=10871


[Bug lto/50601] [4.7 Regression] New LTO failures

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50601

--- Comment #7 from Richard Guenther  2011-12-06 
13:26:20 UTC ---
Author: rguenth
Date: Tue Dec  6 13:26:09 2011
New Revision: 182048

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182048
Log:
2011-12-06  Richard Guenther  

PR middle-end/50601
* varasm.c (assemble_alias): Move DECL_EXTERNAL implementation
quirk adjustjment ...
* passes.c (rest_of_decl_compilation): ... here.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/passes.c
trunk/gcc/varasm.c


[Bug lto/50601] [4.7 Regression] New LTO failures

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50601

Richard Guenther  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #8 from Richard Guenther  2011-12-06 
13:29:11 UTC ---
Fixed.


[Bug c/51437] GCC should warn on the use of reserved identifier/macro names

2011-12-06 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51437

--- Comment #3 from Jonathan Wakely  2011-12-06 
13:34:36 UTC ---
The C standard also reserves various names for future use in 7.26


[Bug tree-optimization/49997] [4.7 Regression] ICE in inline_small_functions with -fnon-call-exceptions

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49997

Richard Guenther  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||WORKSFORME

--- Comment #7 from Richard Guenther  2011-12-06 
13:45:44 UTC ---
This particular testcase no longer ICEs for me.


[Bug tree-optimization/49997] [4.7 Regression] ICE in inline_small_functions with -fnon-call-exceptions

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49997

--- Comment #6 from Richard Guenther  2011-12-06 
13:45:28 UTC ---
Author: rguenth
Date: Tue Dec  6 13:45:19 2011
New Revision: 182049

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182049
Log:
2011-12-06  Richard Guenther  

PR tree-optimization/49997
* gcc.dg/torture/pr49997.c: New testcase.

Added:
trunk/gcc/testsuite/gcc.dg/torture/pr49997.c
Modified:
trunk/gcc/testsuite/ChangeLog


[Bug middle-end/51263] [4.7 regression] ICE in inline_small_functions when compiling scummvm with -O2 -flto

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51263

Richard Guenther  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||DUPLICATE

--- Comment #3 from Richard Guenther  2011-12-06 
13:48:02 UTC ---
Indeed.  My patch fixes this one, too.

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


[Bug middle-end/50823] [4.7 Regression] ICE in inline_small_functions, at ipa-inline.c:1407

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50823

Richard Guenther  changed:

   What|Removed |Added

 CC||matt at use dot net

--- Comment #14 from Richard Guenther  2011-12-06 
13:48:02 UTC ---
*** Bug 51263 has been marked as a duplicate of this bug. ***


[Bug tree-optimization/50744] [4.7 Regression] ice in good_cloning_opportunity_p

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50744

--- Comment #6 from Richard Guenther  2011-12-06 
13:52:58 UTC ---
Also see PR51362 which looks related (is it a dup? size overflowing to zero?)


[Bug middle-end/50913] [4.7 Regression] ICE in scan_tree_for_params_right_scev, at graphite-sese-to-poly.c:633 compiling libgfortran with -floop-interchange -m32

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50913

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1


[Bug tree-optimization/50955] [4.7 Regression] IVopts incorrectly rewrite the address of a global memory access into a local form.

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50955

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1


[Bug rtl-optimization/51069] [4.7 Regression] ICE in verify_loop_structure, at cfgloop.c:1559

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51069

Richard Guenther  changed:

   What|Removed |Added

   Keywords||ice-checking
   Priority|P3  |P1


[Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51094

Richard Guenther  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #22 from Richard Guenther  2011-12-06 
13:58:08 UTC ---
Bootstrap issue fixed.  Open a new bug for anything that is left.


[Bug testsuite/51097] [4.7 Regression] a lot of "FAIL: gcc.dg/vect" on i686 avx build 181167 to 181177

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51097

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P4
  Component|regression  |testsuite
Version|unknown |4.7.0


[Bug libgomp/51132] [4.7 Regression] FAIL: libgomp.graphite/force-parallel-[678].c

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51132

--- Comment #3 from Richard Guenther  2011-12-06 
14:00:06 UTC ---
Can we please lower that stack space requirement?


[Bug fortran/51435] [4.6/4.7 Regression] Bad association status after null() of derived type component

2011-12-06 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51435

Dominique d'Humieres  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2011-12-06
 Ever Confirmed|0   |1

--- Comment #3 from Dominique d'Humieres  2011-12-06 
13:59:21 UTC ---
On x86_64-apple-darwin10, the test always gives

 do_job my_data%head is NOT associated

but valgrind complains as in comment #1.


[Bug c++/51223] [4.5/4.6/4.7 Regression] ICE with invalid function parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51223

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug fortran/51197] [4.7 Regression] Backtrace information less useful

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51197

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P4


[Bug c++/51215] [4.7 Regression] ICE with invalid array initializer

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51215

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug c++/51228] [4.5/4.6/4.7 Regression] ICE with transparent union

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51228

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P2


[Bug target/51135] [4.7 Regression] SIGSEGV during exception cleanup on win32

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51135

Richard Guenther  changed:

   What|Removed |Added

 Target||mingw32
   Priority|P3  |P1
  Component|libstdc++   |target


[Bug c++/51248] [4.6/4.7 Regression] ICE with pointer to enum

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51248

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P2
  Known to work||4.5.3


[Bug c++/51225] [c++0x] [4.7 Regression] ICE with invalid template parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51225

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug c++/51229] [C++0x] [4.7 Regression] Broken diagnostic: 'integer_cst' not supported by dump_dec

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51229

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1
  Known to work||4.6.2

--- Comment #2 from Richard Guenther  2011-12-06 
14:04:27 UTC ---
4.6 prints

> g++-4.6 -S -std=c++0x t.C
t.C:6:10: error: expected identifier before numeric constant
t.C:6:16: error: invalid conversion from ‘void (*)()’ to ‘int’ [-fpermissive]


[Bug debug/51262] [4.7 Regression] ICE: SIGSEGV in primary_template_instantiation_p (pt.c:2874) with -flto -g

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51262

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1


[Bug gcov-profile/51297] [4.7 regression] Many gcov tests FAIL on Tru64, Solaris 8 and 9

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51297

Richard Guenther  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #12 from Richard Guenther  2011-12-06 
14:10:02 UTC ---
Fixed.


[Bug tree-optimization/51315] [4.6/4.7 regression] unaligned memory accesses generated with -ftree-sra

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51315

--- Comment #7 from Richard Guenther  2011-12-06 
14:16:21 UTC ---
Note that in the end it's always us transforming

 a->b.c

to (effectively)

 T *tem = &a->b.c;
 *tem

which expand unfortunately handles differently.  So whenever we do that
we have to either avoid doing that if expand would have a different
idea about the results alignment (there is currently no way that computes
just expands idea of an expressions alignment - one piece of a good
solution would provide that, not only SRA has this "issue"), or, stick
that information somewhere (on the TREE_TYPE of the MEM_REF tree, but
then it will only be honoured if the target provides a movmisalign
optab - the other piece of a good solution would be that expand _always_
honors such alignment information and goes the same way as when expanding
component refs).


[Bug c++/51318] [4.7 Regression] segfault on Eigen3

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51318

Richard Guenther  changed:

   What|Removed |Added

 CC||dcb314 at hotmail dot com

--- Comment #7 from Richard Guenther  2011-12-06 
14:19:48 UTC ---
*** Bug 51384 has been marked as a duplicate of this bug. ***


[Bug c++/51318] [4.7 Regression] segfault on Eigen3

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51318

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1


[Bug c++/51328] [4.4/4.5/4.6/4.7 Regression] ICE on invalid template parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51328

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug c++/51327] [c++0x] [4.7 Regression] ICE with invalid constexpr parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51327

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug target/51354] [4.7 Regression] ICE in maybe_record_trace_start

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51354

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1


[Bug debug/51262] [4.7 Regression] ICE: SIGSEGV in primary_template_instantiation_p (pt.c:2874) with -flto -g

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51262

Richard Guenther  changed:

   What|Removed |Added

 CC||reichelt at gcc dot gnu.org

--- Comment #1 from Richard Guenther  2011-12-06 
14:21:44 UTC ---
*** Bug 51325 has been marked as a duplicate of this bug. ***


[Bug c++/51384] seg fault in tsubst_copy_and_build

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51384

Richard Guenther  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||DUPLICATE

--- Comment #3 from Richard Guenther  2011-12-06 
14:19:48 UTC ---
dup

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


[Bug lto/51325] [4.7 Regression] ICE with -flto -g and union in template

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51325

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1
 Status|NEW |RESOLVED
 Resolution||DUPLICATE
Summary|[4.7 Regression] ICE with   |[4.7 Regression] ICE with
   |-flto and union in template |-flto -g and union in
   ||template

--- Comment #3 from Richard Guenther  2011-12-06 
14:21:43 UTC ---
Dup.

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


[Bug c++/51398] [4.7 Regression] ICE with invalid template parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51398

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug c++/51369] [c++0x] [4.7 Regression] ICE using constexpr in template

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51369

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1
  Known to work||4.6.2


[Bug c++/51403] [4.7 Regression] ICE with invalid template parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51403

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug tree-optimization/51362] [4.7 Regression] ICE: SIGFPE (division by zero) in good_cloning_opportunity_p at ipa-cp.c:2401

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51362

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1


[Bug c++/51431] [4.4/4.5/4.6/4.7 Regression] ICE with invalid use of abstract class

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51431

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug c++/51406] [4.5/4.6/4.7 Regression][c++0x] Incorrect result of static_cast to rvalue reference to base class.

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51406

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P2


[Bug c++/51430] [4.7 Regression] ICE with array as static const member

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51430

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug c++/51370] [4.6/4.7 Regression] ICE with invalid template parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51370

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug gcov-profile/51361] [4.7 Regression] 471.omnetpp of SPEC2006 fails to build with -fprofile-generate

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51361

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P1
 Status|UNCONFIRMED |WAITING
   Last reconfirmed||2011-12-06
 Ever Confirmed|0   |1


[Bug c++/51402] [4.6/4.7 Regression] ICE with invalid template parameter

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51402

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P5


[Bug c++/51416] [c++0x] [4.6/4.7 Regression] ICE with invalid use of auto

2011-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51416

Richard Guenther  changed:

   What|Removed |Added

   Priority|P3  |P2


[Bug libstdc++/51183] pair piecewise_construct_t constructor copies

2011-12-06 Thread chris at bubblescope dot net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51183

--- Comment #16 from Chris Jefferson  2011-12-06 
14:25:24 UTC ---
Created attachment 26006
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26006
Piecewise patch

Patch to make piecewise_construct work properly on std::pair.


[Bug libgomp/51132] [4.7 Regression] FAIL: libgomp.graphite/force-parallel-[678].c

2011-12-06 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51132

--- Comment #4 from Jakub Jelinek  2011-12-06 
14:33:00 UTC ---
Author: jakub
Date: Tue Dec  6 14:32:54 2011
New Revision: 182050

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182050
Log:
PR libgomp/51132
* testsuite/libgomp.graphite/force-parallel-1.c: Move large arrays
to file scope.
* testsuite/libgomp.graphite/force-parallel-3.c: Likewise.
* testsuite/libgomp.graphite/force-parallel-6.c: Likewise.
* testsuite/libgomp.graphite/force-parallel-7.c: Likewise.
* testsuite/libgomp.graphite/force-parallel-8.c: Likewise.
* testsuite/libgomp.graphite/force-parallel-9.c: Likewise.

Modified:
trunk/libgomp/ChangeLog
trunk/libgomp/testsuite/libgomp.graphite/force-parallel-1.c
trunk/libgomp/testsuite/libgomp.graphite/force-parallel-3.c
trunk/libgomp/testsuite/libgomp.graphite/force-parallel-6.c
trunk/libgomp/testsuite/libgomp.graphite/force-parallel-7.c
trunk/libgomp/testsuite/libgomp.graphite/force-parallel-8.c
trunk/libgomp/testsuite/libgomp.graphite/force-parallel-9.c


  1   2   >