Re: [Patch, fortran] Fix temporary allocation for class assignment.

2012-01-15 Thread Tobias Burnus
Dear Paul, Paul Richard Thomas wrote: Please reassure me that class array constructors do not, cannot and will never occur!:-) Do you mean something like the following: ! type t integer :: i = 5 end type t type, extends(t) :: t2 integer :: j = 6 end typ

Re: [Patch,testsuite]: Fix testcase that bangs long and int against void*

2012-01-15 Thread Georg-Johann Lay
Rainer Orth schrieb: Mike Stump writes: On Jan 13, 2012, at 4:33 AM, Georg-Johann Lay wrote: This test case is obviously written for 32-bit platforms, thus added "dg-require-effective-target ilp32" to ensure that the pointer mess won't lead to FAILs because of warning: cast to pointer from

Fix gcc.dg/torture/pr8081.c char signedness assumption

2012-01-15 Thread Andreas Schwab
The condition was always optimized to 1. Tested on powerpc-linux and checked in as obvious. Andreas. 2012-01-15 Andreas Schwab * gcc.dg/torture/pr8081.c: Fix char signedness assumption. Index: gcc.dg/torture/pr8081.c ===

Re: libgo patch committed: Update to weekly.2011-12-22

2012-01-15 Thread Uros Bizjak
On Sat, Jan 14, 2012 at 12:18 AM, Ian Lance Taylor wrote: >> This is the same problem with -fdump-go-spec we discussed a couple of >> months ago [1]. In short, alpha linux doesn't just include >> with hardcoded numbers in asm/ioctls.h, but >> builds ioctl arguments as shown in [1]. Probably, thi

Re: [PATCH, rtl-optimization]: Fix PR 51821, 64bit > 32bit conversion produces incorrect results with optimizations

2012-01-15 Thread Uros Bizjak
On Fri, Jan 13, 2012 at 10:58 AM, Richard Sandiford wrote: The problem my patch solves is the answer to the question "Is the choosen non-live temporary register untouched over the insn sequence?". The answer: "Yes, if it was not set or clobbered by any insn in the sequence". >

Re: [PATCH] MIPS16 TLS support for GCC

2012-01-15 Thread Richard Sandiford
Richard Sandiford writes: > gcc/ > 2012-01-08 Chung-Lin Tang > Richard Sandiford > > * config/mips/mips-protos.h (SYMBOL_FORCE_TO_MEM): Delete. > (SYMBOL_32_HIGH): Likewise. > (mips_output_tls_reloc_directive): Declare. > * config/mips/mips.h (PIC_FUNCTION_AD

[committed] Adjust ssa-dom-thread-4.c for MIPS

2012-01-15 Thread Richard Sandiford
ssa-dom-thread-4.c was failing for MIPS because the mips.h definition: #define LOGICAL_OP_NON_SHORT_CIRCUIT 0 caused "var1 || var2" conditions to be split into two rather than converted into "(var1 != 0) | (var2 != 0)". I don't know whether the MIPS definition still makes sense (probably not for

Re: [patch] Flag-controlled type conversions/promotions

2012-01-15 Thread Zydrunas Gimbutas
Hi, Everything seems to be ok in this patch, except for minor typos in documentation: REAl -> REAL, alignement -> alignment. It would also be nice to extend the type conversion facility to include -fdefault-real-16, -fdefault-double-16, -fdefault-real-10, -fdefault-double-10, and add conversions

Re: [committed] Adjust ssa-dom-thread-4.c for MIPS

2012-01-15 Thread Andrew Pinski
On Sun, Jan 15, 2012 at 10:31 AM, Richard Sandiford wrote: > ssa-dom-thread-4.c was failing for MIPS because the mips.h definition: > > #define LOGICAL_OP_NON_SHORT_CIRCUIT 0 > > caused "var1 || var2" conditions to be split into two rather than > converted into "(var1 != 0) | (var2 != 0)".  I don'

[committed] Fix a MIPS16 hard-float optimisation

2012-01-15 Thread Richard Sandiford
gcc.dg/torture/stackalign/builtin-apply-4.c was failing for MIPS16 hard-float with -O2 -flto -fuse-linker-plugin -fno-fat-lto-objects. We created a specialised out-of-line version of bar() that was called by (rather than inlined into) main(). So we had the equivalent of: static double __attribu

Re: [committed] Adjust ssa-dom-thread-4.c for MIPS

2012-01-15 Thread Andrew Pinski
On Sun, Jan 15, 2012 at 10:45 AM, Andrew Pinski wrote: > On Sun, Jan 15, 2012 at 10:31 AM, Richard Sandiford > wrote: >> ssa-dom-thread-4.c was failing for MIPS because the mips.h definition: >> >> #define LOGICAL_OP_NON_SHORT_CIRCUIT 0 >> >> caused "var1 || var2" conditions to be split into two

Re: [patch] Flag-controlled type conversions/promotions

2012-01-15 Thread Steve Kargl
On Sun, Jan 15, 2012 at 01:39:23PM -0500, Zydrunas Gimbutas wrote: > Hi, > > Everything seems to be ok in this patch, except for minor typos in > documentation: REAl -> REAL, alignement -> alignment. Good catch. > It would also be nice to extend the type conversion facility to > include -fdefaul

[wwwdocs PATCH] for Re: GCC mirror

2012-01-15 Thread Gerald Pfeifer
On Wed, 11 Jan 2012, NetGull Administrator wrote: > I've just set up a new GCC mirror. Information is as follows: > > URL: http://www.netgull.com/gcc/ > Frequency:once a day > Location:San Jose, California, United States, North America > Contact: ad...@netgull.com > > Please, add

Re: useless_type_conversion_p vs pointer sizes

2012-01-15 Thread Eric Botcazou
> So two different sized pointers to aggregate types will also have a > problem? Nope, you misread the test: /* Changes in machine mode are never useless conversions unless we deal with aggregate types in which case we defer to later checks. */ if (TYPE_MODE (inner_type) != TYPE_MODE (o

[PATCH] Fix tree_function_versioning with void type DECL_RESULT (PR tree-optimization/51865)

2012-01-15 Thread Jakub Jelinek
Hi! The recent changes in tree_function_versioning broke the following testcase. If DECL_RESULT exists, but has void type, then we used to remap_decl it before, but now we don't, so in this case the same RESULT_DECL is used in two different functions, which upsets ipa-pta, but could upset other c

Re: [Patch, fortran] Fix temporary allocation for class assignment.

2012-01-15 Thread Paul Richard Thomas
Dear Tobias, The following example that you provided: > Do you mean something like the following: > > ! > type t >  integer :: i = 5 > end type t > type, extends(t) :: t2 >  integer :: j = 6 > end type t2 > > class(t), allocatable :: a(:), b(:), c(:) > allocate

Re: [Patch, fortran] Fix temporary allocation for class assignment.

2012-01-15 Thread Tobias Burnus
Dear Paul, Paul Richard Thomas wrote: > c = [ a, b ] Is exactly the abomination that I did not want to exist! What happens if 'a' and 'b' have different dynamic types? Actually, I start to get lost in the fine prints. One finds: "The dynamic type of an array constructor is the same as its

Re: [patch] [4.6] Backport strict-volatile-bitfields fix PR51200

2012-01-15 Thread Ye Joey
Ping On Tue, Dec 27, 2011 at 10:19 AM, Ye Joey wrote: > Fix PR51200. Backport trunk 182545, 182649, 182685 to 4.6. > > OK to 4.6? > > - Joey > >        2011-12-20  Bernd Schmidt   > >        PR middle-end/51200 >        * expr.c (store_field): Avoid a direct store if the mode is larger >        t