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 > > PR 55982 > * strncat-chk.c (__strncat_chk): Fix loop unroll. Ok. Jakub

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: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195

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

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: > http://gcc.gnu.org/ml/gcc-patches/20

[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 check_

[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 PR 55982 * strncat-chk.c (__strncat_chk): Fix loop unroll. Index: libssp/strncat-chk.c =

[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 possibl

[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

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/regtes

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 run?

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 l

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

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: . 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

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 . 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 t

[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? 2013-0

[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? 2013-01-1

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

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 fo

[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 http://gcc.gnu.org/ml/g

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 wrote: > On Fri, 11 Jan 2013 09:58:49 +0100 > Dodji Seketeli wrote: > >> Thank you Dave for this documentation patch that looks OK to my casual >> eyes. Let's CC the documentation maintainers. >> >> Dave Johansen a écrit: >> >> > >> > +GCC p

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: [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, 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. > > Te

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Jack Howarth
| { | if (strncmp (isl_version (), "isl-0.10", strlen ("isl-0.10")) != 0) | return 1; | | ; | return 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 conf

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

2013-01-14 Thread Mike Stump
On Jan 14, 2013, at 6:23 AM, Mikael Morin 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 d

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 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. Using weak

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 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. Using weak

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. Ther

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", s

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 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 > (which I thi

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 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 corre

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 info

[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: /cvs/gcc/wwwdocs/htdocs/

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 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 b

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 wo

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 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 correctly. Not fully

[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 che

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 > documentatio

Re: [PATCH] Allow new ISL/CLooG versions

2013-01-14 Thread Richard Biener
Jack Howarth 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 probabl

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)

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

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 __UINT2

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

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

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

[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 * config/avr/avr-c

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 irredu

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

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. > > htt

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] A

[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 base_ar

Re: RFA: Update copyright for libatomic

2013-01-14 Thread Richard Henderson
On 01/13/2013 12:25 AM, Richard Sandiford wrote: Here's the result of running the copyright script on libatomic/. Tested on x86_64-linux-gnu. OK to install? Richard libatomic/ Update copyright years Ok. r~

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 gcc/ * config/aarch64/aarch64-simd.md (*aarch64_simd_ld1r): New.

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 gcc/ * config/aarch64/aarch64-simd.md (*aarch6

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 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 similar check

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 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 wrote: >> >> >> +(define_insn "atomic_store_1" >> >> >> + [(set (match_operand:ATOMIC 0 "memory_operand" "=m") >> >> >> + (unspec

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

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 > > * doc/invoke.texi: Document -pr

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 wrote: > On Fri, Jan 11, 2013 at 9:46 PM, Basile Starynkevitch > wrote: >> On Fri, 11 Jan 2013 09:58:49 +0100 >> Dodji Seketeli wrote: >> >>> Thank you Dave for this documentation patch that looks OK to my casual >>> eyes. Let's CC the documentat

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 alte

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. Bootstrapped&tested on powerpc64-unknown-linux-gnu[1] and ia64-unknown-linux-gnu[2], and cross-

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 > > * Makefile.def (install-target-libsanitizer): Depend on > install-target-libstdc++-v3. > * Makefile.in: Regenerate. Okay. > --- Makefile.def (Revision 195136) > +++ Makefile.

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

[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 in

[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 co

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 : > On 10/01/2013, at 12:24 AM, Alexander Ivchenko wrote: > >>> Config/linux-android.h is a better plac

[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: [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 v

[PATCH] Let target get_memmodel know about stores/barriers

2013-01-14 Thread Andi Kleen
From: Andi Kleen 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 HLE is not useful at

[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=revision&revision=195143 http://gcc.gnu.org/viewcvs?view=revision&revision=195145 Fix typos, remove trailing blanks, coding rules, ... http://gcc.gnu.org/viewcvs?view=revision&revision=195151

[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 * tree-vectorizer.h (vect_get_sing

[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 * tree-cfg.c (verify_expr_location, verify_expr_location_1, verify_locatio

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 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. It is not rig

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 wrote: > On Sat, Jan 12, 2013 at 06:04:19PM +0100, Bernhard Reutner-Fischer wrote: >> On 12 January 2013 16:28:41 Andi Kleen wrote: >> >From: Andi Kleen >> >> >+Returns _XBEGIN_STARTED when the transaction >> >+started successfully (not this is not 0,

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: 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 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 that's a decision th

[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 Biene

Re: [RFC, middlend] Fix for PR54218

2013-01-14 Thread Richard Biener
On Fri, Jan 11, 2013 at 6:37 PM, George Thomas wrote: > On Fri, Jan 11, 2013 at 9:53 PM, Andrew Pinski wrote: >> On Fri, Jan 11, 2013 at 8:17 AM, George Thomas >> wrote: >>> Hi, >>> >>> I am sending a patch which solves the debugging issue (PR 54218). >>> >>> The fix is to allocate stack space o

Re: RFA: Update copyright for libcpp

2013-01-14 Thread Tom Tromey
> "Richard" == Richard Sandiford 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 x86_64-linux-gnu. OK to inst

[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 version.

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 don't

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 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 tes

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 gcc/ * config/aarch64/aarch64-simd.md (*aarch64_simd_ld1r):

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 gcc/ * config/aarch64/aarch64-simd.md (*aarch64_simd_ld1r): New. * confi

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.

[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] 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 b