Re: [build, i386] Disable local dynamic TLS model on Solaris/x86 if as/ld cannot handle it

2014-03-07 Thread Paolo Bonzini
Il 05/03/2014 16:53, Rainer Orth ha scritto: When using GNU as with Solaris ld, the TLS local dynamic tests (gcc.dg/torture/tls/run-ld.c etc.) FAIL to execute before Solaris 11: the programs crash with an illegal instruction: e.g. in the gcc.dg/lto/20090210 test, we have 0x8050ce0 main+48: mov

Re: [PATCH] [lto/55113] Fix use of -fshort-double with -flto for powerpc

2014-03-07 Thread Richard Biener
On Fri, Mar 7, 2014 at 12:29 AM, Paulo Matos pa...@matos-sorge.com wrote: On 06/03/14 11:19, Richard Biener wrote: On Wed, Mar 5, 2014 at 12:55 PM, Paulo Matos pa...@matos-sorge.com wrote: On 05/03/2014 11:51, Richard Biener wrote: On Wed, Mar 5, 2014 at 12:43 PM, Dominique Dhumieres

Re: [PATCH] Mark fira-loop-pressure as optimization

2014-03-07 Thread Richard Biener
On Fri, Mar 7, 2014 at 12:36 AM, Paulo J. Matos pa...@matos-sorge.com wrote: On 06/03/14 23:34, Paulo J. Matos wrote: Hi, This patch marks fira-loop-pressure as an optimization so it shows in gcc --help=optimizers. 2014-03-06 Paulo Matos pa...@matos-sorge.com * common.opt

Re: [PATCH] Fix up make bootstrap-lean; make install (PR bootstrap/58572)

2014-03-07 Thread Richard Biener
On Thu, Mar 6, 2014 at 7:58 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! As discussed in the PR, doing make bootstrap-lean make install right now may fail (if host compiler is too old), or recompile various objects of the compiler with system gcc before it is installed. This happens

[PATCH] Fix incorrect byte swap detection (PR tree-optimization/60454)

2014-03-07 Thread Thomas Preud'homme
The bswap optimization has a bug where a code can be incorrectly detected as doing a byte swap, therefore leading to change of behavior for the program compiled. This is tracked as PR60454. The patch below fixes the issue. Best regards, Thomas Preud'homme diff --git a/gcc/ChangeLog

Re: [Testsuite] Skip torture/pr60183.c for AVR target

2014-03-07 Thread Chung-Ju Wu
2014-02-17 20:00 GMT+08:00 Rainer Orth r...@cebitec.uni-bielefeld.de: Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com writes: The newly added gcc.dg/torture/pr60183.c test fails for the AVR target with an array size too large error. This patch marks the test as unsupported for AVR,

Re: [PATCH ARM]: Fix more -mapcs-frame failures

2014-03-07 Thread Christian Bruel
Hi Ramana, Thanks for your comments, Please respin using plus_constant instead of gen_addsi3. Here is my feeling about this: I experimented on using plus_constant instead of gen_addsi3. But there are cases when the emitted code is not equivalent for large frames (!const_ok_for_op (val,

Re: [PATCH 1/4] [GOMP4] [Fortran] OpenACC 1.0+ support in fortran front-end

2014-03-07 Thread Ilmir Usmanov
Hi Tobias! Thanks a lot for your review! !$acc loop outer_loop: do i = 1, n inner_loop: do j = 1,m ! ... cycle outer_loop exit outer_loop end do inner_loop end do outer_loop That's a new Fortran 2008 feature that CYCLE and EXIT can leave the inner DO loop. I added

Re: [PATCH 2/4] [GOMP4] [Fortran] OpenACC 1.0+ support in fortran front-end

2014-03-07 Thread Ilmir Usmanov
OpenACC 1.0 fortran FE support -- matching and resolving. * gcc/fortran/openmp.c (gfc_free_omp_clauses): Remove also OpenACC clauses. (gfc_free_expr_list): New function to clear expression list. (match_oacc_expr_list): New function to match expression list.

Re: [PATCH 3/4] [GOMP4] [Fortran] OpenACC 1.0+ support in fortran front-end

2014-03-07 Thread Ilmir Usmanov
OpenACC 1.0 fortran FE support -- translation to GENERIC. gcc/fortran/ * trans-decl.c (gfc_generate_function_code): Insert OACC_DECLARE GENERIC node. * trans-openmp.c (gfc_convert_expr_to_tree): New helper function. (gfc_trans_omp_array_reduction): Support also OpenACC.

Re: [PATCH] Fix up make bootstrap-lean; make install (PR bootstrap/58572)

2014-03-07 Thread Jakub Jelinek
On Fri, Mar 07, 2014 at 08:26:47AM +0100, Paolo Bonzini wrote: Il 06/03/2014 19:58, Jakub Jelinek ha scritto: Bootstrapped/regtested on x86_64-linux and i686-linux and tested also with make bootstrap-lean; make install, ok for trunk? Nice fix! Did you check that on a regular bootstrap

Re: [PATCH] Fix up make bootstrap-lean; make install (PR bootstrap/58572)

2014-03-07 Thread Paolo Bonzini
Il 07/03/2014 11:45, Jakub Jelinek ha scritto: On Fri, Mar 07, 2014 at 08:26:47AM +0100, Paolo Bonzini wrote: Il 06/03/2014 19:58, Jakub Jelinek ha scritto: Bootstrapped/regtested on x86_64-linux and i686-linux and tested also with make bootstrap-lean; make install, ok for trunk? Nice

Re: Patch RFC: Use internal qsort function in libbacktrace

2014-03-07 Thread Paolo Carlini
Hi, On 03/05/2014 04:34 AM, Ian Lance Taylor wrote: The GNU glibc qsort function will call malloc in some cases. That makes it unsuitable for libbacktrace, which is intended to work when called from a signal handler. This patch changes libbacktrace to use an internal qsort function. I'm

[PATCH] [gomp4] Initial OpenACC support to C++ front-end

2014-03-07 Thread Ilmir Usmanov
Hi Thomas! I prepared simple patch to add support of OpenACC data, kernels and parallel constructs to C++ FE. It adds support of data clauses too. OK to gomp4 branch? -- Ilmir.

Re: [PATCH 2/2] [gomp4] Initial OpenACC support to C++ front-end

2014-03-07 Thread Ilmir Usmanov
Initial OpenACC support to C++ front-end -- tests * gcc/testsuite/c-c++-common/goacc/deviceptr-1.c: Move to ... * gcc/testsuite/gcc.dg/goacc/deviceptr-1.c ... here. * gcc/testsuite/g++.dg/goacc/goacc.exp: New test directory. * gcc/testsuite/g++.dg/goacc-gomp/goacc-gomp.exp:

Re: [PATCH 1/2] [gomp4] Initial OpenACC support to C++ front-end

2014-03-07 Thread Ilmir Usmanov
Initial OpenACC support to C++ front-end: parallel, kernels and data construct with data clauses. gcc/cp/ * cp-tree.h (finish_oacc_data): New function prototype. (finish_oacc_kernels, finish_oacc_parallel): Likewise. * parser.c (cp_parser_omp_clause_name): Support data

Re: [build, i386] Disable local dynamic TLS model on Solaris/x86 if as/ld cannot handle it

2014-03-07 Thread Rainer Orth
Hi Paolo, Perhaps GCC would gain from having a separate maintainer for the gcc/ target-specific configuration (configure.ac/config.gcc/config/*/*.h basically)... guess who I would propose for such a role :) ... and add the corresponding files in libgcc for good measure :-) But seriously,

Re: [PATCH] Fix incorrect byte swap detection (PR tree-optimization/60454)

2014-03-07 Thread Marcus Shawcroft
On 7 March 2014 09:35, Thomas Preud'homme thomas.preudho...@arm.com wrote: +++ b/gcc/testsuite/gcc.dg/optimize-bswapsi-2.c @@ -0,0 +1,26 @@ +/* { dg-do compile { target arm*-*-* alpha*-*-* i?86-*-* powerpc*-*-* rs6000-*-* x86_64-*-* s390*-*-* } } */ I'm fairly sure the target list here

Re: Patch RFC: Use internal qsort function in libbacktrace

2014-03-07 Thread Rainer Orth
Hi Paolo, On 03/05/2014 04:34 AM, Ian Lance Taylor wrote: The GNU glibc qsort function will call malloc in some cases. That makes it unsuitable for libbacktrace, which is intended to work when called from a signal handler. This patch changes libbacktrace to use an internal qsort function.

Re: Patch RFC: Use internal qsort function in libbacktrace

2014-03-07 Thread Paolo Carlini
Hi, On 03/07/2014 01:12 PM, Rainer Orth wrote: Hi Paolo, On 03/05/2014 04:34 AM, Ian Lance Taylor wrote: The GNU glibc qsort function will call malloc in some cases. That makes it unsuitable for libbacktrace, which is intended to work when called from a signal handler. This patch changes

Re: [PATCH] Fix incorrect byte swap detection (PR tree-optimization/60454)

2014-03-07 Thread Richard Biener
On Fri, Mar 7, 2014 at 12:45 PM, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 7 March 2014 09:35, Thomas Preud'homme thomas.preudho...@arm.com wrote: +++ b/gcc/testsuite/gcc.dg/optimize-bswapsi-2.c @@ -0,0 +1,26 @@ +/* { dg-do compile { target arm*-*-* alpha*-*-* i?86-*-*

[jit] Tweaks to location-handling

2014-03-07 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ * internal-api.c (gcc::jit::playback::context::handle_locations): Add a disabled call to line_table_dump, in case it's handy for debugging in the future. (gcc::jit::playback::context::set_tree_location): Assert that

Re: [C++ Patch] Remove dead code

2014-03-07 Thread Jason Merrill
OK. Jason

Re: [RFC] Do not consider volatile asms as optimization barriers #1

2014-03-07 Thread Richard Sandiford
Bernd Schmidt ber...@codesourcery.com writes: On 03/03/2014 11:01 PM, Richard Sandiford wrote: I'll run a full test overnight, but does this look like it might be a way out, at least for 4.9? Pretty much agree with everything you've written in this thread. I think this patch is fine.

Re: Patch RFC: Use internal qsort function in libbacktrace

2014-03-07 Thread Ian Lance Taylor
On Fri, Mar 7, 2014 at 4:12 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Hi Paolo, On 03/05/2014 04:34 AM, Ian Lance Taylor wrote: The GNU glibc qsort function will call malloc in some cases. That makes it unsuitable for libbacktrace, which is intended to work when called from a

libbacktrace patch committed: Speed up sort

2014-03-07 Thread Ian Lance Taylor
The new libbacktrace sort routine has, no doubt, many flaws, but one is quite significant. The backtrace data tends to be roughly sorted in practice (though unfortunately not perfectly sorted). By pivoting on the first element in the array, the sort routine was tending to maximize the number of

PATCH to add -std=c++14

2014-03-07 Thread Jason Merrill
It's looking very likely that we will have a new C++ standard this year, so I'm going to go ahead and add the -std=c++14 flag for 4.9; I just won't advertise it yet. Tested x86_64-pc-linux-gnu, applying to trunk. commit f966b130e3b011c6ac2169472fdf6508b5373eeb Author: Jason Merrill

[patch] Remove two maintainers of avr port

2014-03-07 Thread Denis Chertykov
Committed. 2014-03-07 Denis Chertykov cherty...@gmail.com * MAINTAINERS: Remove avr maintainers: Anatoly Sokolov and Eric Weddington Index: MAINTAINERS === --- MAINTAINERS (revision 208404) +++ MAINTAINERS (working copy)

Re: [PATCH] Fix PR c++/60393

2014-03-07 Thread Jason Merrill
On 03/04/2014 04:46 PM, Adam Butcher wrote: PR c++/60393 * parser.c (cp_parser_parameter_declaration_clause): Move generic function template unwinding on error into a move general location, ... (cp_parser_error): ... here. Hmm, I'm uncomfortable with this

[PATCH, libgcc]: Avoid warning: array subscript is above array bounds when compiling crtstuff.c

2014-03-07 Thread Uros Bizjak
Hello! Attached patch avoids a bunch of: ../../../gcc-svn/trunk/libgcc/crtstuff.c: In function 'frame_dummy': ../../../gcc-svn/trunk/libgcc/crtstuff.c:463:19: warning: array subscript is above array bounds [-Warray-bounds] if (__JCR_LIST__[0]) ^ when compiling libgcc.

Re: [C++ Patch/RFC] PR 60389

2014-03-07 Thread Jason Merrill
Inherited constructors inherit 'constexpr' from the designated base; B::B isn't constexpr because A::A isn't, and we should say that at the beginning of is_valid_constexpr_fn. Jason

[C++ Patch] PR 58609

2014-03-07 Thread Paolo Carlini
Hi, in this regression we ICE during error recovery, much, much later, in the middle-end. In order to avoid that it seems to me that we can safely return NULL_TREE. I'm also taking the occasion to change a pair of errors to error + inform (this is also consistent with eg, the explain-once

Re: [C++ Patch] PR 58609

2014-03-07 Thread Jason Merrill
OK. Jason

Re: [wwwdocs] Add Porting to GCC 4.9 document

2014-03-07 Thread Jonathan Wakely
On 04/03/14 22:32 +, Jonathan Wakely wrote: I've added an initial Porting to GCC 4.9 page at http://gcc.gnu.org/gcc-4.9/porting_to.html So far it only contains a couple of C++ changes that caused some failures during mass rebuilds, other additions are welcome. The attached patch adds some

Merge from trunk to gccgo branch

2014-03-07 Thread Ian Lance Taylor
I merged GCC trunk revision 208403 to the gccgo branch. Ian

Re: [PATCH, libgcc]: Avoid warning: array subscript is above array bounds when compiling crtstuff.c

2014-03-07 Thread Ian Lance Taylor
On Fri, Mar 7, 2014 at 9:36 AM, Uros Bizjak ubiz...@gmail.com wrote: Attached patch avoids a bunch of: ../../../gcc-svn/trunk/libgcc/crtstuff.c: In function 'frame_dummy': ../../../gcc-svn/trunk/libgcc/crtstuff.c:463:19: warning: array subscript is above array bounds [-Warray-bounds] if

Re: [PATCH] Fix PR c++/60393

2014-03-07 Thread Adam Butcher
On 2014-03-07 17:29, Jason Merrill wrote: On 03/04/2014 04:46 PM, Adam Butcher wrote: PR c++/60393 * parser.c (cp_parser_parameter_declaration_clause): Move generic function template unwinding on error into a move general location, ... (cp_parser_error): ... here.

[jit] Reorganize jit.dg/test-functions.c

2014-03-07 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/testsuite/ * jit.dg/test-functions.c: Reorder function definitions, grouping them by subject-matter rather than by create-vs-verify phase. --- gcc/testsuite/ChangeLog.jit | 5 ++ gcc/testsuite/jit.dg/test-functions.c | 107

Re: [wwwdocs] Add Porting to GCC 4.9 document

2014-03-07 Thread Jonathan Wakely
On 07/03/14 18:08 +, Jonathan Wakely wrote: On 04/03/14 22:32 +, Jonathan Wakely wrote: I've added an initial Porting to GCC 4.9 page at http://gcc.gnu.org/gcc-4.9/porting_to.html So far it only contains a couple of C++ changes that caused some failures during mass rebuilds, other

[jit] Add GCC_JIT_FUNCTION_ALWAYS_INLINE and fix GCC_JIT_FUNCTION_INTERNAL.

2014-03-07 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ * libgccjit.h (enum gcc_jit_function_kind): Add GCC_JIT_FUNCTION_ALWAYS_INLINE. * internal-api.c (gcc::jit::recording::function::write_to_dump): Handle GCC_JIT_FUNCTION_ALWAYS_INLINE.

Re: [PATCH] Fix ARM TLS handling (PR target/58595)

2014-03-07 Thread Christophe Lyon
Hi, It seems the new test fails at execution on aarch64-none-elf (and aarch64_be-none-elf) using the ARM Foundation Model as simulator. Sorry, I don't have access to more detail right now. Christophe. On 6 March 2014 06:44, Ramana Radhakrishnan ramana@googlemail.com wrote: On Wed, Mar 5,

C++ PATCHes for C++14 testsuite regressions

2014-03-07 Thread Jason Merrill
These patches address regressions revealed by my work to run the testsuite in C++14 mode. The first patch addresses multiple failures caused by force_paren_expr wrapping simple expressions so that decltype(auto) will treat them differently. The static_cast to reference trick was breaking

Re: [PATCH] Fix PR c++/60393

2014-03-07 Thread Jason Merrill
On 03/07/2014 02:10 PM, Adam Butcher wrote: Ah right. I did wonder about tentative parsing. What about in cp_parser_skip_to_end_of_statement? That sounds OK. Jason

Re: [PATCH] Fix ARM TLS handling (PR target/58595)

2014-03-07 Thread Andrew Pinski
On Fri, Mar 7, 2014 at 11:56 AM, Christophe Lyon christophe.l...@linaro.org wrote: Hi, It seems the new test fails at execution on aarch64-none-elf (and aarch64_be-none-elf) using the ARM Foundation Model as simulator. The new testcase should have been disabled for the aarch64-elf target as

Re: [PATCH] Fix ARM TLS handling (PR target/58595)

2014-03-07 Thread Jakub Jelinek
On Fri, Mar 07, 2014 at 12:10:05PM -0800, Andrew Pinski wrote: On Fri, Mar 7, 2014 at 11:56 AM, Christophe Lyon christophe.l...@linaro.org wrote: It seems the new test fails at execution on aarch64-none-elf (and aarch64_be-none-elf) using the ARM Foundation Model as simulator. The new

Re: [PATCH] Fix ARM TLS handling (PR target/58595)

2014-03-07 Thread Jakub Jelinek
On Fri, Mar 07, 2014 at 09:14:04PM +0100, Jakub Jelinek wrote: On Fri, Mar 07, 2014 at 12:10:05PM -0800, Andrew Pinski wrote: On Fri, Mar 7, 2014 at 11:56 AM, Christophe Lyon christophe.l...@linaro.org wrote: It seems the new test fails at execution on aarch64-none-elf (and

Re: [PATCH] Fix PR c++/60033

2014-03-07 Thread Jason Merrill
On 03/04/2014 02:58 PM, Adam Butcher wrote: PR c++/60033 * pt.c (retrieve_specialization): When retrieving a capture pack from a generic lambda, remove the lambda's own template argument list prior to fetching the specialization. I think I'd prefer to handle

[wwwdocs, committed] svn.html-branches - Add NEW fortran-caf branch, retire fortran-exp

2014-03-07 Thread Tobias Burnus
Hi all, I have retired the old fortran-exp branch in svn.html. In addition, I have started a new branch fortran-caf, which I will use for coarray changes. Besides the new fortran-caf branch, there is still the somewhat active fortran-dev branch, used for the new array descriptor. Tobias

[Ada] PR ada/60411

2014-03-07 Thread Arnaud Charlet
This change enables ZCX on armel linux, and should fix PR ada/60411, at least for the native part reported in comment #1 PR ada/60411 * system-linux-armel.ads (Backend_Overflow_Checks): Set to True. (Support_64_Bit_Divides): Removed, no longer used.

Re: [patch] Remove two maintainers of avr port

2014-03-07 Thread Anatoly Sokolov
Hello. Please retain me write-after-approval status. I must return to active work on the GCC in the near future. Anatoly. - Original Message - From: Denis Chertykov cherty...@gmail.com To: gcc-patches gcc-patches@gcc.gnu.org; Анатолий Соколов ae...@post.ru; Eric Weddington

[PATCH, PR58066] preferred_stack_boundary update for tls expanded call

2014-03-07 Thread Wei Mi
Hi, This patch is to fix the problem described here: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066 I follow Ian's suggestion and set ix86_tls_descriptor_calls_expanded_in_cfun in tls_global_dynamic_64_mode and tls_local_dynamic_base_64_mode. Although 32bit doesn't have the problem,

Re: [PATCH, PR58066] preferred_stack_boundary update for tls expanded call

2014-03-07 Thread Wei Mi
Regression test is ok. Thanks, Wei. On Fri, Mar 7, 2014 at 1:26 PM, Wei Mi w...@google.com wrote: Hi, This patch is to fix the problem described here: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066 I follow Ian's suggestion and set ix86_tls_descriptor_calls_expanded_in_cfun in

Re: [PATCH, PR58066] preferred_stack_boundary update for tls expanded call

2014-03-07 Thread H.J. Lu
On Fri, Mar 7, 2014 at 1:26 PM, Wei Mi w...@google.com wrote: Hi, This patch is to fix the problem described here: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58066 I follow Ian's suggestion and set ix86_tls_descriptor_calls_expanded_in_cfun in tls_global_dynamic_64_mode and

Re: [PATCH, PR58066] preferred_stack_boundary update for tls expanded call

2014-03-07 Thread Wei Mi
Yes, x32 has the same problem. It should be tested. Fixed. Thanks, Wei. On Fri, Mar 7, 2014 at 2:06 PM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Mar 7, 2014 at 1:26 PM, Wei Mi w...@google.com wrote: Hi, This patch is to fix the problem described here:

[PATCH] Fix PR c++/60393

2014-03-07 Thread Adam Butcher
PR c++/60393 * parser.c (cp_parser_parameter_declaration_clause): Move generic function template unwinding on error into a more general location, ... (cp_parser_skip_to_end_of_statement): ... here. PR c++/60393 * g++.dg/cpp1y/pr60393.C: New

Re: [patch, libgfortran] PR60128 Wrong ouput using en edit descriptor

2014-03-07 Thread Jerry DeLisle
On 03/07/2014 02:05 PM, Dominique d'Humières wrote: Hi all! Patch for pr60128. It is basically the patch posted at = http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D60128#c7. I have made tmp = volatile (although I did not see any differences between -m32 and -m64) = and added a comment

Re: [PATCH] Fix PR c++/60033

2014-03-07 Thread Jason Merrill
On 03/07/2014 06:07 PM, Adam Butcher wrote: + /* When retrieving a capture pack from a generic lambda, remove the +lambda call op's own template argument list from ARGS. Only the +template arguments active for the closure type should be used to +

Re: [PATCH] Fix PR c++/60393

2014-03-07 Thread Jason Merrill
OK. Jason

[patch, libfortran] [4.7/4.8/4.9 Regression] PR38199 missed optimization: I/O performance

2014-03-07 Thread Jerry DeLisle
The attached patch addresses the problem identified in comment #22 of the PR. For character array internal unit reads, eat_spaces must call next_char to advance every single character until the end of the string is reached. In the case sited which is very contrived, this amounts to about 10

Re: [Patch, Fortran] -fcoarray=lib: Change handling of this_image() and num_images()

2014-03-07 Thread Tobias Burnus
Tobias Burnus wrote: Hence, the attached patch adds a this_image() and num_images() library function. As the library is only build statically and -fcoarray=lib is not widely used due to lacking communication support, I wouldn't count this patch as real ABI break and there shouldn't be a

[Patch, Fortran] Update gfortran.texi's 2003/2008 status

2014-03-07 Thread Tobias Burnus
An update the gfortran.texi's F2003/F2008 status. OK for the trunk? Tobias 2014-03-08 Tobias Burnus bur...@net-b.de * gfortran.texi (Fortran 2003 Status): Mention finalization, deferred-length character support and input rounding. (Fortran 2008 Status): Mention that at termination