Re: [patch, fortran] Handle -Wextra, -fcompare-reals is implied with -Wextra

2012-09-23 Thread Thomas Koenig
Am 23.09.2012 23:46, schrieb Manuel López-Ibáñez: However, the actual problem seems to be that gfortran uses their own option variables and not the ones generated by the common machinery (which are generated in any case, so it is a bit of a waste to duplicate them). It is up to the gfortran mai

Re: [Patch] catch builtin_bswap16 construct

2012-09-23 Thread Andrew Pinski
On Fri, Sep 21, 2012 at 1:04 AM, Christophe Lyon wrote: > On 20 September 2012 09:55, Christophe Lyon > wrote: >> On 20 September 2012 09:12, Eric Botcazou wrote: The attached patch catches C constructs: (A << 8) | (A >> 8) where A is unsigned 16 bits and maps them to builti

Re: [Patch] catch builtin_bswap16 construct

2012-09-23 Thread Oleg Endo
On Thu, 2012-09-20 at 09:12 +0200, Eric Botcazou wrote: > > The attached patch catches C constructs: > > (A << 8) | (A >> 8) > > where A is unsigned 16 bits > > and maps them to builtin_bswap16(A) which can provide more efficient > > implementations on some targets. > > This belongs in tree-ssa-ma

Re: [PATCH] fix gcc.dg/torture/pr53922.c on darwin

2012-09-23 Thread Dominique Dhumieres
Jack, The patch does not seem to work on powerpc-apple-darwin9: Executing on host: /opt/gcc/darwin_buildw/gcc/xgcc -B/opt/gcc/darwin_buildw/gcc/ /opt/gcc/work/gcc/testsuite/gcc.dg/torture/pr53922.c -fno-diagnostic s-show-caret -O0 -Wl,-undefined,dynamic_lookup -lm -m32 -o ./pr53922.exe

Re: [patch, fortran] Handle -Wextra, -fcompare-reals is implied with -Wextra

2012-09-23 Thread Manuel López-Ibáñez
On 23 September 2012 21:58, Thomas Koenig wrote: > Hello world, > > the attatched patch (this time for real!) implements > -Wextra for the Fortran front end, and adds -fcompare-reals > to -Wextra. > > @manu: In the PR, you mentioned that it would be nice if > LangEnabledBy was used in the opt fil

Re: [PATCH] Cilk Plus merging to trunk (2 of n)

2012-09-23 Thread Joseph S. Myers
On Sun, 23 Sep 2012, Iyer, Balaji V wrote: > Hello Everyone, > Attached, please find a patch that will implement Cilk Plus Array > Notations for the C compiler. Array notations are indented to allow > programmers to directly express parallelism in their programs. Array > notations can be us

[patch, fortran] Handle -Wextra, -fcompare-reals is implied with -Wextra

2012-09-23 Thread Thomas Koenig
Hello world, the attatched patch (this time for real!) implements -Wextra for the Fortran front end, and adds -fcompare-reals to -Wextra. @manu: In the PR, you mentioned that it would be nice if LangEnabledBy was used in the opt files. I tried adding that to the common.opt file for Wextra, but

Re: [PATCH, libstdc++]: Avoid the badname __alpha

2012-09-23 Thread Paolo Carlini
On 09/23/2012 01:30 PM, Uros Bizjak wrote: Approved by Paolo in the PR, committed to mainline SVN. Thanks Uros! By the way, I'm wondering if the front-ends couldn't help us about this?!? For sure there is the issue that the list of BADNAMES evolves over times, but otherwise I don't see why we

Re: Fix PR tree-optimization/54669

2012-09-23 Thread Richard Guenther
On Sat, Sep 22, 2012 at 10:30 PM, Eric Botcazou wrote: > As demonstrated by the testcase, the new propagate_constants_for_unrolling can > change the EH properties of a statement when it turns array accesses with > variable index into array accesses with fixed index if -fnon-call-exception is > ena

Re: Remove duplicated block of code

2012-09-23 Thread Richard Guenther
On Sat, Sep 22, 2012 at 1:53 PM, Marc Glisse wrote: > Hello, > > this patch removes one copy of a block of code that somehow ended up > duplicated. Richard Henderson apparently already fixed part of the problem > here: > > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00120.html > > but missed this

[Patch, committed] testsuite/54677

2012-09-23 Thread Paolo Carlini
Hi, committed as obvious. Thanks, Paolo. 2012-09-23 Paolo Carlini Dominique Dhumieres PR testsuite/54677 * g++.dg/cpp0x/decltype32.C: Add -ftemplate-depth=10. Index: g++.dg/cpp0x/decltype32.C

[PATCH, libstdc++]: Avoid the badname __alpha

2012-09-23 Thread Uros Bizjak
Hello! __alpha interferes with system define for alpha-unknown-linux-gnu targets. I have took the same approach as for PR libstdc++/39835 and for consistency renamed also other greek letters to __letter_val. 2012-09-23 Uros Bizjak PR libstdc++/54675 * include/ext/random: Avoi

robustify guality self-check

2012-09-23 Thread Alexandre Oliva
I found out I wasn't getting any guality tests run. The old gdb I used wouldn't recognize line number information at -O0 or something, and this caused guality.exp to give up. -O2 -g would have worked, but passing them to guality_check in either order would cause only the first of the two to make

[PR54551] global dead debug pseudo tracking in fast-dce

2012-09-23 Thread Alexandre Oliva
This patch introduces a global mode of dead_debug tracking for use in fast DCE. If a debug use reaches the top of a basic block before finding its death point, the pending and subsequent uses of the pseudo in debug insns will all be substituted with the same debug temp, and death points will get t

Re: Fix code regression

2012-09-23 Thread Paolo Carlini
Hi, On 09/23/2012 12:08 PM, François Dumont wrote: Hi I detect this regression while running performance tests. I think it comes from: 2012-04-12 Benjamin Kosnik * include/bits/unordered_map.h (__unordered_map): Remove. (__unordered_multimap): Remove. Add aliases for __um

Fix code regression

2012-09-23 Thread François Dumont
Hi I detect this regression while running performance tests. I think it comes from: 2012-04-12 Benjamin Kosnik * include/bits/unordered_map.h (__unordered_map): Remove. (__unordered_multimap): Remove. Add aliases for __umap_traits, __umap_hashtable, __ummap_traits, __um

[RFC] Move ehcleanup pass to before early SRA

2012-09-23 Thread Eric Botcazou
Hi, we have run into optimization regressions in Ada caused by the interaction between the new GIMPLE clobbers and SRA: on the one hand GIMPLE clobbers create artificial EH handlers for aggregate variables, on the other hand SRA refuses to scalarize objects that appear in a statement that can t