Re: [PATCH] libstdc++: Add mem_order_hle_acquire/release to atomic.h v2

2013-01-14 Thread Jonathan Wakely
On 14 January 2013 06:06, Andi Kleen wrote: The underlying compiler supports additional __ATOMIC_HLE_ACQUIRE/RELEASE memmodel flags for TSX, but this was not exposed to the C++ wrapper. Handle it there. These are additional flags, so some of assert checks need to mask off the flags before

[PATCH] Do not allow sharing of handled_component_p's

2013-01-14 Thread Richard Biener
For some reason they were allowed, which is definitely not a good idea (they are unshared by all regular unsharing). Direct fallout is minor - it's the graphite changes which makes sure not to share a[0]. The rest of the adjustments are for bugs sharing ADDR_EXPRs (which is also bad as that has

Re: [patch, Fortran] PR 55806 - Inefficient ANY with array constructors

2013-01-14 Thread Mikael Morin
Le 13/01/2013 23:14, Thomas Koenig a écrit : Hi Mikael, thanks a lot for your comments! + actual_arglist-expr = gfc_copy_expr (e); + actual_arglist-next = gfc_get_actual_arglist (); Another one is needed. I get a segmentation fault with SUM. Fixed by using gfc_build_intrisic_call. Nice.

Re:[PING] [Patch, AArch64] Implement support for LD1R.

2013-01-14 Thread Tejas Belagod
PING. Tejas Belagod wrote: Hi, Attached is a patch that implements support for AdvSIMD instruction LD1R. Tested on aarch64-none-elf. OK for trunk? Thanks, Tejas Belagod ARM. 2013-01-09 Tejas Belagod tejas.bela...@arm.com gcc/ * config/aarch64/aarch64-simd.md

Re: [PING][Patch, AArch64-4.7] Implement support for LD1R.

2013-01-14 Thread Tejas Belagod
PING. Tejas Belagod wrote: Hi, Attached is a patch that implements support for AdvSIMD instruction LD1R. Tested on aarch64-none-elf. OK to commit on aarch64-4.7-branch? Thanks, Tejas Belagod ARM. 2013-01-09 Tejas Belagod tejas.bela...@arm.com gcc/ * config/aarch64/aarch64-simd.md

Re: testsuite: missing or wrong dg-* directives?

2013-01-14 Thread Mikael Morin
Le 13/01/2013 21:30, Harald Anlauf a écrit : On 01/12/13 22:02, Mikael Morin wrote: Le 08/01/2013 22:32, Harald Anlauf a écrit : On 12/28/12 21:49, Harald Anlauf wrote: Hello all, is there a default directive that is assumed when the testsuite is run? Running an fgrep -L on the fortran

Re: [PATCH] PR c++/55663 - constexpr function templ instantiation considered non-const as alias templ arg

2013-01-14 Thread Jason Merrill
OK. Jason

Re: testsuite: missing or wrong dg-* directives?

2013-01-14 Thread Mikael Morin
Le 14/01/2013 00:37, Manfred Schwarb a écrit : Am 14.01.2013 00:10, schrieb Manfred Schwarb: There are several other oddities: d_g-final, braces have to be separated by spaces. Want to send a patch? Not sure about the double braces in lto, I guess they act simply as single braces. I

[PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Richard Biener
This makes us accept the CLooG 0.18.0 and ISL 0.11.1 combo. It's probably not the best stage to move the version checks to gcc/ where we can rely on built in-tree ISL/CLooG, so this avoids it with the caveat that in-tree CLooG 0.18.0 will fail the version check (they no longer ship built

Re: RFA: Update copyright for libcpp

2013-01-14 Thread Tom Tromey
Richard == Richard Sandiford rdsandif...@googlemail.com writes: Richard Here's the result of running the copyright script on libcpp/. Richard It's nice and short, but for consistency with the other patches, Richard I've not listed out every file in the changelog. Richard Tested on

Re: [RFC, middlend] Fix for PR54218

2013-01-14 Thread Richard Biener
On Fri, Jan 11, 2013 at 6:37 PM, George Thomas georgethomas@gmail.com wrote: On Fri, Jan 11, 2013 at 9:53 PM, Andrew Pinski pins...@gmail.com wrote: On Fri, Jan 11, 2013 at 8:17 AM, George Thomas georgethomas@gmail.com wrote: Hi, I am sending a patch which solves the debugging issue

[PATCH] Fix fixinclude common/non-common mix

2013-01-14 Thread Richard Biener
fixincludes mixes common declarations (in fixlib.h) and non-common decls (where they are defined) for verbose_level and fixinc_mode. XCode on Mountain Lion doesn't like this and its linker (rightfully) complains. Fixed with the following, committed as obvious. Richard. 2013-01-14 Richard

Re: RFA: Allow widening multiplication in tree-ssa/slsr-8.c

2013-01-14 Thread Richard Biener
On Sun, Jan 13, 2013 at 9:50 AM, Richard Sandiford rdsandif...@googlemail.com wrote: gcc.dg/tree-ssa/slsr-8.c does (int) index * (size_t) element_size, which on LP64 targets can use a widening multiplication. Of course, for simple element sizes like these, we end up using a shift instead, but

Re: [PATCH] Fix PR55876 - Make generation of paradoxical subreg in widen_operand more robust

2013-01-14 Thread Tom de Vries
On 07/01/13 18:48, Richard Henderson wrote: On 01/07/2013 02:16 AM, Tom de Vries wrote: - /* If MODE is no wider than a single word, we return a paradoxical - SUBREG. */ + /* If MODE is no wider than a single word, we return a + lowpart or paradoxical SUBREG. */ if

Re: [PATCH 1/2] Document HLE / RTM intrinsics

2013-01-14 Thread Richard Biener
On Sat, Jan 12, 2013 at 6:20 PM, Andi Kleen a...@firstfloor.org wrote: On Sat, Jan 12, 2013 at 06:04:19PM +0100, Bernhard Reutner-Fischer wrote: On 12 January 2013 16:28:41 Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com +Returns _XBEGIN_STARTED when the

Re: [PATCH] Fix fixinclude common/non-common mix

2013-01-14 Thread Bruce Korb
Hi, On Mon, Jan 14, 2013 at 12:42 AM, Richard Biener rguent...@suse.de wrote: fixincludes mixes common declarations (in fixlib.h) and non-common decls (where they are defined) for verbose_level and fixinc_mode. XCode on Mountain Lion doesn't like this and its linker (rightfully) complains.

[PATCH] Location verifier

2013-01-14 Thread Richard Biener
This adds a verification that we do not refer to BLOCKs not in the functions BLOCK tree from locations. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2013-01-14 Richard Biener rguent...@suse.de * tree-cfg.c (verify_expr_location, verify_expr_location_1,

[4.7/4.6] Backport typo fix

2013-01-14 Thread Eric Botcazou
Having vect_get_single_scalar_iteraion_cost is really not pretty. This was fixed on the mainline only (r188611) so this patch completes the work. Tested on x86-64/Linux, applied on the 4.7 branch and 4.6 branch as obvious. 2013-01-14 Eric Botcazou ebotca...@adacore.com *

[avr,committed] Some no-op changes: Typo, Add copyright notices, Coding rules, Trailing blanks, etc.

2013-01-14 Thread Georg-Johann Lay
Added copyright notices (avr-lib.h, avr-modes.def): http://gcc.gnu.org/viewcvs?view=revisionrevision=195143 http://gcc.gnu.org/viewcvs?view=revisionrevision=195145 Fix typos, remove trailing blanks, coding rules, ... http://gcc.gnu.org/viewcvs?view=revisionrevision=195151

[PATCH] Let target get_memmodel know about stores/barriers

2013-01-14 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com For HLE stores are only valid with __ATOMIC_HLE_RELEASE. The middle end didn't know this. This adds a new parameter to the get_memmodel target hook to distingush stores and give an warning for acquire stores. I also did a similar check for the barriers where

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Tobias Grosser
On 01/14/2013 03:29 PM, Richard Biener wrote: This makes us accept the CLooG 0.18.0 and ISL 0.11.1 combo. It's probably not the best stage to move the version checks to gcc/ where we can rely on built in-tree ISL/CLooG, so this avoids it with the caveat that in-tree CLooG 0.18.0 will fail the

[PATCH] Fix PR55792

2013-01-14 Thread Richard Biener
This fixes LTO profiledbootstrap by pruning locations at tree_function_versioning time - there seems to be at least one uncatched case in IPA opts. LTO profilebootstrapped and tested on x86_64-unknown-linux-gnu. I also had the extended lto-cgraph.c verification in-tree but do not plan to commit

Re: [RFC PATCH] Implementing ifunc target hook

2013-01-14 Thread Alexander Ivchenko
thank you very much for your review! I fixed the arm build and all other issues that you raised. the patch is attached. Bootstrap and tested on x86-64 linux Alexander 2013/1/11 Maxim Kuvyrkov maxim.kuvyr...@gmail.com: On 10/01/2013, at 12:24 AM, Alexander Ivchenko wrote:

[PATCH][RFC] Fix PR55964

2013-01-14 Thread Richard Biener
I happen to have a patch for PR55964 around - preparatory work to make loop distribution (and vectorization) handle nested loops. It mostly kills the broken way loop distribution copies loops (which is where we ICE in this PR). So instead of trying to make that old logic slightly less broken I

[patch] fix relinking libasan and libtsan on installation

2013-01-14 Thread Matthias Klose
with the recent libsanitizer update, libasan and libtsan got a dependency on libstdc++.so.6, however when installing into an empty directory, and libstdc++.so.6 isn't installed there first, both libasan and libtsan are relinked against the system libstdc++.so.6. So make sure that libstdc++-v3 is

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Jack Howarth
On Mon, Jan 14, 2013 at 04:44:04PM +0100, Tobias Grosser wrote: On 01/14/2013 03:29 PM, Richard Biener wrote: This makes us accept the CLooG 0.18.0 and ISL 0.11.1 combo. It's probably not the best stage to move the version checks to gcc/ where we can rely on built in-tree ISL/CLooG, so this

Re: [patch] fix relinking libasan and libtsan on installation

2013-01-14 Thread Jakub Jelinek
On Mon, Jan 14, 2013 at 05:01:28PM +0100, Matthias Klose wrote: 2013-01-13 Matthias Klose d...@ubuntu.com * Makefile.def (install-target-libsanitizer): Depend on install-target-libstdc++-v3. * Makefile.in: Regenerate. Okay. --- Makefile.def (Revision 195136) +++

Re: [lra] resync with trunk

2013-01-14 Thread Vladimir Makarov
On 13-01-11 5:39 PM, Steven Bosscher wrote: Hello Vlad, The attached patch back-ports to the LRA branch (or foward-ports, depending on how you look at it :-) all the changes made on the trunk to LRA. Bootstrappedtested on powerpc64-unknown-linux-gnu[1] and ia64-unknown-linux-gnu[2], and

Re: [PATCH] Fix PR55833 + cheaper checking

2013-01-14 Thread Marek Polacek
On Thu, Jan 10, 2013 at 11:19:43PM +0100, Zdenek Dvorak wrote: I agree -- at the very least, unswitch_single_loop should check whether there is any possiblity it could have affected irreducible loops information (this should only be the case when some already existing irreducible loop is

Re: Mention of gcc 4.5 as first version with plugin support

2013-01-14 Thread Dave Johansen
On Mon, Jan 14, 2013 at 12:21 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Jan 11, 2013 at 9:46 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: On Fri, 11 Jan 2013 09:58:49 +0100 Dodji Seketeli do...@seketeli.org wrote: Thank you Dave for this documentation patch

Re: ping Re: [patch] multiarch backports for the 4.7 and 4.6 branches

2013-01-14 Thread Jakub Jelinek
On Mon, Jan 14, 2013 at 05:09:36PM +0100, Matthias Klose wrote: ping on the backports, attaching two new versions which do include the powerpcspe and kfreebsd fixes from 2012-12-19/20. Ok, but please watch for fallouts. 2012-11-20 Matthias Klose d...@ubuntu.com * doc/invoke.texi:

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Jack Howarth
On Mon, Jan 14, 2013 at 11:03:41AM -0500, Jack Howarth wrote: On Mon, Jan 14, 2013 at 04:44:04PM +0100, Tobias Grosser wrote: On 01/14/2013 03:29 PM, Richard Biener wrote: This makes us accept the CLooG 0.18.0 and ISL 0.11.1 combo. It's probably not the best stage to move the version

Re: [PATCH 2/2] Support __ATOMIC_HLE_RELEASE for __atomic_clear/store_n

2013-01-14 Thread Uros Bizjak
On Sun, Jan 13, 2013 at 11:29 PM, Andi Kleen a...@firstfloor.org wrote: On Sun, Jan 13, 2013 at 11:23:24PM +0100, Uros Bizjak wrote: On Sun, Jan 13, 2013 at 11:12 PM, Andi Kleen a...@firstfloor.org wrote: +(define_insn atomic_storemode_1 + [(set (match_operand:ATOMIC 0 memory_operand =m)

Re: [PATCH] Let target get_memmodel know about stores/barriers

2013-01-14 Thread Uros Bizjak
On Mon, Jan 14, 2013 at 4:38 PM, Andi Kleen a...@firstfloor.org wrote: For HLE stores are only valid with __ATOMIC_HLE_RELEASE. The middle end didn't know this. This adds a new parameter to the get_memmodel target hook to distingush stores and give an warning for acquire stores. I also did a

Re: [Patch, AArch64-4.7] Implement support for LD1R.

2013-01-14 Thread Marcus Shawcroft
On 09/01/13 16:36, Tejas Belagod wrote: Hi, Attached is a patch that implements support for AdvSIMD instruction LD1R. Tested on aarch64-none-elf. OK to commit on aarch64-4.7-branch? Thanks, Tejas Belagod ARM. 2013-01-09 Tejas Belagod tejas.bela...@arm.com gcc/ *

Re: [Patch, AArch64] Implement support for LD1R.

2013-01-14 Thread Marcus Shawcroft
On 09/01/13 16:41, Tejas Belagod wrote: Hi, Attached is a patch that implements support for AdvSIMD instruction LD1R. Tested on aarch64-none-elf. OK for trunk? Thanks, Tejas Belagod ARM. 2013-01-09 Tejas Belagod tejas.bela...@arm.com gcc/ * config/aarch64/aarch64-simd.md

[patch,avr]:

2013-01-14 Thread Georg-Johann Lay
This patch is a no-op change that uses typedefs starting with avr_ in order to make clear that they are in the avr namespace. Old types were: struct base_arch_s, struct arch_info_s, struct mcu_type_s. avr-cross builds fine. Ok to apply? * config/avr/avr-arch.h (struct

RE: [PATCH][ARM][1/3] Add vectorization support for rounding functions

2013-01-14 Thread Kyrylo Tkachov
Ping. Thanks, Kyrill -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov Sent: 07 January 2013 10:35 To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][1/3] Add

RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions

2013-01-14 Thread Kyrylo Tkachov
Ping. Thanks, Kyrill -Original Message- From: Kyrylo Tkachov Sent: 07 January 2013 10:35 To: Kyrylo Tkachov; gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: RE: [PATCH][ARM][2/3] Add vectorization support for rounding functions Ping.

RE: [patch,avr]:

2013-01-14 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay Sent: Monday, January 14, 2013 10:31 AM To: gcc-patches@gcc.gnu.org Cc: Denis Chertykov; Weddington, Eric Subject: [patch,avr]: This patch is a no-op change that uses typedefs starting with avr_ in order to make clear that they are in

Re: [PATCH] Fix PR55833 + cheaper checking

2013-01-14 Thread Zdenek Dvorak
Hi, On Thu, Jan 10, 2013 at 11:19:43PM +0100, Zdenek Dvorak wrote: I agree -- at the very least, unswitch_single_loop should check whether there is any possiblity it could have affected irreducible loops information (this should only be the case when some already existing irreducible

[patch,avr]: Minor fixes to some built-in macros

2013-01-14 Thread Georg-Johann Lay
This patch does: 1) With 8-bit int, __INT24_MAX__ and __UINT24_MAX__ need a long long suffix. 2) Defines like __FLASH that indicate if address space is available should be like a predicate, i.e. 1 if the space is available. Obvious and lightly tested. Ok? Johann *

[Patch, AArch64, AArch64-4.7] Fix sqrdmulhq_laneq_* intrinsics.

2013-01-14 Thread Tejas Belagod
Hi, Attached is a patch that fixes sqrdmulhq_lane_* intrinsics. Previously they, used to accept 128-bit lane index range. This fixes this bug to accept 64-bit lane index range. sqrdmulhq_laneq_* and AdvSIMD scalar ones still accept 128-bit lane index range as before. It has passed

Re: [PATCH 2/2] Support __ATOMIC_HLE_RELEASE for __atomic_clear/store_n

2013-01-14 Thread Andi Kleen
This cannot happen, we reject code that sets both __HLE* flags. Good thanks. BTW I found more HLE bugs, it looks like some of the fetch_op_* patterns do not match always and fall back to cmpxchg, which does not generate HLE code correctly. Not fully sure what's wrong, can you spot any obvious

RE: [patch,avr]: Minor fixes to some built-in macros

2013-01-14 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay Sent: Monday, January 14, 2013 10:52 AM To: gcc-patches@gcc.gnu.org Cc: Denis Chertykov; Weddington, Eric Subject: [patch,avr]: Minor fixes to some built-in macros This patch does: 1) With 8-bit int, __INT24_MAX__ and __UINT24_MAX__

Re: [patch,avr]: Minor fixes to some built-in macros

2013-01-14 Thread Georg-Johann Lay
Weddington, Eric wrote: This patch does: 1) With 8-bit int, __INT24_MAX__ and __UINT24_MAX__ need a long long suffix. 2) Defines like __FLASH that indicate if address space is available should be like a predicate, i.e. 1 if the space is available. I'm not seeing #2 being done in

RE: [patch,avr]: Minor fixes to some built-in macros

2013-01-14 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay Sent: Monday, January 14, 2013 11:15 AM To: Weddington, Eric Cc: gcc-patches@gcc.gnu.org; Denis Chertykov Subject: Re: [patch,avr]: Minor fixes to some built-in macros Weddington, Eric wrote: This patch does: 1) With 8-bit int,

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Richard Biener
Jack Howarth howa...@bromo.med.uc.edu wrote: On Mon, Jan 14, 2013 at 11:03:41AM -0500, Jack Howarth wrote: On Mon, Jan 14, 2013 at 04:44:04PM +0100, Tobias Grosser wrote: On 01/14/2013 03:29 PM, Richard Biener wrote: This makes us accept the CLooG 0.18.0 and ISL 0.11.1 combo. It's

Re: [PATCH 1/2] Document HLE / RTM intrinsics

2013-01-14 Thread Andi Kleen
I think (somewhere else) we agreed to only document intrinsics, not the __builtin_ia32_ variants (they are an implementation detail). They are all (poorly) documented. I didn't really document them, just list them. Yes, we're not consistent with that, but we do miss a lot of documentation

[patch, libstdc++, testsuite] Fix simulator timouts.

2013-01-14 Thread Steve Ellcey
Here is a patch for the last of the C++ library tests that fail for me when run under the gnu simulator due to memory usage. I am using the same define mechanism to shrink them when run on a simulator as other tests are already using. Tested on mips-mti-elf target with no regressions. OK to

Re: [PATCH 2/2] Support __ATOMIC_HLE_RELEASE for __atomic_clear/store_n

2013-01-14 Thread Uros Bizjak
On Mon, Jan 14, 2013 at 7:06 PM, Andi Kleen a...@firstfloor.org wrote: This cannot happen, we reject code that sets both __HLE* flags. BTW I found more HLE bugs, it looks like some of the fetch_op_* patterns do not match always and fall back to cmpxchg, which does not generate HLE code

Re: Support for MIPS r5900

2013-01-14 Thread Maciej W. Rozycki
Hi Jürgen, Now if that failed for you, then it's a plain bug in GAS that should be fixed. Can you therefore check whether a piece like: .setmips2 ll $2, ($3) assembles correctly with -march=r5900? This seems to work. I didn't know that this would work. I

Re: [PATCH 2/2] Support __ATOMIC_HLE_RELEASE for __atomic_clear/store_n

2013-01-14 Thread Andi Kleen
On Mon, Jan 14, 2013 at 07:40:56PM +0100, Uros Bizjak wrote: On Mon, Jan 14, 2013 at 7:06 PM, Andi Kleen a...@firstfloor.org wrote: This cannot happen, we reject code that sets both __HLE* flags. BTW I found more HLE bugs, it looks like some of the fetch_op_* patterns do not match always

[wwwdocs] Mention MIPS changes for 4.8.

2013-01-14 Thread Richard Sandiford
Update the MIPS section of the 4.8 release notes. Please let me know if I missed anything, or if you think the entries should be changed. Committed. Richard Index: htdocs/gcc-4.8/changes.html === RCS file:

Re: [PATCH] Fix PR55833 + cheaper checking

2013-01-14 Thread Marek Polacek
On Mon, Jan 14, 2013 at 06:51:54PM +0100, Zdenek Dvorak wrote: Hi, On Thu, Jan 10, 2013 at 11:19:43PM +0100, Zdenek Dvorak wrote: I agree -- at the very least, unswitch_single_loop should check whether there is any possiblity it could have affected irreducible loops information

Re: [PATCH 2/2] Support __ATOMIC_HLE_RELEASE for __atomic_clear/store_n

2013-01-14 Thread Uros Bizjak
On Mon, Jan 14, 2013 at 8:01 PM, Andi Kleen a...@firstfloor.org wrote: This cannot happen, we reject code that sets both __HLE* flags. BTW I found more HLE bugs, it looks like some of the fetch_op_* patterns do not match always and fall back to cmpxchg, which does not generate HLE code

Re: [PATCH 2/2] Support __ATOMIC_HLE_RELEASE for __atomic_clear/store_n

2013-01-14 Thread Uros Bizjak
On Mon, Jan 14, 2013 at 8:01 PM, Andi Kleen a...@firstfloor.org wrote: Well the C++ test cases I wrote didn't work. It may be related to how complex the program is. Simple calls as in the original test suite seem to work. e.g. instead of xacquire lock and ... it ended up with a cmpxchg loop

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Dominique Dhumieres
In order to bootstrap r195167 with the new ISL/CLooG versions, I had to apply the following patch: --- ../work/configure 2013-01-14 19:32:00.0 +0100 +++ configure 2013-01-14 19:42:15.0 +0100 @@ -5848,7 +5848,7 @@ else int main () { -if (strncmp (isl_version (), isl-0.10,

libgcc patch committed: Change __real_pthread_create ref to be non-weak

2013-01-14 Thread Ian Lance Taylor
The declaration of __real_pthread_create in generic-morestack-thread.c had attribute weak. Note that this code is linked with --wrap pthread_create, so the reference to __real_pthread_create is really to pthread_create. Using weak was incorrect, because the reference really is a strong one.

Re: libgcc patch committed: Change __real_pthread_create ref to be non-weak

2013-01-14 Thread Ian Lance Taylor
On Mon, Jan 14, 2013 at 11:33 AM, Ian Lance Taylor i...@google.com wrote: The declaration of __real_pthread_create in generic-morestack-thread.c had attribute weak. Note that this code is linked with --wrap pthread_create, so the reference to __real_pthread_create is really to pthread_create.

Re: libgcc patch committed: Change __real_pthread_create ref to be non-weak

2013-01-14 Thread H.J. Lu
On Mon, Jan 14, 2013 at 11:33 AM, Ian Lance Taylor i...@google.com wrote: The declaration of __real_pthread_create in generic-morestack-thread.c had attribute weak. Note that this code is linked with --wrap pthread_create, so the reference to __real_pthread_create is really to pthread_create.

Re: testsuite: missing or wrong dg-* directives?

2013-01-14 Thread Mike Stump
On Jan 14, 2013, at 6:23 AM, Mikael Morin mikael.mo...@sfr.fr wrote: Le 14/01/2013 00:37, Manfred Schwarb a écrit : Am 14.01.2013 00:10, schrieb Manfred Schwarb: There are several other oddities: d_g-final, braces have to be separated by spaces. Want to send a patch? Not sure about the

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Jack Howarth
0; | } configure:5868: result: no configure:5889: checking for version 0.11 of ISL configure:5921: result: no configure:5956: error: Unable to find a usable ISL. See config.log for details. This is with gcc trunk configured as... $ ../gcc-4.8-20130114/configure --prefix=/sw --prefix=/sw/lib

Re: [patch, libstdc++, testsuite] Fix simulator timouts.

2013-01-14 Thread Jonathan Wakely
On 14 January 2013 18:40, Steve Ellcey wrote: Here is a patch for the last of the C++ library tests that fail for me when run under the gnu simulator due to memory usage. I am using the same define mechanism to shrink them when run on a simulator as other tests are already using. Tested on

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Dominique Dhumieres
Jack, How exactly did you test this? I am using isl 0.11.1 and cloog 0.18.0 from fink installed in /sw out of tree. This fails in config.log as... Same here (the libs are not in /sw but in /mp) and I got the same errors before I did the changed in my previous post. Dominique

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Jack Howarth
On Mon, Jan 14, 2013 at 08:27:12PM +0100, Dominique Dhumieres wrote: In order to bootstrap r195167 with the new ISL/CLooG versions, I had to apply the following patch: --- ../work/configure 2013-01-14 19:32:00.0 +0100 +++ configure 2013-01-14 19:42:15.0 +0100 @@ -5848,7

Re: Mention of gcc 4.5 as first version with plugin support

2013-01-14 Thread Richard Biener
On Fri, Jan 11, 2013 at 9:46 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: On Fri, 11 Jan 2013 09:58:49 +0100 Dodji Seketeli do...@seketeli.org wrote: Thank you Dave for this documentation patch that looks OK to my casual eyes. Let's CC the documentation maintainers. Dave

[patch] --with-build-sysroot requires --with-sysroot

2013-01-14 Thread Matthias Klose
this is documented in doc/install.texi: This option is only useful when you are already using @option{--with-sysroot}. now, lets error out if you configure --with-build-sysroot without --with-sysroot. Ok for the trunk? This was hit in PR55743. I think the patch in

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Dominique Dhumieres
Jack, Without the change for isl, I get at configure time: -g -O2 -I/opt/mp/include -I/opt/mp/include checking for version 0.10 of ISL... no -g -O2 -I/opt/mp/include -I/opt/mp/include checking for version 0.11 of ISL... (cached) no configure: error: Unable to find a usable ISL. See config.log

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Jack Howarth
On Mon, Jan 14, 2013 at 09:28:53PM +0100, Dominique Dhumieres wrote: Jack, Without the change for isl, I get at configure time: -g -O2 -I/opt/mp/include -I/opt/mp/include checking for version 0.10 of ISL... no -g -O2 -I/opt/mp/include -I/opt/mp/include checking for version 0.11 of ISL...

[PATCH] Fix up ICE with reduction % (PR tree-optimization/55955)

2013-01-14 Thread Jakub Jelinek
Hi! For *SHIFT_EXPR/*ROTATE_EXPR with VECTOR_TYPE types, optab_for_tree_code requires us to decide if we want optab_vector or optab_scalar. It is likely not worth it to handle these four, they would fail soon anyway. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

[PATCH] Handle -fwrapv -ftrapv (PR tree-optimization/48766)

2013-01-14 Thread Jakub Jelinek
Hi! With this patch, -fno-[wt]rapv cancels corresponding -f[wt]rapv if any as before, and later -f[wt]rapv disables -f[tw]rapv, which is IMHO better behavior than if we use Negative(f[tw]rapv) on f[wt]rapv in common.opt. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

PING: gcc.target/arm: skip 5 tests for flag conflicts

2013-01-14 Thread Janis Johnson
Richard, Back in September I submitted a patch to fix five ARM tests in http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01515.html. You responded in http://gcc.gnu.org/ml/gcc-patches/2012-10/msg00972.html and I answered your questions in a reply. The first of the modified tests doesn't need to

PING: replace gcc.target/arm/ftest-*.c with compile-only tests

2013-01-14 Thread Janis Johnson
Richard, In December I submitted a patch to replace ftest*.c in gcc.target/arm with compile-only tests: http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01339.html. There has been no response to the patch. One reason for the replacement is that the arch_v*_multilib effective target checks don't do

Re: [patch, Fortran] PR 55806 - Inefficient ANY with array constructors

2013-01-14 Thread Thomas Koenig
Am 14.01.2013 14:29, schrieb Mikael Morin: Le 13/01/2013 23:14, Thomas Koenig a écrit : OK for trunk? OK with the changes suggested above. Thanks. Committed as rev. 195179 with your changes. Thanks a lot for the thorough review! Thomas

Re: [PATCH] Remove unnecessaily included limits.h in libgcc2.c

2013-01-14 Thread Matthias Klose
Am 04.01.2013 20:01, schrieb Wookey: I filed http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55743 (my first upstream gcc bug so be gentle :-) Details are there but the short version is that the limits.h inclusion in libgcc2.c is now a relic because the constants that it brings in are no longer

Re: testsuite: missing or wrong dg-* directives?

2013-01-14 Thread Harald Anlauf
On 01/14/13 00:10, Manfred Schwarb wrote: Am 13.01.2013 21:30, schrieb Harald Anlauf: On 01/12/13 22:02, Mikael Morin wrote: Le 08/01/2013 22:32, Harald Anlauf a écrit : On 12/28/12 21:49, Harald Anlauf wrote: Hello all, is there a default directive that is assumed when the testsuite is

Re: [PATCH] Handle -fwrapv -ftrapv (PR tree-optimization/48766)

2013-01-14 Thread Joseph S. Myers
On Mon, 14 Jan 2013, Jakub Jelinek wrote: Hi! With this patch, -fno-[wt]rapv cancels corresponding -f[wt]rapv if any as before, and later -f[wt]rapv disables -f[tw]rapv, which is IMHO better behavior than if we use Negative(f[tw]rapv) on f[wt]rapv in common.opt. Bootstrapped/regtested on

[testsuite] skip ARM tests fma.c and fma-sp.c for conflicting options

2013-01-14 Thread Janis Johnson
The options specified for gcc.target/arm/fma.c and fma-sp.c can conflict with several multilib options. This patch skips the tests for multilibs with conflicting options, and it adds option -mfloat-abi=hard which is needed for the test. OK for trunk? Janis 2013-01-14 Janis Johnson

[testsuite] fix ARM test gcc.target/arm/neon-vld1_dupQ.c

2013-01-14 Thread Janis Johnson
Test gcc.target/arm/neon-vld1_dupQ.c started failing with r194594, a C front end change that causes the test to get warnings. The test passes local variables of type int64x1_t to functions declared with arguments of type int64_t. This patch casts the values passed to those arguments. It's

[patch] PR55982 Fix buglet in __strncat_chk

2013-01-14 Thread Paul Pluzhnikov
Greetings, In libssp/strncat-chk.c, the loop was unrolled 5 times (apparently by accident). Ok for trunk? Thanks, -- Paul Pluzhnikov 2013-01-14 Paul Pluzhnikov ppluzhni...@google.com PR 55982 * strncat-chk.c (__strncat_chk): Fix loop unroll. Index: libssp/strncat-chk.c

[testsuite] remove ARM big-endian from some vect effective targets

2013-01-14 Thread Janis Johnson
As described in PR testsuite/54622, lots of tests from gcc.dg/vect fail for big-endian ARM, apparently with bits of vectorization support disabled here and there. This patch removes big-endian ARM from check_effective_target_vect_perm_byte check_effective_target_vect_perm_short

Re: [RFA/dwarf] Add DW_AT_use_GNAT_descriptive_type flag for Ada units.

2013-01-14 Thread Joel Brobecker
Ping? Thank you! On Tue, Jan 08, 2013 at 07:24:14AM -0500, Joel Brobecker wrote: Hello, I just noticed that part of the proposal we made for... http://gcc.gnu.org/wiki/DW_AT_GNAT_descriptive_type ... got missed in the patch that got checked in:

Re: [RFC PATCH] Implementing ifunc target hook

2013-01-14 Thread Maxim Kuvyrkov
On 15/01/2013, at 4:55 AM, Alexander Ivchenko wrote: thank you very much for your review! I fixed the arm build and all other issues that you raised. the patch is attached. Bootstrap and tested on x86-64 linux The patch is OK with the cleanups mentioned below (no need to resubmit for

Fwd: [Bug fortran/54286] [4.8 Regression] Accepts invalid proc-pointer assignments involving proc-ptr function result

2013-01-14 Thread Paul Richard Thomas
Dear, I missed the null interface case in the fix for PR54286. The fix is 'obvious' and 'safe'. Thanks to Dominique for identifying the problem and the fix. Cheers Paul Author: pault Date: Tue Jan 15 05:29:01 2013 New Revision: 195185 URL:

Re: [patch] PR55982 Fix buglet in __strncat_chk

2013-01-14 Thread Jakub Jelinek
On Mon, Jan 14, 2013 at 04:57:42PM -0800, Paul Pluzhnikov wrote: In libssp/strncat-chk.c, the loop was unrolled 5 times (apparently by accident). 2013-01-14 Paul Pluzhnikov ppluzhni...@google.com PR 55982 * strncat-chk.c (__strncat_chk): Fix loop unroll. Ok.