Re: [Patch] Adjust diag-scans in vect-tests to fix fails on AVX/AVX2

2011-12-11 Thread Jakub Jelinek
On Mon, Dec 12, 2011 at 11:06:37AM +0400, Michael Zolotukhin wrote: diff --git a/gcc/testsuite/gcc.dg/vect/no-section-anchors-vect-31.c b/gcc/testsuite/gcc.dg/vect/no-section-anchors-vect-31.c index 21b87a3..f75253e 100644 --- a/gcc/testsuite/gcc.dg/vect/no-section-anchors-vect-31.c +++ b/gcc/test

Re: warn about deprecated access declarations

2011-12-11 Thread Fabien Chêne
2011/12/11 Jonathan Wakely : > On 11 December 2011 22:22, Fabien Chêne wrote: >> >> Consequently, I propose to deprecate them with a warning, as clang already >> does. >> So that you get a warning for the following code: >> >> struct A { int i; }; >> struct B : A >> { >>  A::i; // <- warning here

[Patch] Adjust diag-scans in vect-tests to fix fails on AVX/AVX2

2011-12-11 Thread Michael Zolotukhin
Hi, This patch fixes dg-final scans in tests from vect.exp suite, which currently fail when avx2 is used. Ok for trunk? Changelog: 2011-12-12 Michael Zolotukhin * gcc.dg/vect/no-section-anchors-vect-31.c: Adjust diagnostic test to fix fail on AVX. * gcc.dg/vect/no-s

Re: [PATCH, SMS] Add missing free operation in mark_loop_unsched

2011-12-11 Thread Revital Eres
Hello, > OK for 3.7? Sorry, I meant GCC 4.7.0... Thanks, Revital

[PATCH, SMS] Add missing free operation in mark_loop_unsched

2011-12-11 Thread Revital Eres
Hello, The patch below adds a missing free operation in mark_loop_unsched. Tested (bootstrap and regtest) ppc64-redhat-linux. OK for 3.7? Thanks, Revital Changelog: * modulo-sched.c (mark_loop_unsched): Free bbs. Index: modulo-sched.c

Re: [google] Backport r171347 and r181549 from trunk (strict volatile bitfield) (issue5434084)

2011-12-11 Thread 關振德
Sorry about my oversight. I am on vacation now until Dec 19. I don't have good internet access now and I will backport this to upstream 4.6 after I come back if the 4.6 maintainers agree to take this. -Doug On Wed, Dec 7, 2011 at 9:02 PM, Richard Guenther wrote: > On Wed, Dec 7, 2011 at 1:34 PM

[google] Add support for delete operator that takes the size of the object as a parameter

2011-12-11 Thread Easwaran Raman
In the tcmalloc memory allocator(http://google-perftools.googlecode.com/svn/trunk/doc/tcmalloc.html), deallocation involves a costly lookup to get the size of the deleted object. The size is required to find the right free list to release back the object. By passing the size of the object to the d

Re: [PATCH 5/6] mips: Implement vec_perm_const.

2011-12-11 Thread Hans-Peter Nilsson
On Sun, 11 Dec 2011, Richard Sandiford wrote: > Hans-Peter Nilsson writes: > > Please also consider incrementing __mips_loongson_vector_rev > For avoidance of doubt, that only applies to the latter ("as H-P > suggests") option. The patch as posted keeps the public interface > the same. Correct;

Re: PR/50076 make c-c++-common/cxxbitfields-3.c work in Darwin

2011-12-11 Thread Mike Stump
On Dec 9, 2011, at 11:45 AM, Aldy Hernandez wrote: > How about the patch below? I'm fine with whatever you guys come up with...

Re: warn about deprecated access declarations

2011-12-11 Thread Jonathan Wakely
On 11 December 2011 22:22, Fabien Chêne wrote: > > Consequently, I propose to deprecate them with a warning, as clang already > does. > So that you get a warning for the following code: > > struct A { int i; }; > struct B : A > { >  A::i; // <- warning here > }; > > warning: access declarations ar

warn about deprecated access declarations

2011-12-11 Thread Fabien Chêne
Hi, According to § 11.3/1 from c++98, access delarations are deprecated: The access of a member of a base class can be changed in the derived class by mentioning its qualified-id in the derived class declaration. Such mention is called an access declaration. The effect of an access declaration qu

Re: [committed] 4 backports from trunk to 4.6 branch

2011-12-11 Thread Jakub Jelinek
On Sun, Dec 11, 2011 at 02:48:52PM +0100, Mikael Pettersson wrote: > This patch, r182112 on 4.6 branch, caused a test suite regression on > arm-linux-gnueabi: > > +FAIL: gcc.c-torture/execute/20050713-1.c compilation, -O2 (internal > compiler error) > +UNRESOLVED: gcc.c-torture/execute/2005071

Re: [Patch, fortran] - Arrays of classes for fortran

2011-12-11 Thread Paul Richard Thomas
Dear Tobias, On Sun, Dec 11, 2011 at 7:39 PM, Tobias Burnus wrote: > Dear Paul, dear all, > > first, thanks again for the patch. Thank you for the continuous reviewing over the last couple of months - also to Dominique, Salvatore and Damian; all of whom have kept the test cases coming in. > > >

Re: [Patch, Fortran] PR50923 - [4.4-4.7] Print warning function does not return a value

2011-12-11 Thread Paul Richard Thomas
Dear Tobias, >> Build and regtested on x86-64-linux. >> OK for the trunk? How far should this be backported - all the way down to >> 4.4? This is OK for trunk and... well, I don't know. 4.4 is likely a bit too far. I guess that the linux distros are already at 4.5? I will leave it to you. Ch

Re: [PATCH 5/6] mips: Implement vec_perm_const.

2011-12-11 Thread Richard Henderson
On 12/11/2011 04:50 AM, Richard Sandiford wrote: > [Mingjie, please could you help with the Loongson question near the end?] Actually, can you tell me how to test these abi combinations? I keep trying to use mips-sim or mips64-sim and get linker errors complaining of abi combinations. > L

Re: [PATCH 4/6] rs6000: Cleanup interleave/even_odd/vec_perm.

2011-12-11 Thread Richard Henderson
On 12/11/2011 10:44 AM, Iain Sandoe wrote: > Note that there is no default for TARGET_BIG_ENDIAN in rs600.h - it is only > defined in config/rs6000/sysv4.h. Gah. And of course that's all I tested w/ linux. Just change the TARGET_BIG_ENDIAN to BYTES_BIG_ENDIAN. r~

Re: [PATCH 4/6] rs6000: Cleanup interleave/even_odd/vec_perm.

2011-12-11 Thread Iain Sandoe
On 8 Dec 2011, at 21:06, Richard Henderson wrote: ; + + high = (highp == TARGET_BIG_ENDIAN ? 0 : nelt); + This breaks bootstrap on ppc-darwin, which came as a bit f a surprise to me... ... however, it would appear that TARGET_BIG_ENDIAN is not used anywhere else... (plenty of {BYTES,

Re: [Patch, fortran] - Arrays of classes for fortran

2011-12-11 Thread Tobias Burnus
Dear Paul, dear all, first, thanks again for the patch. Paul Richard Thomas wrote: Boostrapped and regtested on x86_64/FC9 - OK for trunk? I have now re-read the patch and it is OK from my side. It wouldn't harm is someone else with experience with CLASS or with the scalarizer could also re

[patch] PR51388

2011-12-11 Thread Steven Bosscher
Hello, The configure scripts check for -Wno-narrowing, but GCC ignores rather than rejects unknown -Wno-* warnings. Fixed by checking for the positive warning, -Wnarrowing. OK for trunk? Ciao! Steven gcc/ PR bootstrap/51388 configure.ac: Check for Wnarrowing instead of Wno-narro

Re: [patch] add __is_final trait to fix libstdc++/51365

2011-12-11 Thread Jonathan Wakely
ping On 3 December 2011 12:04, Jonathan Wakely wrote: > On 3 December 2011 12:00, Jonathan Wakely wrote: >> This implements a new C++ trait, __is_final, to query the 'final' >> specifier that 4.7 supports. The trait is needed for the library so we >> can detect when it's not possible to derive fro

Re: [libstdc++] doc/xml/manual/abi.xml -- fix references to GCC as well as GNU/Linux

2011-12-11 Thread Jonathan Wakely
On 5 December 2011 00:04, Jonathan Wakely wrote: > > Yep, here's another patch with some more duplication removed.  WIth > this, the document only needs to be updated when a new symbol version > is added or a library filename changes, not for every point release > with identical library versions.  

Re: [patch] PR51347 alias problem

2011-12-11 Thread Patrick Marlier
On 12/11/2011 09:11 AM, Aldy Hernandez wrote: On 12/10/11 17:13, Patrick Marlier wrote: On 12/10/2011 02:16 PM, Aldy Hernandez wrote: Using the parent node for aliases (as in your patch) makes sense, but I don't see tree_function_versioning() segfaulting as you claim. What I see is estimate_fun

Re: [PATCH 5/6] mips: Implement vec_perm_const.

2011-12-11 Thread Richard Sandiford
Hans-Peter Nilsson writes: > On Sun, 11 Dec 2011, Richard Sandiford wrote: >> [Mingjie, please could you help with the Loongson question near the end?] > >> As H-P mentioned, this changes the __builtin_* interface for the PSHUFH >> intrinsics. These intrinsics are supposed to be used via the inli

Re: [patch] PR51347 alias problem

2011-12-11 Thread Aldy Hernandez
On 12/10/11 17:13, Patrick Marlier wrote: On 12/10/2011 02:16 PM, Aldy Hernandez wrote: Using the parent node for aliases (as in your patch) makes sense, but I don't see tree_function_versioning() segfaulting as you claim. What I see is estimate_function_body_sizes() Humm you are seeing this I

[C++ Patch] for c++/14258 (aka using typename)

2011-12-11 Thread Fabien Chêne
Hi, Here, we weren't creating a typename_type for a dependent type introduced by a using declaration. A USING_DECL was not recording the fact that it refers to a dependent type, so I've created a new macro USING_DECL_TYPENAME_P to record that information (using the free slot DECL_LANG_FLAG_1 for U

Re: [PATCH 5/6] mips: Implement vec_perm_const.

2011-12-11 Thread Hans-Peter Nilsson
On Sun, 11 Dec 2011, Richard Sandiford wrote: > [Mingjie, please could you help with the Loongson question near the end?] > As H-P mentioned, this changes the __builtin_* interface for the PSHUFH > intrinsics. These intrinsics are supposed to be used via the inline > wrappers in loongson.h, so we

Re: [committed] 4 backports from trunk to 4.6 branch

2011-12-11 Thread Mikael Pettersson
Jakub Jelinek writes: > Hi! > > Bootstrapped/regtested on x86_64-linux and i686-linux, committed to 4.6 > branch: ... > 2011-12-08 Jakub Jelinek > > Backport from mainline > 2011-12-05 Jakub Jelinek > Eric Botcazou > > PR middle-end/51323 >

Re: [committed] Another 4.6 backport (PR tree-optimization/50078)

2011-12-11 Thread Mikael Pettersson
Jakub Jelinek writes: > Hi! > > Bootstrapped/regtested on x86_64-linux and i686-linux, committed to 4.6 > branch. > > 2011-12-09 Jakub Jelinek > > Backport from mainline > 2011-12-08 Jakub Jelinek > > PR tree-optimization/51466 > * tree-ssa-forwprop.c (forw

Re: [patch, fortran] Fix PR 50690

2011-12-11 Thread Tobias Burnus
Thomas Koenig wrote: Regression-tested. OK for trunk (finally)? Looks also OK from my side - except for the left over which Jakub has already noticed. I think the following lines can be simply removed: - case EXEC_OMP_DO: + in_omp_workshare = true; + + brea

Re: [PATCH 5/6] mips: Implement vec_perm_const.

2011-12-11 Thread Richard Sandiford
[Mingjie, please could you help with the Loongson question near the end?] Richard Henderson writes: > @@ -89,61 +89,102 @@ >DONE; > }) > > -; pul.ps - Pair Upper Lower > -(define_insn "mips_pul_ps" > +(define_insn "vec_perm_const_ps" >[(set (match_operand:V2SF 0 "register_operand" "=f"

[Ada] Fix small elaboration regression

2011-12-11 Thread Eric Botcazou
We now generate elaboration code to initialize an aggregate that contains a null access-to-unconstrained-array value. Fixed thusly, tested on i586-suse-linux, applied on the mainline. 2011-12-11 Eric Botcazou * gcc-interface/utils2.c (gnat_build_constructor): Test the TREE_STATIC

[Ada] Leverage 6.2(12) to restrict parameter aliasing

2011-12-11 Thread Eric Botcazou
The 6.2(12) paragraph reads: "If one name denotes a part of a formal parameter, and a second name denotes a part of a distinct formal parameter or an object that is not part of a formal parameter, then the two names are considered distinct access paths. If an object is of a type for which the

[Ada] Assertion to detect conflicting alignments in renaming

2011-12-11 Thread Eric Botcazou
As suggested by Robert, it would be valuable to have an assertion in gigi to detect conflicting alignments in renaming, i.e. when an alignment value set on the renaming cannot be honored because the renamed object isn't sufficiently aligned. Tested on i586-suse-linux, applied on the mainline.

Re: [patch, fortran] Fix PR 50690

2011-12-11 Thread Jakub Jelinek
On Sun, Dec 11, 2011 at 11:11:28AM +0100, Thomas Koenig wrote: > @@ -1330,16 +1345,38 @@ gfc_code_walker (gfc_code **c, walk_code_fn_t code > WALK_SUBEXPR (co->ext.dt->extra_comma); > break; > > - case EXEC_OMP_DO: > + in_omp_workshare = true; > + > +

Re: [patch, fortran] Fix PR 50690

2011-12-11 Thread Thomas Koenig
Am 08.12.2011 22:57, schrieb Jakub Jelinek: Reading it again, isn't it overkill to keep the vector? All you need is a bool and a way to restore its previous state. Good catch. The vector was a leftover from the time when I was searching up the call chain to check for any enclosing workshare.

[Ada] Small tweak to gigi

2011-12-11 Thread Eric Botcazou
This moves the initialization of the linemap up to before the first assertion at the beginning of the gigi routine. As experienced under PR ada/49084, if the assertion fails and the linemap isn't initialized yet, things can go awry. Tested on i586-suse-linux, applied on the mainline. 2011-12-