Re: Testsuite tweaks for the SPARC

2013-07-28 Thread Andreas Schwab
Eric Botcazou ebotca...@adacore.com writes: Index: gcc.dg/vect/pr57705.c === --- gcc.dg/vect/pr57705.c (revision 201177) +++ gcc.dg/vect/pr57705.c (working copy) @@ -61,5 +61,6 @@ main () return 0; } -/* {

Re: [PATCH 03/11] Handwritten part of conversion of passes to C++ classes

2013-07-28 Thread Basile Starynkevitch
On Fri, 2013-07-26 at 11:04 -0400, David Malcolm wrote: This patch is the hand-written part of the conversion of passes from C structs to C++ classes. It does not work without the subsequent autogenerated part, which is huge. Given that the autogenerated part of the conversion is very large

Re: [Patch] Refractor Thompson matcher

2013-07-28 Thread Paolo Carlini
Hi, On 07/28/2013 06:13 AM, Tim Shen wrote: Refractor the whole Thompson matcher using the queue-based(BFS) Bellman-Ford algorithm. Fix the grouping problem. Refactor, refactoring, etc, no 'r'. If the grouping problem is now fixed, would it make sense to add corresponding testcases? Paolo.

Re: [Patch] Refractor Thompson matcher

2013-07-28 Thread Tim Shen
On Sun, Jul 28, 2013 at 5:12 PM, Paolo Carlini paolo.carl...@oracle.com wrote: Refactor, refactoring, etc, no 'r'. Thanks :) If the grouping problem is now fixed, would it make sense to add corresponding testcases? They are already added by http://gcc.gnu.org/ml/gcc-cvs/2013-07/msg00643.html

[Patch, Aarch64]: Handle return address via. frame pointer

2013-07-28 Thread Venkataramanan Kumar
Hi Maintainers, This patch adds supports to handle return address via. frame pointer. gcc/ChangeLog - 2013-07-28 Venkataramanan Kumar venkataramanan.ku...@linaro.org * config/aarch64/aarch64.c (aarch64_return_addr): Handle returning address from a frame.

[patch, fortran] PR Detect same values in vector expression subscripts

2013-07-28 Thread Thomas Koenig
Hello world, this patch yields an error for identical values in vector expression subscripts. The algorithm is O(n**2) because a) It would be impossible to detect a([i,i]) otherwise b) This is not likely to be a performance bottleneck because people don't use large vector indices. (as

Re: [Patch] Refractor Thompson matcher

2013-07-28 Thread Paolo Carlini
Hi, On 07/28/2013 12:18 PM, Tim Shen wrote: They are already added by http://gcc.gnu.org/ml/gcc-cvs/2013-07/msg00643.html (though I found the changelog entry used old file names, I'll fix it later). This time it's the BFS approach that can correctly handle the problem instead of the DFS one.

Re: [Patch] Refractor Thompson matcher

2013-07-28 Thread Tim Shen
On Sun, Jul 28, 2013 at 9:44 PM, Paolo Carlini paolo.carl...@oracle.com wrote: I see. I was wondering if in this development stage it would be convenient to have somewhere a parameter allowing to switch by hand such internal details, useful for testing purposes too. Eventually may or may not go

Re: [patch, fortran] PR Detect same values in vector expression subscripts

2013-07-28 Thread Tobias Burnus
Hello Thomas, Thomas Koenig wrote: this patch yields an error for identical values in vector expression Regression-tested. OK for trunk? + { + if (n-iterator != NULL) + continue; + +

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue 4440072)

2013-07-28 Thread mrich30
I'm not aware of any significant use of -g1. It is very rare for anyone to mention it in a bug report for instance. Once upon a time (before 2002-03-19), it was used for compiling libgcc, but that was just to ensure that it got tested somewhere. From my Cisco experience, I would agree

[PATCH]: Fix PR middle-end/56382 -- Only move MODE_COMPLEX_FLOAT by parts if we can create pseudos

2013-07-28 Thread John David Anglin
This patch fixes PR middle-end/56382 on hppa64-hp-hpux11.11. The patch prevents moving a complex float by parts if we can't create pseudos. On a big endian 64-bit target, we need a psuedo to move a complex float and this fails during reload. OK for trunk? Dave -- John David Anglin

Re: Testsuite tweaks for the SPARC

2013-07-28 Thread Eric Botcazou
That doesn't work on ia64. Yeah, there are so many vectorizer failures on IA-64 that I gave up looking at them some time ago. Maybe vect_pack_trunc should be false there too. At least bb-slp-32.c now passes, so the overall number of failures hasn't increased. :-) -- Eric Botcazou

Re: [PATCH] Enable non-complex math builtins from C99 for Bionic

2013-07-28 Thread Michael Eager
On 07/27/13 15:18, Alexander Ivchenko wrote: Hi Joseph, thanks for your comments. I updated the patch: 2013/7/9 Joseph S. Myers jos...@codesourcery.com: * It looks rather like microblaze*-*-* don't use elfos.h, so meaning semantics aren't preserved for those (non-Linux) targets either.

Re: [patch, fortran] PR Detect same values in vector expression subscripts

2013-07-28 Thread Mikael Morin
Le 28/07/2013 14:57, Thomas Koenig a écrit : Hello world, this patch yields an error for identical values in vector expression subscripts. The algorithm is O(n**2) because a) It would be impossible to detect a([i,i]) otherwise b) This is not likely to be a performance bottleneck because

Re: [Patch] Refractor Thompson matcher

2013-07-28 Thread Paolo Carlini
On 07/28/2013 05:50 PM, Tim Shen wrote: On Sun, Jul 28, 2013 at 9:44 PM, Paolo Carlini paolo.carl...@oracle.com wrote: I see. I was wondering if in this development stage it would be convenient to have somewhere a parameter allowing to switch by hand such internal details, useful for testing

Re: Testsuite tweaks for the SPARC

2013-07-28 Thread Andreas Schwab
Eric Botcazou ebotca...@adacore.com writes: At least bb-slp-32.c now passes, so the overall number of failures hasn't increased. :-) It's still an XFAIL, though. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 And

Re: [PATCH] Add atomic type qualifier

2013-07-28 Thread Joseph S. Myers
On Fri, 26 Jul 2013, Andrew MacLeod wrote: What it doesn't do: * It doesn't implement the stdatomic.h header - do you intend that to be provided by GCC or glibc? (Substantive review of the full patch still to come.) * It doesn't implement the C11 expression expansion into atomic

Re: [patch, fortran] PR Detect same values in vector expression subscripts

2013-07-28 Thread Thomas Koenig
Hi Tobias and Mikael, Something went wrong with the indentation of the last two lines. Fixed. Additionally: How about simply returning with an return false;? After some more thinking, I used the option that you suggested. We'll see if we get feedback from users who want something else, if

[PATCH, libgcc] Fix licenses on several files

2013-07-28 Thread Maxim Kuvyrkov
While verifying license compliance for GCC and its libraries I noticed that several libgcc files that end up in the final library are licensed under GPL-3.0+ instead of GPL-3.0-with-GCC-exception. This is, obviously, was not the intention of developers who just copied wrong boilerplate text,

Re: [Patch] Refractor Thompson matcher

2013-07-28 Thread Tim Shen
On Mon, Jul 29, 2013 at 1:08 AM, Paolo Carlini paolo.carl...@oracle.com wrote: Oh well, thanks. But then I expect specific testcases to come with it, for the various values of the parameter, both the default and the other other values. Otherwise, the idea isn't really immediately useful. See

Re: [patch] implement Cilk Plus simd loops on trunk

2013-07-28 Thread Jason Merrill
On 07/27/2013 05:31 AM, Aldy Hernandez wrote: trunk, but it depends on the OMP_SIMD patch which is also awaiting review (actually, just the vectorizer bits since Jakub wrote and can pre-approve the actual OMP changes): Oh, right. I can rebase off a more recent trunk if you prefer, or I can

Re: [PATCH] Enable non-complex math builtins from C99 for Bionic

2013-07-28 Thread Alexander Ivchenko
2013/7/28 Michael Eager ea...@eagerm.com: On 07/27/13 15:18, Alexander Ivchenko wrote: Hi Joseph, thanks for your comments. I updated the patch: 2013/7/9 Joseph S. Myers jos...@codesourcery.com: * It looks rather like microblaze*-*-* don't use elfos.h, so meaning semantics aren't

[Patch, Aarch64] : Macros for profile code generation to enable gprof support

2013-07-28 Thread Venkataramanan Kumar
Hi Maintainers, This patch defines some macros that are needed for profile generation support in Aarch64. I tested this patch on top of the patch http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01333.html Regression tested with aarch64-none-elf with V8 foundation model after re basing to latest