[PATCH] [libiberty] MAX_PATH problems with mingw gcc

2013-11-05 Thread Joey Ye
Ping ChangeLog 2013-10-27 Vladimir Simonov (include) filename.h (FILENAME_NORMALIZE): New macro. (filename_normalize): New declare. (libiberty) filename_cmp.c (memmove_left): New function. (filename_normalize): Likewise. getpwd.c (getpwd): Use FILENAME_NOR

[C] Fix PR57258: unused variable warning is emitted for volatile variables

2013-11-05 Thread Mingjie Xing
Hi, Changes: * c/c-decl.c (pop_scope): Skip volatile variables while emit warnings for unused variables. Tested on i686-pc-linux-gnu. OK? Mingjie Index: gcc/c/c-decl.c === --- gcc/c/c-decl.c (revision 204285) +++ gcc/c/c-d

RE: [PATCH 1/n] Add conditional compare support

2013-11-05 Thread Zhenqiang Chen
> -Original Message- > From: Richard Henderson [mailto:r...@redhat.com] > Sent: Tuesday, November 05, 2013 4:39 AM > To: Zhenqiang Chen > Cc: Richard Earnshaw; 'Richard Biener'; GCC Patches > Subject: Re: [PATCH 1/n] Add conditional compare support > > On 11/04/2013 08:00 PM, Zhenqiang Ch

Re: Re-factor tree.h - Part 1

2013-11-05 Thread Jakub Jelinek
On Wed, Nov 06, 2013 at 08:04:51AM +0100, Marc Glisse wrote: > On Tue, 5 Nov 2013, Diego Novillo wrote: > > >This is the first patch in a series of patches to cleanup tree.h to > >reduce the exposure it has all over the compiler. > > > >In this patch, I'm moving functions that are used once into t

Re: [PATCH i386 4/8] [AVX512] [8/8] Add substed patterns: `sae-only for expand' subst.

2013-11-05 Thread Kirill Yukhin
Hello, This patch introduces sae-only feature for structureless expands. Bootstrapped. Is it ok for trunk? -- Thanks, K --- gcc/config/i386/sse.md | 18 ++ gcc/config/i386/subst.md | 20 2 files changed, 30 insertions(+), 8 deletions(-) diff --git a/gcc/

Re: [PATCH i386 4/8] [AVX512] [7/8] Add substed patterns: `round for expand' subst.

2013-11-05 Thread Kirill Yukhin
Hello, This patch introduces dedicated subst to add rounding to structureless expands. Bootstrapped. Is it ok for trunk? -- Thanks, K --- gcc/config/i386/sse.md | 24 gcc/config/i386/subst.md | 18 ++ 2 files changed, 30 insertions(+), 12 deletions(-)

Re: [PATCH i386 4/8] [AVX512] [6/8] Add substed patterns: `sae' subst.

2013-11-05 Thread Kirill Yukhin
Hello, This patch introduces `sae-only' (EVEX feature to Supress Arithmetic Exceptions) subst. Bootstrapped. Is it ok for trunk? -- Thanks, K --- gcc/config/i386/sse.md | 198 +++ gcc/config/i386/subst.md | 31 2 files changed, 130 insert

Re: [PATCH i386 4/8] [AVX512] [4/n] Add substed patterns: `sd' subst.

2013-11-05 Thread Kirill Yukhin
Hello, This small patch introduces `sd' subst. `sd' (Source-Destination) subst is almost the same, as the usual mask-subst, but it's only used for zero-masking. The reason is that some patterns already have an operand with constraint "0" and we can't add a new operand with the same constraint. So,

Re: [RFA][PATCH] Minor erroneous path isolation followups

2013-11-05 Thread Jakub Jelinek
On Tue, Nov 05, 2013 at 11:15:30PM -0700, Jeff Law wrote: > * java/builtins.c (initialize_builtins): Provide __builtin_trap. Without java/ ;) > --- a/gcc/java/builtins.c > +++ b/gcc/java/builtins.c > @@ -580,6 +580,9 @@ initialize_builtins (void) > build_function_type_list (

Re: [PATCH] RE: Testsuite / Cilk Plus: Include library path in compile flags in gcc.dg/cilk-plus/cilk-plus.exp

2013-11-05 Thread Jakub Jelinek
On Wed, Nov 06, 2013 at 02:24:01AM +, Iyer, Balaji V wrote: > Fixed patch is attached. The responses to your question are given below. > Is this patch OK? > > Here is the ChangeLog entry: > > +2013-11-05 Balaji V. Iyer > + > + * c-c++-common/cilk-plus/CK/fib.c: Reduced the itera

Re: Re-factor tree.h - Part 1

2013-11-05 Thread Marc Glisse
On Tue, 5 Nov 2013, Diego Novillo wrote: This is the first patch in a series of patches to cleanup tree.h to reduce the exposure it has all over the compiler. In this patch, I'm moving functions that are used once into the files that use them, and make them private to that file. These functions

Re: [RFA][PATCH] Isolate erroneous paths optimization

2013-11-05 Thread Marc Glisse
On Tue, 5 Nov 2013, Ian Lance Taylor wrote: This patch actually breaks the Go testsuite. In Go dereferencing a nil pointer is well-defined: it causes panic that can be caught. This breaks a test for that functionality by changing the panic to a builtin_trap. That's not a big deal; I'll just d

Re: [fortran,doc] Fix typo in doc

2013-11-05 Thread Tobias Burnus
FX wrote: I think the doc says “assumed-shape” where it means “assumed-rank”. Is that OK? OK. Thanks for the patch! Tobias PS: I assume you still can commit.

[RFA][PATCH] Minor erroneous path isolation followups

2013-11-05 Thread Jeff Law
Per a discussion between Richi and myself, this patch adds a __builtin_trap to the Java front end. If someone could look at that code closely to ensure I didn't muck it up, I'd appreciated it. With that code in the Java front-end, we can remove the hack from gate_isolate_erroneous_paths.

[Patch, ARM] New feature to minimize the literal load for armv7-m target

2013-11-05 Thread Terry Guo
Hi, This patch intends to minimize the use of literal pool for some armv7-m targets that have slower speed to load data from flash than to fetch instruction from flash. The normal literal load instruction is now replaced by MOVW/MOVT instructions. A new option -mslow-flash-data is created for this

RE: Fix scheduler ix86_issue_rate and ix86_adjust_cost for modern x86 chips

2013-11-05 Thread Gopalasubramanian, Ganesh
Thanks Honza! I have committed changes ( for default ). http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=204442 I will add lookahead value 8 for O3 after experimenting with it. Regards Ganesh -Original Message- From: Jan Hubicka [mailto:hubi...@ucw.cz] Sent: Wednesday, October 30,

Re: [RFA][PATCH] Isolate erroneous paths optimization

2013-11-05 Thread Jeff Law
On 11/05/13 22:24, Ian Lance Taylor wrote: On Mon, Nov 4, 2013 at 5:57 PM, Jeff Law wrote: * Makefile.in (OBJS): Add gimple-ssa-isolate-paths.o * common.opt (-fisolate-erroneous-paths): Add option and documentation. * gimple-ssa-isolate-paths.c: New file.

Re: [RFA][PATCH] Isolate erroneous paths optimization

2013-11-05 Thread Ian Lance Taylor
On Mon, Nov 4, 2013 at 5:57 PM, Jeff Law wrote: > > * Makefile.in (OBJS): Add gimple-ssa-isolate-paths.o > * common.opt (-fisolate-erroneous-paths): Add option and > documentation. > * gimple-ssa-isolate-paths.c: New file. > * gimple.c (check_loadstore): New

mismatched decls w/ both builtin and explicit decl

2013-11-05 Thread DJ Delorie
Consider this source: extern char *index(const char *,int); static int index; "index" is a builtin as well, but because it's a builtin gcc skips the "previous declaration was here..." despite having *a* previous decl it could complain about. Note that newlib provides decls for many builtins

Re: [PATCH] PR58985: testcase error.

2013-11-05 Thread Wei Mi
+Release manager. Thanks, committed to trunk as r204438. Ok for 4.8 branch? On Tue, Nov 5, 2013 at 11:19 AM, Jeff Law wrote: > On 11/04/13 12:07, Wei Mi wrote: >> >> Hi, >> >> This is to fix testcase error reported in PR58985. >> >> The intention of the testcase was to ensure there was no REG_EQ

Re: [PATCH, rs6000] Enable VSX code generation in little endian mode

2013-11-05 Thread David Edelsohn
On Tue, Nov 5, 2013 at 1:43 PM, Bill Schmidt wrote: > Hi, > > With the recent set of patches, there are no longer any vector-related > regressions for powerpc64le-linux-gnu that do not also occur for > powerpc64-linux-gnu. (gcc.dg/vect/vect-96.c is still broken for both > endiannesses and is bein

Re: [PATCH, rs6000] (3/3) Fix mulv4si3 and mulv8hi3 patterns for little endian

2013-11-05 Thread David Edelsohn
On Mon, Nov 4, 2013 at 7:33 PM, Bill Schmidt wrote: > Hi, > > Here's a revised version of this patch according to Richard's > suggestions. It differs from the previous version only in the method > used to ensure vmulouh is generated; we now call the new > gen_altivec_vmulouh to accomplish this. >

Re: [PATCH, rs6000] (1/3) Reverse meanings of multiply even/odd for little endian

2013-11-05 Thread David Edelsohn
On Mon, Nov 4, 2013 at 7:28 PM, Bill Schmidt wrote: > Hi, > > Here's a new version of this patch, revised according to Richard > Sandiford's suggestions. Unfortunately the diffing is a little bit ugly > for this version. > > Bootstrapped and tested on powerpc64{,le}-unknown-linux-gnu with no > re

Re: [patch] Move C front end to its own directory (1/3)

2013-11-05 Thread Mike Stump
On May 29, 2010, at 2:28 PM, Steven Bosscher wrote: > This is the first patch of 3 planned, to move the C front end to its > own directory. > This first patch moves all the code common to the C-family of front > ends to a new directory c-family/. You forgot to clean up the built bits?!:

RE: [PATCH] RE: Testsuite / Cilk Plus: Include library path in compile flags in gcc.dg/cilk-plus/cilk-plus.exp

2013-11-05 Thread Iyer, Balaji V
Hi Jakub, Fixed patch is attached. The responses to your question are given below. Is this patch OK? Here is the ChangeLog entry: +2013-11-05 Balaji V. Iyer + + * c-c++-common/cilk-plus/CK/fib.c: Reduced the iteration from + 40 to 30. Replaced iteration variable with a #defin

Re: [PATCH] Time profiler - phase 1

2013-11-05 Thread Martin Liška
Hello, there's updated version of the first phase patch (including 2 new tests). Martin On 4 November 2013 11:46, Jan Hubicka wrote: >> diff --git a/gcc/ChangeLog b/gcc/ChangeLog >> index fca665b..3b62bcc 100644 >> --- a/gcc/ChangeLog >> +++ b/gcc/ChangeLog >> @@ -1,3 +1,31 @@ >> +2013-10-29

Re: Cleanup Linux libc selection and Android support

2013-11-05 Thread Maxim Kuvyrkov
On 19/09/2013, at 8:26 am, Maxim Kuvyrkov wrote: > Following recent breakage caused by adding nominal Android support to all > *linux* targets [*] this patch series cleans up libc selection for Linux > targets (-mglibc/-muclibc/-mbionic), splits libc selection logic from Android > support, and

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Ian Lance Taylor
On Tue, Nov 5, 2013 at 2:12 PM, Jeff Law wrote: > > I can't speak for Andrew, but my experience with this kind of object type > casting in a large C++ project is that it's a red flag for a design problem. I'm going to chime in to say that I think it's a pretty typical way to represent a compiler

Re: [SH] PR 30807 - Add test case

2013-11-05 Thread Kaz Kojima
Oleg Endo wrote: > This adds a test case for PR 30807 which is based on the PR's attachment > 17961. Tested with > make -k check-gcc RUNTESTFLAGS="sh-torture.exp --target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb} > > OK to add? OK with the changes suggested by

[v3 patch] Implement C++14 N3655 TransformationTraits Redux

2013-11-05 Thread Jonathan Wakely
Another C++14 library proposal. std::aligned_union_t is missing because we don't have std::aligned_union yet. I also changed a few tests to use static_assert instead of VERIFY so they only need to be compiled, not executed, saving a few milliseconds when running the testsuite ;-) 2013-11-05 Jon

Re: [SH] Split addi,subdi,negdi before reload

2013-11-05 Thread Kaz Kojima
Oleg Endo wrote: > It seems that splitting multi-word insns before reload results in > slightly better code on average (according to CSiBE). The attached > patch implements that. > Tested on rev. 204263 with > make -k -j4 check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a

Re: [wide-int] Commit wide-int version of doloop rework

2013-11-05 Thread Kenneth Zadeck
This is the right way to fix this. thanks. kenny On 11/05/2013 03:00 PM, Richard Sandiford wrote: I've committed the patch below, which is a trivial wide-intification of: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00335.html I won't commit the mainline patch for a couple of days to give

Re: [PATCH] Fix for PR bootstrap/58951

2013-11-05 Thread Gerald Pfeifer
Hi Balaji, On Tue, 5 Nov 2013, Iyer, Balaji V wrote: > Attached, please find a patch to fix PR 58951. The usage of -ldl is not > necessary. The patch is tested in x86_64 and x86. It is committed as > obvious. thanks for looking into this. Unfortunately, with SVN revision 204424 (that is, after

Re: [Patch, libgfortran] Set close-on-exec flag when opening files

2013-11-05 Thread Janne Blomqvist
PING On Wed, Oct 30, 2013 at 1:39 AM, Janne Blomqvist wrote: > Hello, > > the attached patch sets the close-on-exec flag when opening files, as > is usually considered good practice these days. See e.g. > http://www.python.org/dev/peps/pep-0446/ and links therein for more > information. > > The p

Re: [PATCH] Handling == or != comparisons that may affect range test optimization.

2013-11-05 Thread Jeff Law
On 11/05/13 15:00, Cong Hou wrote: Can you factor those two hunks of nearly identical code into a single function and call it twice? I'm also curious if you really need the code to swap lhs/rhs. When can the LHS of a cond be an integer constant? Don't we canonicalize it as ? I was not aw

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Andrew MacLeod
On 11/05/2013 04:57 PM, David Malcolm wrote: Thanks for looking through this. Both you and Andrew objected to my use of the is-a.h stuff. Is this due to the use of C++ templates in that code? If I were to rewrite things in a more C idiom, would that be acceptable? Thats too strong a statem

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Jeff Law
On 11/05/13 14:57, David Malcolm wrote: Thanks for looking through this. Both you and Andrew objected to my use of the is-a.h stuff. Is this due to the use of C++ templates in that code? If I were to rewrite things in a more C idiom, would that be acceptable? I can't speak for Andrew, but my

[fortran,doc] Fix typo in doc

2013-11-05 Thread FX
I think the doc says “assumed-shape” where it means “assumed-rank”. Is that OK? FX Index: gfortran.texi === --- gfortran.texi (revision 204389) +++ gfortran.texi (working copy) @@ -2624,7 +2624,7 @@ other hand, assumed-ty

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Andrew MacLeod
On 11/05/2013 04:18 PM, Jeff Law wrote: * Patch 4 of 6: This patch implement further specializations of is_a_helper ::test, for gimple_has_ops and gimple_has_mem_ops. Here's where I start to get more concerned. * Patch 5 of 6: This patch does the rest of porting from union acces

Re: [PATCH] Handling == or != comparisons that may affect range test optimization.

2013-11-05 Thread Cong Hou
On Tue, Nov 5, 2013 at 12:23 PM, Jeff Law wrote: > On 10/31/13 18:03, Cong Hou wrote: >> >> (This patch is for the bug 58728: >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58728) >> >> As in the bug report, consider the following loop: >> >> int foo(unsigned int n) >> { >>if (n != 0) >>if

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread David Malcolm
On Tue, 2013-11-05 at 14:18 -0700, Jeff Law wrote: > On 10/31/13 10:26, David Malcolm wrote: > > The gimple statement types are currently implemented using a hand-coded > > C inheritance scheme, with a "union gimple_statement_d" holding the > > various possible structs for a statement. > > > > The

Re: [SH] PR 30807 - Add test case

2013-11-05 Thread Mike Stump
On Nov 5, 2013, at 1:45 PM, Oleg Endo wrote: > You're right, it's redundant. It should be just > /* { dg-do compile } */ > > shouldn't it? Yup, that's my take. > I can change that before committing, no problem. Thanks.

Re: libsanitizer merge from upstream r191666

2013-11-05 Thread Evgeniy Stepanov
On Tue, Nov 5, 2013 at 10:45 AM, Jakub Jelinek wrote: > On Tue, Nov 05, 2013 at 10:17:09AM -0800, Evgeniy Stepanov wrote: >> On Tue, Nov 5, 2013 at 10:07 AM, Peter Bergner wrote: >> > On Tue, 2013-11-05 at 09:57 -0600, Peter Bergner wrote: >> >> On Tue, 2013-11-05 at 08:19 +0100, Jakub Jelinek wr

Re: [SH] PR 30807 - Add test case

2013-11-05 Thread Oleg Endo
On Tue, 2013-11-05 at 13:33 -0800, Mike Stump wrote: > On Nov 5, 2013, at 12:44 PM, Oleg Endo wrote: > > +/* { dg-do compile { target "sh*-*-*" } } */ > > Why do this, when sh-torture.exp does this: > > if { ![istarget sh*-*-*] } then { > return > } > > ? I suspect you merely copied the styl

Re: [PATCH] Handling == or != comparisons that may affect range test optimization.

2013-11-05 Thread Cong Hou
It seems there are some changes in GCC. But if you change the type of n into signed int, the issue appears again: int foo(int n) { if (n != 0) if (n != 1) if (n != 2) if (n != 3) if (n != 4) return ++n; return n; } Also, ifcombine also suffers from the same issue here. t

Re: [SH] PR 30807 - Add test case

2013-11-05 Thread Mike Stump
On Nov 5, 2013, at 12:44 PM, Oleg Endo wrote: > +/* { dg-do compile { target "sh*-*-*" } } */ Why do this, when sh-torture.exp does this: if { ![istarget sh*-*-*] } then { return } ? I suspect you merely copied the style from the other test cases. If there isn't a good reason to do this, p

[v3 patch] update C++11 status table in manual

2013-11-05 Thread Jonathan Wakely
I just tried to implement std::aligned_union, but was blocked by PR 59012 and PR 59013, so this documents it as missing. 2013-11-05 Jonathan Wakely * doc/xml/manual/status_cxx2011.xml: Document aligned_union as missing. Committed to trunk, 4.8 and 4.7. I've also regenerated t

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Jeff Law
On 10/31/13 10:26, David Malcolm wrote: The gimple statement types are currently implemented using a hand-coded C inheritance scheme, with a "union gimple_statement_d" holding the various possible structs for a statement. The following series of patches convert it to a C++ hierarchy, using the e

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Jeff Law
On 11/04/13 15:28, Andrew MacLeod wrote: That said, the patch which enables this is more self contained, so wouldn't be subject to that. Its a matter of whether it has enough merit of its own to go in. Having the first patch in mainline would actually allow someone to experiment more easily du

Re: [patch] Move block_in_transaction out of gimple.h

2013-11-05 Thread Jeff Law
On 11/05/13 14:06, Andrew MacLeod wrote: Looks like another location of convenience perhaps... Anyway, block_in_transaction (bb) really belongs in basic-block.h... The only oddity is that it also checks flag_tm... Is this really necessary? One would think the flag would never be set if flag_tm

[patch] Move block_in_transaction out of gimple.h

2013-11-05 Thread Andrew MacLeod
Looks like another location of convenience perhaps... Anyway, block_in_transaction (bb) really belongs in basic-block.h... The only oddity is that it also checks flag_tm... Is this really necessary? One would think the flag would never be set if flag_tm wasn't on... In any case, basic-blo

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Jeff Law
On 11/04/13 15:25, Jakub Jelinek wrote: On Mon, Nov 04, 2013 at 04:43:33PM -0500, David Malcolm wrote: I tried converting gimple_call_set_lhs to accept a gimple_call, rather than a gimple, and excitingly, it was easiest to also convert cgraph_edge's call_stmt to also be a gimple_call, rather tha

wwwdocs: gcc-4.9/changes.html: Add OpenMP4 + -Wdate-time

2013-11-05 Thread Tobias Burnus
Hi Jakub and Gerald, this patch updates the release notes. For OpenMPv4, I only mention C and C++ as the parsing support for Fortran is still missing. (Hopefully, it will be included in the GCC 4.9.0 release.) Additionally, I mention the just added -Wdate-time warning. OK? Tobias Index: ch

Re: [PATCH, MPX, 2/X] Pointers Checker [7/25] Suppress BUILT_IN_CHKP_ARG_BND optimizations.

2013-11-05 Thread Ilya Enkovich
2013/11/5 Jeff Law : > On 11/04/13 06:27, Richard Biener wrote: >> >> On Thu, Oct 31, 2013 at 10:02 AM, Ilya Enkovich >> wrote: >>> >>> Hi, >>> >>> Here is a patch which hadles the problem with optimization of >>> BUILT_IN_CHKP_ARG_BND calls. Pointer Bounds Checker expects that argument >>> of th

Re: [PATCH] Handling == or != comparisons that may affect range test optimization.

2013-11-05 Thread Jakub Jelinek
On Tue, Nov 05, 2013 at 01:23:00PM -0700, Jeff Law wrote: > On 10/31/13 18:03, Cong Hou wrote: > >(This patch is for the bug 58728: > >http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58728) > > > >As in the bug report, consider the following loop: > > > >int foo(unsigned int n) > >{ > > if (n != 0) >

Re: [wide-int] Commit wide-int version of doloop rework

2013-11-05 Thread Mike Stump
On Nov 5, 2013, at 12:00 PM, Richard Sandiford wrote: > I've committed the patch below, which is a trivial wide-intification of: > >http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00335.html > > I won't commit the mainline patch for a couple of days So, please feel free to do a svn merge ^/tr

[SH] PR 30807 - Add test case

2013-11-05 Thread Oleg Endo
Hi, This adds a test case for PR 30807 which is based on the PR's attachment 17961. Tested with make -k check-gcc RUNTESTFLAGS="sh-torture.exp --target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb} OK to add? Cheers, Oleg testsuite/ChangeLog: PR target/308

Re: [PATCH] Handling == or != comparisons that may affect range test optimization.

2013-11-05 Thread Jeff Law
On 10/31/13 18:03, Cong Hou wrote: (This patch is for the bug 58728: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58728) As in the bug report, consider the following loop: int foo(unsigned int n) { if (n != 0) if (n != 1) if (n != 2) if (n != 3) if (n != 4) return ++n; ret

Re: [Patch: libcpp, c-family, Fortran] Re: Warning about __DATE__ and __TIME__

2013-11-05 Thread Steve Kargl
On Mon, Nov 04, 2013 at 09:58:30PM +0100, Tobias Burnus wrote: > Tobias Burnus wrote: > > Gerald Pfeifer wrote: > >> To make it easier to reproduce builds of software and entire GNU/Linux > >> distributions, RMS had the idea of adding a warning to GCC that warns > >> about the use of __DATE__ and _

[wide-int] Commit wide-int version of doloop rework

2013-11-05 Thread Richard Sandiford
I've committed the patch below, which is a trivial wide-intification of: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00335.html I won't commit the mainline patch for a couple of days to give target maintainers a chance to comment, but I thought it'd better go into wide-int now to unbreak pow

Re: [PATCH] Fix get_bit_range and expand_assignment for negative bitpos (PR middle-end/58970)

2013-11-05 Thread Jeff Law
On 11/05/13 01:01, Jakub Jelinek wrote: Hi! This is an updated version of the get_bit_range fix, this time it ensures that bitpos is not negative already right after the get_inner_reference call, because various parts of the expansion might be confused by the negative values. As the second test

Re: [PATCH] PR fortran/58989

2013-11-05 Thread Paul Richard Thomas
Hi Steve, It looks good to me - OK for trunk. Thanks for the patch. Paul On 5 November 2013 18:07, Steve Kargl wrote: > The inlined patch fixes a problem where an array constructor > is used as the shape argument in RESHAPE. The patch is > straightforward and self-explanatory. > Regression te

Re: [RFA][PATCH] Isolate erroneous paths optimization

2013-11-05 Thread Jeff Law
On 11/05/13 04:53, Richard Biener wrote: Fortran front-end doesn't provide this IIRC. Are you sure? omp lowering makes unconditional use of it and I see it created in f95-lang.c. There are various other unconditional uses one covering vararg functions, one exceptions. I doubt we have a lang

Re: [PATCH] loop-{iv,unswitch}.c fixes (PR rtl-optimization/58997)

2013-11-05 Thread Jeff Law
On 11/05/13 06:09, Jakub Jelinek wrote: Hi! Apparently, for iv->extend == IV_UNKNOWN_EXTEND if iv->mode != iv->extend_mode get_iv_value returns a value in iv->mode mode, while otherwise in iv->extend_mode. This makes some sense, because with IV_UNKNOWN_EXTEND the upper bits are unknown, so ther

Re: [PATCH] PR58985: testcase error.

2013-11-05 Thread Jeff Law
On 11/04/13 12:07, Wei Mi wrote: Hi, This is to fix testcase error reported in PR58985. The intention of the testcase was to ensure there was no REG_EQUIV notes generated for a reg which was used in a paradoxical subreg. When target was x86, there was subreg generated so I omitted to add the su

Re: [PATCH, i386]: Fix MULTILIB_MATCHES option typos in t-rtems

2013-11-05 Thread Uros Bizjak
On Tue, Nov 5, 2013 at 7:43 PM, Ralf Corsepius wrote: >> 2013-11-05 Uros Bizjak >> >> * config/i386/t-rtems (MULTILIB_MATCHES): Fix option typos. >> >> Committed to SVN mainline as obvious, should be backported to other >> release branches. > > Yikes! Could you please apply it to all open

Re: patch implementing a new pass for register-pressure relief through live range shrinkage

2013-11-05 Thread Vladimir Makarov
On 11/05/2013 01:13 PM, Jeff Law wrote: > On 11/05/13 08:35, Vladimir Makarov wrote: >>I'd like to add a new experimental optimization to the trunk. This >> optimization was discussed on RA BOF of this summer GNU Cauldron. > > [Good stuff snipped] > >> >>The single but significant drawback

Re: libsanitizer merge from upstream r191666

2013-11-05 Thread Jakub Jelinek
On Tue, Nov 05, 2013 at 10:17:09AM -0800, Evgeniy Stepanov wrote: > On Tue, Nov 5, 2013 at 10:07 AM, Peter Bergner wrote: > > On Tue, 2013-11-05 at 09:57 -0600, Peter Bergner wrote: > >> On Tue, 2013-11-05 at 08:19 +0100, Jakub Jelinek wrote: > >> > Note, not even glibc itself includes , so the ch

Re: [PATCH, i386]: Fix MULTILIB_MATCHES option typos in t-rtems

2013-11-05 Thread Ralf Corsepius
On 11/05/2013 07:21 PM, Uros Bizjak wrote: Hello! 2013-11-05 Uros Bizjak * config/i386/t-rtems (MULTILIB_MATCHES): Fix option typos. Committed to SVN mainline as obvious, should be backported to other release branches. Yikes! Could you please apply it to all open GCC branches? Ralf

[PATCH, rs6000] Enable VSX code generation in little endian mode

2013-11-05 Thread Bill Schmidt
Hi, With the recent set of patches, there are no longer any vector-related regressions for powerpc64le-linux-gnu that do not also occur for powerpc64-linux-gnu. (gcc.dg/vect/vect-96.c is still broken for both endiannesses and is being tracked.) I did a code walkthrough on the back-end code and d

Re: [RFA][PATCH] Isolate erroneous paths optimization

2013-11-05 Thread Jeff Law
On 11/05/13 04:53, Richard Biener wrote: Fortran front-end doesn't provide this IIRC. Are you sure? omp lowering makes unconditional use of it and I see it created in f95-lang.c. There are various other unconditional uses one covering vararg functions, one exceptions. I doubt we have a lang

Re: [Patch, AArch64] Define vec_extract.

2013-11-05 Thread Marcus Shawcroft
On 5 November 2013 13:49, Tejas Belagod wrote: > 2013-11-05 Tejas Belagod > > gcc/ > * config/aarch64/aarch64-simd.md (vec_extract): New. OK /Marcus

[PATCH, i386]: Fix MULTILIB_MATCHES option typos in t-rtems

2013-11-05 Thread Uros Bizjak
Hello! 2013-11-05 Uros Bizjak * config/i386/t-rtems (MULTILIB_MATCHES): Fix option typos. Committed to SVN mainline as obvious, should be backported to other release branches. Uros. Index: ChangeLog === --- ChangeLog (revi

Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3)

2013-11-05 Thread Andrew MacLeod
On 11/01/2013 05:36 PM, Andrew MacLeod wrote: On 10/31/2013 12:26 PM, David Malcolm wrote: [Shamelessly hijacking Andrew's thread about gimple.h refactoring, since this seems on-topic for that, and I'm keen to hear from Andrew on how the following would interact with his work - I *think* our two

Re: libsanitizer merge from upstream r191666

2013-11-05 Thread Evgeniy Stepanov
On Tue, Nov 5, 2013 at 10:07 AM, Peter Bergner wrote: > On Tue, 2013-11-05 at 09:57 -0600, Peter Bergner wrote: >> On Tue, 2013-11-05 at 08:19 +0100, Jakub Jelinek wrote: >> > Note, not even glibc itself includes , so the chances of that >> > header actually working for you are low. glibc instead

Re: patch implementing a new pass for register-pressure relief through live range shrinkage

2013-11-05 Thread Jeff Law
On 11/05/13 08:35, Vladimir Makarov wrote: I'd like to add a new experimental optimization to the trunk. This optimization was discussed on RA BOF of this summer GNU Cauldron. [Good stuff snipped] The single but significant drawback is additional compilation time (4%-6%) as the 1st in

Re: libsanitizer merge from upstream r191666

2013-11-05 Thread Peter Bergner
On Tue, 2013-11-05 at 09:57 -0600, Peter Bergner wrote: > On Tue, 2013-11-05 at 08:19 +0100, Jakub Jelinek wrote: > > Note, not even glibc itself includes , so the chances of that > > header actually working for you are low. glibc instead just defines the > > structures itself for each of the arch

Re: [PATCH] Introducing SAD (Sum of Absolute Differences) operation to GCC vectorizer.

2013-11-05 Thread Cong Hou
Thank you for your detailed explanation. Once GCC detects a reduction operation, it will automatically accumulate all elements in the vector after the loop. In the loop the reduction variable is always a vector whose elements are reductions of corresponding values from other vectors. Therefore in

Re: [AArch64] Fix size of memory store for the vst_lane intrinsics

2013-11-05 Thread Marcus Shawcroft
On 29 October 2013 12:04, James Greenhalgh wrote: > 2013-10-29 James Greenhalgh > > * config/aarch64/arm_neon.h > (__ST2_LANE_FUNC): Better model data size. > (__ST3_LANE_FUNC): Likewise. > (__ST4_LANE_FUNC): Likewise. OK /Marcus

Re: [Patch, AArch64] Add w -> w constraint to vec_set.

2013-11-05 Thread Marcus Shawcroft
On 5 November 2013 13:59, Tejas Belagod wrote: > Changelog: > > 2013-11-05 Tejas Belagod > > gcc/ > * config/aarch64/aarch64-simd.md (vec_set): Add w -> w option > to > the constraint. OK /Marcus

Re: [PATCH, i386]: Enable soft-float multilibs for x86-32 RTEMS

2013-11-05 Thread Ralf Corsepius
On 11/05/2013 05:38 PM, Joel Sherrill wrote: On 11/5/2013 10:26 AM, Uros Bizjak wrote: On Tue, Nov 5, 2013 at 5:23 PM, Uros Bizjak wrote: Attached patch enables soft-float multilibs for x86-32 RTEMS. The patch activates SFmode and DFmode soft-float support routines. The XFmode is mapped to D

Re: [PATCH] RE: Testsuite / Cilk Plus: Include library path in compile flags in gcc.dg/cilk-plus/cilk-plus.exp

2013-11-05 Thread Jakub Jelinek
On Tue, Nov 05, 2013 at 05:04:21PM +, Iyer, Balaji V wrote: > --- gcc.dg/cilk-plus/cilk-plus.exp(revision 204396) > +++ gcc.dg/cilk-plus/cilk-plus.exp(working copy) > dg-runtest [lsort [glob -nocomplain $srcdir/c-c++-common/cilk-plus/CK/*.c]] > " -fcilkplus $ALWAYS_CFLAGS " " " > -dg-

Re: [patch] Create gimple-expr..[ch] ... was Re: RFC: gimple.[ch] break apart

2013-11-05 Thread Jeff Law
On 11/05/13 09:26, Andrew MacLeod wrote: On 10/30/2013 11:18 PM, Andrew MacLeod wrote: As a result, any gimple queries regarding types, decls, or expressions are actually tree queries. They are sprinkled throughout gimple.[ch] and gimplify.[ch], not to mention tree.[ch] as well as other parts

Re: [PATCH] RE: Testsuite / Cilk Plus: Include library path in compile flags in gcc.dg/cilk-plus/cilk-plus.exp

2013-11-05 Thread Jeff Law
On 11/05/13 10:04, Iyer, Balaji V wrote: Hello Jakub et al., I went through all your emails and the patch above will fix the following issues: 1. Bug in the fib<>.c test case where it should recursively call fib_serial instead of fib() in the serial case. 2. Remove duplicate or unwanted

[PATCH] Add gimple subclasses for every gimple code (was Re: [PATCH 0/6] Conversion of gimple types to C++ inheritance (v3))

2013-11-05 Thread David Malcolm
On Fri, 2013-11-01 at 17:36 -0400, Andrew MacLeod wrote: > On 10/31/2013 12:26 PM, David Malcolm wrote: > > [Shamelessly hijacking Andrew's thread about gimple.h refactoring, > > since this seems on-topic for that, and I'm keen to hear from Andrew on > > how the following would interact with his wo

Re: [PATCH, MPX, 2/X] Pointers Checker [7/25] Suppress BUILT_IN_CHKP_ARG_BND optimizations.

2013-11-05 Thread Jeff Law
On 11/05/13 05:02, Ilya Enkovich wrote: If I just set abnormal phi flag for SSA_NAME, SSA verifier should fail because it does not used in abnormal phi, shouldn't it? Well, we can also change the verifier to not fail when it finds that flag set on a pointer when bounds checking is enabled.

[PATCH] PR fortran/58989

2013-11-05 Thread Steve Kargl
The inlined patch fixes a problem where an array constructor is used as the shape argument in RESHAPE. The patch is straightforward and self-explanatory. Regression tested on x86_64-*-freebsd11 Ok? 2013-11-05 Steven G. Kargl PR fortran/58989 * check.c (gfc_check_reshape): ensu

Re: [PATCH, MPX, 2/X] Pointers Checker [7/25] Suppress BUILT_IN_CHKP_ARG_BND optimizations.

2013-11-05 Thread Jeff Law
On 11/04/13 06:27, Richard Biener wrote: On Thu, Oct 31, 2013 at 10:02 AM, Ilya Enkovich wrote: Hi, Here is a patch which hadles the problem with optimization of BUILT_IN_CHKP_ARG_BND calls. Pointer Bounds Checker expects that argument of this call is a default SSA_NAME of the PARM_DECL who

[PATCH] RE: Testsuite / Cilk Plus: Include library path in compile flags in gcc.dg/cilk-plus/cilk-plus.exp

2013-11-05 Thread Iyer, Balaji V
Hello Jakub et al., I went through all your emails and the patch above will fix the following issues: 1. Bug in the fib<>.c test case where it should recursively call fib_serial instead of fib() in the serial case. 2. Remove duplicate or unwanted test case flags (e.g. there were cases were

Re: [PATCH] gengtype: support empty GTY-marked structs

2013-11-05 Thread Jeff Law
On 11/05/13 09:20, David Malcolm wrote: I'm attaching a patch which allows gengtype to handle GTY-marked structs that have no fields. These are useful for supporting the "have a subclass for every gimple code" idea from: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00057.html (and am bootstra

Re: [RFC] libgcov.c re-factoring and offline profile-tool

2013-11-05 Thread Xinliang David Li
Those functions are not referencing each other, but they get used in different logical groups, so at least control granularity can be improved. For instance, 1) all gcov user interfaces such as gcov_dump, gcov_flush, gcov_reset are either not used at all, or all of them are used -- there is no need

Re: [PATCH, i386]: Enable soft-float multilibs for x86-32 RTEMS

2013-11-05 Thread Uros Bizjak
On Tue, Nov 5, 2013 at 5:23 PM, Uros Bizjak wrote: > Attached patch enables soft-float multilibs for x86-32 RTEMS. The > patch activates SFmode and DFmode soft-float support routines. The > XFmode is mapped to DFmode due to lack of XFmode support in > soft-float. We already disable FPU return reg

Re: [PATCH, i386]: Enable soft-float multilibs for x86-32 RTEMS

2013-11-05 Thread Joel Sherrill
On 11/5/2013 10:26 AM, Uros Bizjak wrote: > On Tue, Nov 5, 2013 at 5:23 PM, Uros Bizjak wrote: > >> Attached patch enables soft-float multilibs for x86-32 RTEMS. The >> patch activates SFmode and DFmode soft-float support routines. The >> XFmode is mapped to DFmode due to lack of XFmode support

[PATCH, i386]: Enable soft-float multilibs for x86-32 RTEMS

2013-11-05 Thread Uros Bizjak
Hello! Attached patch enables soft-float multilibs for x86-32 RTEMS. The patch activates SFmode and DFmode soft-float support routines. The XFmode is mapped to DFmode due to lack of XFmode support in soft-float. We already disable FPU return registers for -mno-80387, so ABI is already changed for

[PATCH] gengtype: support empty GTY-marked structs

2013-11-05 Thread David Malcolm
I'm attaching a patch which allows gengtype to handle GTY-marked structs that have no fields. These are useful for supporting the "have a subclass for every gimple code" idea from: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00057.html (and am bootstrapping a patch that does that, which needed

Re: libsanitizer merge from upstream r191666

2013-11-05 Thread Peter Bergner
On Tue, 2013-11-05 at 08:19 +0100, Jakub Jelinek wrote: > On Mon, Nov 04, 2013 at 05:48:31PM -0800, Konstantin Serebryany wrote: > > Hi Peter. > > Does this also mean that asan in llvm trunk is broken for Power? > > We'll need to fix it there too (or, in fact, first). > > I bet on all targets, not

Re: [Patch] Implementation of n3793

2013-11-05 Thread Paolo Carlini
Hi, On 11/05/2013 04:51 PM, Jonathan Wakely wrote: On 3 November 2013 11:30, Paolo Carlini wrote: On 11/03/2013 12:19 PM, Jonathan Wakely wrote: Yes, Paolo pointed out these are failing on 32-bit targets, I've got a patch coming. Luc is there any reason not to just replace all those large cons

[C11-atomic] Miscellaneous fixes 8/n

2013-11-05 Thread Joseph S. Myers
I've applied this further patch to C11-atomic branch to ensure that the c11-atomic-exec-5.c test is only run where floating-point exceptions are actually supported, to avoid spurious failures for soft-float configurations (or any systems that have pthreads but not ). 2013-11-05 Joseph Myers

Re: [Patch] Implementation of n3793

2013-11-05 Thread Jonathan Wakely
On 3 November 2013 11:30, Paolo Carlini wrote: > On 11/03/2013 12:19 PM, Jonathan Wakely wrote: >> >> Yes, Paolo pointed out these are failing on 32-bit targets, I've got a >> patch coming. Luc is there any reason not to just replace all those large >> constants with 0x1234ABCD, which fits in a lon

patch implementing a new pass for register-pressure relief through live range shrinkage

2013-11-05 Thread Vladimir Makarov
I'd like to add a new experimental optimization to the trunk. This optimization was discussed on RA BOF of this summer GNU Cauldron. It is a register pressure relief through live-range shrinkage. It is implemented on the scheduler base and uses register-pressure insn scheduling infrastructur

  1   2   >