Re: [PATCH] cplus-demangler, free resource after a failed call to gnu_special.

2014-05-11 Thread Ian Lance Taylor
On Sat, May 10, 2014 at 12:13 PM, Andrew Burgess wrote: > On 09/05/2014 9:53 PM, Ian Lance Taylor wrote: >> On Fri, May 9, 2014 at 7:35 AM, Andrew Burgess wrote: >> >>>if ((AUTO_DEMANGLING || GNU_DEMANGLING)) >>> { >>> success = gnu_special (work, &mangled, &decl); >>> +

[PATCH] Fix issue in uninit analysis (PR middle-end/61112)

2014-05-11 Thread Patrick Palka
Hi, This patch fixes a bogus warning generated by -Wmaybe-uninitialized. The problem is that we sometimes fail to acknowledge a defining edge belonging to a control-dependence chain because we assume that each defining edge shares the same control-dependence root. But this may not be true if a de

Re: [PATCH ARM] Improve ARM memset inlining

2014-05-11 Thread Bin.Cheng
Ping. Thanks, bin On Tue, May 6, 2014 at 12:59 PM, bin.cheng wrote: > > >> -Original Message- >> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- >> ow...@gcc.gnu.org] On Behalf Of bin.cheng >> Sent: Monday, May 05, 2014 3:21 PM >> To: Richard Earnshaw >> Cc: gcc-patches@gcc.gnu.

[DOC Patch] Remove reference to deleted macro

2014-05-11 Thread David Wohlferd
I don't have permissions to commit this patch, but I do have a release on file with the FSF. Problem description: The existing docs make reference to a macro which "is described below." However, this is the final sentence in the section; there is no "below." Turns out the macro was deleted a

Re: [RFC][ARM] TARGET_ATOMIC_ASSIGN_EXPAND_FENV hook

2014-05-11 Thread Kugan
Ping ? Thanks, Kugan On 02/05/14 19:04, Kugan wrote: > On 02/05/14 10:15, Joseph S. Myers wrote: >> It doesn't seem a good idea to me for a host-side GCC file to use the FE_* >> names for the target's FE_* values; you'd run into problems if that file >> ever ends up including the host's , direc

Re: [RFC][AARCH64] TARGET_ATOMIC_ASSIGN_EXPAND_FENV hook

2014-05-11 Thread Kugan
Ping ? Thanks, Kugan On 02/05/14 22:27, Kugan wrote: > > > On 02/05/14 20:06, Marcus Shawcroft wrote: >> On 29 April 2014 03:37, Kugan wrote: >>> >>> On 28/04/14 21:01, Ramana Radhakrishnan wrote: On 04/26/14 11:57, Kugan wrote: > Attached patch implements TARGET_ATOMIC_ASSIGN_EXPAND_F

Re: [RS6000] Fix PR61098, Poor code setting count register

2014-05-11 Thread Alan Modra
On Mon, May 12, 2014 at 08:23:16AM +0930, Alan Modra wrote: > On Sat, May 10, 2014 at 10:24:34PM -0400, David Edelsohn wrote: > > On Thu, May 8, 2014 at 10:40 PM, Alan Modra wrote: > > > rs6000_emit_set_const ... always returns a non-zero result ... > > > > Can you help clarify the removal of the

Re: [RS6000] Fix PR61098, Poor code setting count register

2014-05-11 Thread Alan Modra
On Sat, May 10, 2014 at 10:24:34PM -0400, David Edelsohn wrote: > On Thu, May 8, 2014 at 10:40 PM, Alan Modra wrote: > > rs6000_emit_set_const ... always returns a non-zero result ... > > Can you help clarify the removal of the code that tests if the > splitter failed? See above. -- Alan Modra

Committed, MMIX: Another target apologist blurb in gcc.c-torture/execute/20101011-1.c

2014-05-11 Thread Hans-Peter Nilsson
That test now looks a bit silly with the dozen+1 exceptions. But, I guess with just this one(?) test there's little sense in adding an effective target to describe that division by 0 doesn't signal. Other than keeping it in just one place, of course. But, committed. gcc/testsuite: * gcc

Re: [C PATCH] Make attributes accept enum values (PR c/50459)

2014-05-11 Thread Marek Polacek
On Sun, May 11, 2014 at 09:18:47PM +0200, Rainer Orth wrote: > No, that's wrong: avoid hardcoding target lists if at all possible. > Besides, it's wrong since it doesn't cover the Solaris (and other > non-gld linker) case. Use the init_priority effective-target keyword > instead. Also, please che

Re: [C PATCH] Make attributes accept enum values (PR c/50459)

2014-05-11 Thread Rainer Orth
Marek Polacek writes: >> The errors are >> >> /opt/gcc/work/gcc/testsuite/c-c++-common/pr50459.c:8:1: error: constructor >> priorities are not supported >> /opt/gcc/work/gcc/testsuite/c-c++-common/pr50459.c:9:1: error: destructor >> priorities are not supported > > Ah. The following untested

Re: [PING] Add const char* constructors for exception classes in

2014-05-11 Thread Oleg Endo
On 02 May 2014, at 17:57, Jonathan Wakely wrote: > On 1 May 2014 17:18, Oleg Endo wrote: >> Jonathan, >> >> now that we're in stage 1 again, I'd like to revive the issue below. Do >> you have any particular plans? How should we proceed? > > Hi Oleg, sorry for letting the thread die in Januar

Re: [Google/4-8] Support for user-guided feedback-directed library optimization

2014-05-11 Thread Xinliang David Li
> === > --- gcc/builtins.c(revision 210019) > +++ gcc/builtins.c(working copy) > @@ -87,6 +87,9 @@ static rtx result_vector (int, rtx); > #endif > static void expand_builtin_update_setjmp_buf (rtx); > static void expand_buil

[Patch, Fortran] Reject OpenMP parallelization for DO CONCURRENT

2014-05-11 Thread Tobias Burnus
While it would be nice to support "!$OMP do" for "do concurrent" loops, the OpenMP spec does not support it, yet. (Syntactically, it is a not a that simple feature as do concurrent can optionally have a MASK=, which has to be evaluated before the loop.) Thus, this patch avoids an ICE by simply

[Patch, Fortran, committed] (-fcoarray=lib) Change type of second argument to _gfortran_caf_num_images

2014-05-11 Thread Tobias Burnus
This patch changes the type of second argument to _gfortran_caf_num_images from Boolean to integer. The reason is that we have to represent three different states: a) num_images() - all existing images (in the current/"distance"-th parent team) b/c) num_images(...,failed) - those states which h

Re: [C PATCH] Make attributes accept enum values (PR c/50459)

2014-05-11 Thread Marek Polacek
On Sat, May 10, 2014 at 04:10:47PM +0200, Dominique Dhumieres wrote: > > ... > > Tested again x86_64-linux, ok now? > > > > 2014-05-02 Marek Polacek > > > > PR c/50459 > > This caused on x86_64-apple-darwin13 > > FAIL: c-c++-common/pr50459.c -std=gnu++98 (test for excess errors) > FAIL: c-

Re: Fix gcse leak

2014-05-11 Thread Richard Biener
On May 11, 2014 3:54:30 PM CEST, Steven Bosscher wrote: >Hello, > >This patch plugs a GGC leak in gcse.c, which will hold on to test insn >to save memory. But this results in holding on to entire RTL function >bodies, the function's CFG, and a lot more. > >Fix is simple: Clear the part of test_ins

Fix gcse leak

2014-05-11 Thread Steven Bosscher
Hello, This patch plugs a GGC leak in gcse.c, which will hold on to test insn to save memory. But this results in holding on to entire RTL function bodies, the function's CFG, and a lot more. Fix is simple: Clear the part of test_insn that causes this. Bootstrapped&tested on powerpc64-unknown-li

Re: [PATCH, libgfortran] Use -std=gnu11

2014-05-11 Thread Janne Blomqvist
PING On Thu, May 1, 2014 at 12:57 AM, Janne Blomqvist wrote: > Hello, > > the attached patch switches libgfortran C sources to be compiled in > gnu11 mode instead of gnu99. As the 4.9 release notes > http://gcc.gnu.org/gcc-4.9/changes.html say, > > "ISO C11 support is now at a similar level of co

Re: [PATCH, libgfortran] PR 61035 Stack overflow in GETCWD

2014-05-11 Thread Janne Blomqvist
PING On Sat, May 3, 2014 at 11:47 PM, Janne Blomqvist wrote: > Hello, > > the attached patch avoids a stack overflow crash due to not trying to > create a null-terminated duplicate of the argument char array on the > stack. Also, for the common case it avoids an extra allocation and an > extra me

Re: [PATCH GCC]Pick up more address lowering cases for ivopt and tree-affine.c

2014-05-11 Thread Bin.Cheng
On Thu, May 8, 2014 at 5:08 PM, Bin.Cheng wrote: > On Tue, May 6, 2014 at 6:44 PM, Richard Biener > wrote: >> On Tue, May 6, 2014 at 10:39 AM, Bin.Cheng wrote: >>> On Fri, Dec 6, 2013 at 6:19 PM, Richard Biener >>> wrote: > >>> Hi, >>> I split the patch into two and updated the test case. >>> T

Re: PR 61136: wide-int fallout in int_const_binop_1

2014-05-11 Thread Richard Biener
On Sat, May 10, 2014 at 9:11 PM, Richard Sandiford wrote: > The wide-int version of int_const_binop_1 has: > > static tree > int_const_binop_1 (enum tree_code code, const_tree arg1, const_tree parg2, >int overflowable) > { > [...] > tree type = TREE_TYPE (arg1); > [...] >

[PATCH] Enable Java on Cygwin-64

2014-05-11 Thread Bernd Edlinger
Hi, currently it is not possible to build GCC's libjava support on Cygwin-64. The attached patch fixes the current build-problems and the most fundamental bugs on that platform. Note you must still add --enable-threads=posix to successfully build the java language support. Boot-Strapped on Cygwi

Re: [Fortran-caf] Fix stride calculation in libgfortran/caf/single.c

2014-05-11 Thread Tobias Burnus
For those with a broken crystal ball (and being too lazy to look in the repository), here's the attachment. Tobias Burnus: A rather obvious copy and paste error ... Committed as Rev. 210309. Index: libgfortran/ChangeLog.fortran-caf =

Re: PR61140: check the phi is unique in value_replacement

2014-05-11 Thread Marc Glisse
On Sat, 10 May 2014, Andrew Pinski wrote: On Sat, May 10, 2014 at 3:53 PM, Marc Glisse wrote: Hello, in my recent phiopt patch enhancing value_replacement to optimize x!=0?x+y:y, I forgot to check that there is no other PHI (not sure how I managed to miss that since I copy-pasted the line jus

[Fortran-caf] Fix stride calculation in libgfortran/caf/single.c

2014-05-11 Thread Tobias Burnus
A rather obvious copy and paste error ... Committed as Rev. 210309. Tobias

[Fortran-caf] Extend CAF send test case

2014-05-11 Thread Tobias Burnus
The test case checked array[idx]=array and array[idx]=scalar assignment, but not scalar[idx]=scalar assignments. I have now added the check; committed as Rev. 210308. Tobias Index: gcc/testsuite/ChangeLog.fortran-caf === --- gcc/te