PR 66685: parallel returns being misclassified

2015-07-01 Thread Richard Sandiford
This patch should restore bootstrap on hppa (and probably other targets besides). The change to use target-insns.def put more stress on the emit()/classify_insn() group of functions, which were missing a case for parallel returns. Tested with a cross-compiler that it fixes the hppa problem.

Re: [AArch64] Fall back to -fPIC if no support of -fpic relocation modifer in assembler

2015-07-01 Thread James Greenhalgh
On Mon, Jun 29, 2015 at 01:42:13PM +0100, Jiong Wang wrote: This patch fix the breakage caused by https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01913.html We fall back to -fPIC if there is no assembler support on those new relocation modifiers for -fpic. OK for trunk? gcc/ *

Re: PR 66685: parallel returns being misclassified

2015-07-01 Thread Richard Biener
On Wed, Jul 1, 2015 at 10:27 AM, Richard Sandiford richard.sandif...@arm.com wrote: This patch should restore bootstrap on hppa (and probably other targets besides). The change to use target-insns.def put more stress on the emit()/classify_insn() group of functions, which were missing a case

Re: [AArch64/testsuite] Restrict pic-small.c by a new directive check_effective_target_aarch64_small_fpic

2015-07-01 Thread James Greenhalgh
On Tue, Jun 30, 2015 at 03:17:00PM +0100, Jiong Wang wrote: As discussed here https://gcc.gnu.org/ml/gcc-patches/2015-06/msg02151.html, Since have enabled binutils feature detection when configuring gcc, -fpic will not be enabled if there is no binutils support on those new relocation

[COMMITTED][AArch64] Fall back to -fPIC if no support of -fpic relocation modifer in assembler

2015-07-01 Thread Jiong Wang
James Greenhalgh writes: On Mon, Jun 29, 2015 at 01:42:13PM +0100, Jiong Wang wrote: This patch fix the breakage caused by https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01913.html We fall back to -fPIC if there is no assembler support on those new relocation modifiers for -fpic.

[COMMITTED][AArch64/testsuite] Restrict pic-small.c by a new directive check_effective_target_aarch64_small_fpic

2015-07-01 Thread Jiong Wang
James Greenhalgh writes: On Tue, Jun 30, 2015 at 03:17:00PM +0100, Jiong Wang wrote: As discussed here https://gcc.gnu.org/ml/gcc-patches/2015-06/msg02151.html, Since have enabled binutils feature detection when configuring gcc, -fpic will not be enabled if there is no binutils

[COMMITTED][obivious] Document several AArch64-specific test directives

2015-07-01 Thread Jiong Wang
committed as 225233. 2015-07-01 Jiong Wang jiong.w...@arm.com gcc/ * doc/sourcebuild.texi (AArch64-specific attributes): Document aarch64_tiny, aarch64_small, aarch64_large, aarch64_little_endian, aarch64_big_endian. Index: gcc/ChangeLog

[PATCH] Adjust argument ordering in pa/linux-atomic.c

2015-07-01 Thread John David Anglin
The attached change revises the argument order of the __kernel_cmpxchg and __kernel_cmpxchg2 calls so that they now match the argument order of the light-weight syscalls. All callers were updated. The clobbers and written arguments of the asms in these two functions were reviewed and updated

C++ PATCH to change default dialect to C++14

2015-07-01 Thread Jason Merrill
I've been threatening to do this for a couple of months, and now that the regressions are under control I think it's time. This patch changes the default C++ dialect to C++14. Tested x86_64-pc-linux-gnu, applying to trunk. commit 6f2b3941ffb3734cecc05507dd8ed46f6a6d4f2d Author: Jason Merrill

Re: [PATCH/RFC] Make loop-header-copying more aggressive, rerun before tree-if-conversion

2015-07-01 Thread Jeff Law
On 06/30/2015 10:04 AM, Alan Lawrence wrote: Jeff Law wrote: Thanks. Does running the phi-only propagator after the loop header copying help? At first glance it would seem that it ought to propagate the values of those degenerate PHIs then eliminate those PHIs. It was written to cleanup

Re: RFA: Use target-insns.def for casesi and tablejump

2015-07-01 Thread Jeff Law
On 06/30/2015 02:58 PM, Richard Sandiford wrote: This patch uses target-insns.def to define casesi and tablejump. It's the first point in the series at which we need access to the instruction code, so the patch adds that to targetm as well. Bootstrapped regression-tested on x86_64-linux-gnu

Re: C++ PATCH to change default dialect to C++14

2015-07-01 Thread Marek Polacek
On Wed, Jul 01, 2015 at 02:26:59PM -0400, Jason Merrill wrote: I've been threatening to do this for a couple of months, and now that the regressions are under control I think it's time. This patch changes the default C++ dialect to C++14. Great! It seems that our cppopts.texi is very

Re: libgomp: libgomp.oacc-c-c++-common/lib-62.c is for acc_device_nvidia only

2015-07-01 Thread Tom de Vries
On 19/06/15 09:50, Thomas Schwinge wrote: Hi! On Tue, 5 May 2015 16:09:18 +0200, I wrote: I don't know why libgomp.oacc-c-c++-common/lib-62.c contains this explicit acc_init call with acc_device_nvidia -- generally, the test cases should not contain such unconditional statements. So, let's

[scalar-storage-order] Add pragma scalar_storage_order

2015-07-01 Thread Eric Botcazou
This adds a #pragma scalar_storage_order directive to the C family of languages that makes it possible to specify a default SSO. Tested on x86_64-suse-linux. applied on the branch. * doc/extend.texi (Structure-Packing Pragmas): Rename into... ( Structure-Layout Pragmas):

Re: [PING] [RFC] Sanitize rtx_addr_can_trap_p_1

2015-07-01 Thread Jakub Jelinek
On Wed, Jul 01, 2015 at 02:31:41PM +0200, Bernd Edlinger wrote: the bogus offsets-issue is now entered in bugzilla, see PR66614. It is however only a very minor issue, and does probably have no impact on the generated code at all. How should I continue with the rtx_addr_can_trap-patch?

[PATCH][10/n] Remove GENERIC stmt combining from SCCVN

2015-07-01 Thread Richard Biener
This merges the complete comparison patterns from the match-and-simplify branch, leaving incomplete implementations of fold-const.c code alone. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-07-01 Richard Biener rguent...@suse.de * fold-const.c

[PATCH] Improve DOM condition handling

2015-07-01 Thread Richard Biener
I've just noticed that DOM doesn't handle if (x = y) return 1; if (y == x) abort (); in that it records y != x on the else edge but not !(y == x). The following fixes that. Bootstrap regtest running on x86_64-unknown-linux-gnu. Richard. 2015-07-01 Richard Biener

[gomp4] implicit firstprivate and other testcase fixes

2015-07-01 Thread Chung-Lin Tang
This patch notices the index variable of an acc loop (internally an OMP_FOR) inside an OpenACC construct, and completes the implicit firstprivate behavior as described in the spec. The firstprivate clauses and FIXME in libgomp.oacc-c-c++-common/parallel-loop-2.h has also been removed together in

[patch] Change valid arguments for --with-default-libstdcxx-abi

2015-07-01 Thread Jonathan Wakely
I intend to commit this to trunk and the gcc-5-branch either today or tomorrow. I made a mistake in using c++98 and c++11 as the valid arguments for the ABI configure option, because it confuses people into thinking that it is affected by, or affects, the -std=c++11 option. In fact the ABI mode

Re: [Patch 0/4] PowerPC64 Linux split stack support

2015-07-01 Thread Lynn A. Boger
If further testing is needed on this patch I can do it, but I need more information what variations need to be tested? It's not clear to me what distro/gcc/glibc versions and type of build causes the error. I have not been able to reproduce the original problem. On 06/15/2015 01:58 PM,

Re: [Patch 0/4] PowerPC64 Linux split stack support

2015-07-01 Thread Andreas Schwab
Lynn A. Boger labo...@linux.vnet.ibm.com writes: It's not clear to me what distro/gcc/glibc versions and type of build causes the error. I have not been able to reproduce the original problem. The failure mode is quite obvious: go_langhook_init_options_struct is called before the options are

[gomp4] Remove clause from device_type options

2015-07-01 Thread James Norris
Hi, The independent clause is not available for use with device_type clauses associated with loop directives. This patch removes the usage. Committed to gomp-4_0-branch Jim diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c index bcbd163..88e68ae 100644 --- a/gcc/c/c-parser.c +++

[PATCH, committed] PR jit/66700: set TREE_ADDRESSABLE when building an ADDR_EXPR

2015-07-01 Thread David Malcolm
Tested by visual inspection of gimple dump of reproducer supplied by bug reporter [1], and via make check-jit; takes jit.sum from 8234 to 8289 passes. Committed to trunk as r225248. [1] https://gcc.gnu.org/ml/jit/2015-q2/msg00134.html gcc/jit/ChangeLog: PR jit/66700 *

Re: [gomp] openacc reduction cleanup

2015-07-01 Thread Cesar Philippidis
On 06/30/2015 11:01 AM, Cesar Philippidis wrote: This patch has been applied to gomp-4_0-branch. It was brought to my attention that I forgot to attach the patch. Cesar 2015-06-30 Cesar Philippidis ce...@codesourcery.com gcc/ * omp-low.c (oacc_get_reduction_array_id): Remove.

Re: [PATCH] config/bfin/bfin.c (hwloop_optimize): Use return false instead of gcc_assert for checking jump_insn.

2015-07-01 Thread Bernd Schmidt
On 07/01/2015 03:04 AM, Chen Gang wrote: For me, the more details are: - The insns have 2 loops which can be lsetup optimized. - After hwloop_optimize finishes 1st lsetup optimization, it generates new lsetup insn which appends to jump insn in the basic block (which causes the

Re: Add .def file for public target instructions

2015-07-01 Thread Trevor Saunders
On Tue, Jun 23, 2015 at 07:41:42PM +0100, Richard Sandiford wrote: [A fair bit later than promised, sorry...] Mikhail posted a patch to make genflags generate the default HAVE_foo and gen_foo definitions that have recently been added to defaults.h:

Re: [02/13] Replace handle_cache_entry with new interface

2015-07-01 Thread Trevor Saunders
On Tue, Jun 16, 2015 at 09:45:56AM +0100, Richard Sandiford wrote: As described in the covering note, this patch replaces handle_cache_entry with a new function keep_cache_entry. It also ensures that elements are deleted using the proper function, so that m_n_deleted is updated. Thanks for

Re: Add .def file for public target instructions

2015-07-01 Thread Richard Biener
On Wed, Jul 1, 2015 at 11:39 AM, Trevor Saunders tbsau...@tbsaunde.org wrote: On Tue, Jun 23, 2015 at 07:41:42PM +0100, Richard Sandiford wrote: [A fair bit later than promised, sorry...] Mikhail posted a patch to make genflags generate the default HAVE_foo and gen_foo definitions that have

Re: Add .def file for public target instructions

2015-07-01 Thread Richard Sandiford
Richard Biener richard.guent...@gmail.com writes: On Wed, Jul 1, 2015 at 11:39 AM, Trevor Saunders tbsau...@tbsaunde.org wrote: On Tue, Jun 23, 2015 at 07:41:42PM +0100, Richard Sandiford wrote: [A fair bit later than promised, sorry...] Mikhail posted a patch to make genflags generate the

[gomp4, committed] Gracefully handle acc_init (nvidia) when not supported

2015-07-01 Thread Tom de Vries
Hi, I noticed on gomp-4_0-branch that lib-62.c was failing with a sigsegv: ... Program received signal SIGSEGV, Segmentation fault. acc_init_1 (d=acc_device_nvidia) at src/libgomp/oacc-init.c:179 179 ndevs = base_dev-get_num_devices_func (); (gdb) p base_dev $1 = (struct gomp_device_descr

Re: [PATCH, PR66432] Handle PARM_DECL in remap_gimple_op_r

2015-07-01 Thread Richard Biener
On Wed, Jul 1, 2015 at 1:43 PM, Tom de Vries tom_devr...@mentor.com wrote: Hi, I. When running test libgomp.c/appendix-a/a.29.1.c with '--target_board unix/-O2/-g', we run into this failure: ... FAIL: libgomp.c/appendix-a/a.29.1.c (test for excess errors) Excess errors:

Re: [ARM] Fix typos in ARMv8 __sync builtins tests.

2015-07-01 Thread Ramana Radhakrishnan
On Wed, Jul 1, 2015 at 12:32 PM, Matthew Wahab matthew.wa...@foss.arm.com wrote: Hello, Tests I recently added for the code generated by the ARM backend for the __sync builtins had - 'do-require-effective-target', instead of the directive 'dg-require-effective-target' and - 'stlex'

[PATCH, committed] jit: document union types

2015-07-01 Thread David Malcolm
Committed to trunk as r225246. gcc/jit/ChangeLog: * docs/topics/types.rst (gcc_jit_context_new_union_type): Add documentation. * docs/_build/texinfo/libgccjit.texi: Regenerate. gcc/testsuite/ChangeLog: * jit.dg/test-accessing-union.c: Add comments for use by

Re: Add .def file for public target instructions

2015-07-01 Thread Richard Sandiford
Richard Sandiford richard.sandif...@arm.com writes: Richard Biener richard.guent...@gmail.com writes: On Wed, Jul 1, 2015 at 11:39 AM, Trevor Saunders tbsau...@tbsaunde.org wrote: On Tue, Jun 23, 2015 at 07:41:42PM +0100, Richard Sandiford wrote: [A fair bit later than promised, sorry...]

Re: [02/13] Replace handle_cache_entry with new interface

2015-07-01 Thread Trevor Saunders
On Wed, Jul 01, 2015 at 10:59:15AM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Tue, Jun 16, 2015 at 09:45:56AM +0100, Richard Sandiford wrote: I couldn't tell whether the unusual name of the function (gt_cleare_cache) is deliberate or not, but I left it

Re: [PATCH, C++-1z] Implement N4197 - Adding u8 character literals

2015-07-01 Thread Richard Biener
On Mon, Jun 29, 2015 at 3:41 PM, Jason Merrill ja...@redhat.com wrote: On 06/27/2015 08:14 PM, Ed Smith-Rowland wrote: In c-ada-specs.c/print_ada_macros() I just write these as a char constant rather than spelling the token. We could do the latter. You'd see the u8 then I think. I couldn't

[gomp4.1] Handle linear clause modifiers in declare simd

2015-07-01 Thread Jakub Jelinek
Hi! I've committed following patch, which per the new ABI additions mangles and handles the various new linear clause modifiers in declare simd functions. The vectorizer side is not done yet, for now we'll never use SIMD_CLONE_ARG_TYPE_LINEAR_*_CONSTANT_STEP in vectorized routines, that waits

[gomp4, PATCH] Fix libgomp.oacc-c-c++-common/lib-3.c

2015-07-01 Thread Tom de Vries
Hi, testcase libgomp.oacc-c-c++-common/lib-3.c is supposed to fail. It fails currently in two ways: - no device found, if there is no nonhost device type supported, so just host and host_nonshm - no device initialized, if there is a nonhost device type supported, f.i. nvidia The reason for

Re: [PATCH, C++-1z] Implement N4197 - Adding u8 character literals

2015-07-01 Thread Richard Biener
On Wed, Jul 1, 2015 at 1:24 PM, Richard Biener richard.guent...@gmail.com wrote: On Mon, Jun 29, 2015 at 3:41 PM, Jason Merrill ja...@redhat.com wrote: On 06/27/2015 08:14 PM, Ed Smith-Rowland wrote: In c-ada-specs.c/print_ada_macros() I just write these as a char constant rather than

[ARM] Fix typos in ARMv8 __sync builtins tests.

2015-07-01 Thread Matthew Wahab
Hello, Tests I recently added for the code generated by the ARM backend for the __sync builtins had - 'do-require-effective-target', instead of the directive 'dg-require-effective-target' and - 'stlex' instead of 'strex' in the expected output for armv8-sync-op-acquire.c This meant the tests

[PATCH, PR66432] Handle PARM_DECL in remap_gimple_op_r

2015-07-01 Thread Tom de Vries
Hi, I. When running test libgomp.c/appendix-a/a.29.1.c with '--target_board unix/-O2/-g', we run into this failure: ... FAIL: libgomp.c/appendix-a/a.29.1.c (test for excess errors) Excess errors: src/libgomp/testsuite/libgomp.c/appendix-a/a.29.1.c:6:1: error: type mismatch between an

Re: [PATCH, libstdc++]: Avoid libsupc++/guard.cc:118:18: warning: variable ‘__p’ set but not used warning

2015-07-01 Thread Jonathan Wakely
On 01/07/15 12:22 +0200, Uros Bizjak wrote: Hello! 2015-07-01 Uros Bizjak ubiz...@gmail.com * libsupc++/guard.cc (__test_and_acquire): Use __p after __atomic_load to avoid unused variable warning. (__set_and_release): Use __p after __atomic_store to avoid unused variable warning.

RE: [PING] [RFC] Sanitize rtx_addr_can_trap_p_1

2015-07-01 Thread Bernd Edlinger
Hi, the bogus offsets-issue is now entered in bugzilla, see PR66614. It is however only a very minor issue, and does probably have no impact on the generated code at all. How should I continue with the rtx_addr_can_trap-patch? Is it OK to commit? Thanks Bernd.

Commit: MSP430: Fix zero-extending a 16-bit value to a 20-bit value, in memory

2015-07-01 Thread Nick Clifton
Hi Guys, I am applying the patch below to fix a problem with the MSP430's zero_extendhipsi2 pattern. When the destination is memory, four bytes must be written, even though only a 20-bit value is being stored, as otherwise the top nibble will be lost. Cheers Nick gcc/ChangeLog

Re: [v3 patch] Implement N4502 Standard Library Support for the C++ Detection Idiom

2015-07-01 Thread Jonathan Wakely
On 01/07/15 13:23 +0100, Jonathan Wakely wrote: This adds the detection idiom API from N4502 http://open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4502.pdf I've added the low-level parts to type_traits, with public aliases in experimental/type_traits, so that I can use them elsewhere in the

Re: [v3 patch] Implement N4502 Standard Library Support for the C++ Detection Idiom

2015-07-01 Thread Jonathan Wakely
On 01/07/15 13:25 +0100, Jonathan Wakely wrote: On 01/07/15 13:23 +0100, Jonathan Wakely wrote: This adds the detection idiom API from N4502 http://open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4502.pdf I've added the low-level parts to type_traits, with public aliases in

[PATCH, lto-plugin]: Avoid warning: implicit declaration of function ‘asprintf’ when building lto-plugin.o

2015-07-01 Thread Uros Bizjak
Hello! We have to enable OS extensions (GNU_SOURCE) to use asprintf. 2015-07-01 Uros Bizjak ubiz...@gmail.com * configure.ac: Add AC_USE_SYSTEM_EXTENSIONS. * configure: Regenerate. * config.h.in: Ditto. Bootstrapped on x86_64-linux-gnu. OK for mainline? Uros. Index:

[PATCH, committed] jit: clarify (lack of) lifetime requirements on input const char *

2015-07-01 Thread David Malcolm
Committed to trunk as r225245. gcc/jit/ChangeLog: * docs/topics/contexts.rst (gcc_jit_context_set_bool_option): Clarify lack of lifetime requirements on (const char *) parameter. * docs/topics/expressions.rst (gcc_jit_context_new_string_literal): Likewise.

Re: [02/13] Replace handle_cache_entry with new interface

2015-07-01 Thread Richard Sandiford
Trevor Saunders tbsau...@tbsaunde.org writes: On Tue, Jun 16, 2015 at 09:45:56AM +0100, Richard Sandiford wrote: I couldn't tell whether the unusual name of the function (gt_cleare_cache) is deliberate or not, but I left it be. I'm not sure what's particularly unusual about it? cleare rather

[PATCH, libiberty]: Avoid enum conversion when passing argument 1 of ‘getrusage’ is invalid in C++ warning

2015-07-01 Thread Uros Bizjak
Hello! This patch avoids enum conversion when passing argument 1 of ‘getrusage’ is invalid in C++ warning when compiling getruntime.c. The condition is copied from sys/resource.h. 2015-07-01 Uros Bizjak ubiz...@gmail.com * getruntime.c (get_run_time) [__USE_GNU]: Use RUSAGE_SELF as

[PATCH, libstdc++]: Avoid libsupc++/guard.cc:118:18: warning: variable ‘__p’ set but not used warning

2015-07-01 Thread Uros Bizjak
Hello! 2015-07-01 Uros Bizjak ubiz...@gmail.com * libsupc++/guard.cc (__test_and_acquire): Use __p after __atomic_load to avoid unused variable warning. (__set_and_release): Use __p after __atomic_store to avoid unused variable warning. Bootstrapped and regression tested on

[PATCH] X86: Optimize access to globals in PIE with copy reloc

2015-07-01 Thread H.J. Lu
Normally, with PIE, GCC accesses globals that are extern to the module using GOT. This is two instructions, one to get the address of the global from GOT and the other to get the value. Examples: --- extern int a_glob; int main () { return a_glob; } --- With PIE, the generated code accesses

[v3 patch] Implement N4502 Standard Library Support for the C++ Detection Idiom

2015-07-01 Thread Jonathan Wakely
This adds the detection idiom API from N4502 http://open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4502.pdf I've added the low-level parts to type_traits, with public aliases in experimental/type_traits, so that I can use them elsewhere in the library. Tested powerpc64le-linux, committed to

Re: [PATCH] config/bfin/bfin.c (hwloop_optimize): Use return false instead of gcc_assert for checking jump_insn.

2015-07-01 Thread Chen Gang
On 7/1/15 21:52, Bernd Schmidt wrote: On 07/01/2015 03:04 AM, Chen Gang wrote: For me, the more details are: - The insns have 2 loops which can be lsetup optimized. - After hwloop_optimize finishes 1st lsetup optimization, it generates new lsetup insn which appends to jump insn in

Eight more jit backports to gcc 5 branch

2015-07-01 Thread David Malcolm
I've gone over the changes to the gcc/jit and gcc/testsuite/jit.dg directories in trunk since the last backports to gcc 5 [1] and backported the following 8 changes from trunk to the gcc-5-branch: gcc-5-branch's r225251: * trunk's r225203 (9bd7a189c372465fb757ecd67336379779f4ea60, jit: fix some

Re: Four jit backports to gcc 5 branch

2015-07-01 Thread David Malcolm
On Tue, 2015-06-30 at 09:01 -0400, David Malcolm wrote: On Tue, 2015-06-30 at 08:43 +0200, Basile Starynkevitch wrote: On Mon, Jun 29, 2015 at 11:44:31AM -0400, David Malcolm wrote: I've gone over the changes to the gcc/jit and gcc/testsuite/jit.dg directories in trunk since gcc 5 and

Re: PR 66685: parallel returns being misclassified

2015-07-01 Thread Richard Sandiford
Richard Biener richard.guent...@gmail.com writes: On Wed, Jul 1, 2015 at 10:27 AM, Richard Sandiford richard.sandif...@arm.com wrote: This patch should restore bootstrap on hppa (and probably other targets besides). The change to use target-insns.def put more stress on the

Re: [s390] Revert TPF C++ library changes

2015-07-01 Thread DJ Delorie
Ah, then approved. Thanks, committed.

[C++/preprocessor Patch] PR c++/53690

2015-07-01 Thread Paolo Carlini
Hi, this issue isn't a regression but it's getting a lot of attention and duplicates. Essentially, due to the lines at the very end of _cpp_valid_ucn: if (result == 0) result = 1; return result; } we can't possibly get the encoding of \U right in C++, that is zero, even

[gomp4] Fix syntax error

2015-07-01 Thread James Norris
Hi, The attached patch fixes a syntax error in one of the OpenACC runtime tests. Committed to gomp-4_0-branch Jim diff --git a/libgomp/testsuite/libgomp.oacc-fortran/if-1.f90 b/libgomp/testsuite/libgomp.oacc-fortran/if-1.f90 index 3ea9681..1729a3b 100644 ---

Regressions with [committed] Use target-insns.def for prologue epilogue insns

2015-07-01 Thread Hans-Peter Nilsson
From: Richard Sandiford richard.sandif...@arm.com Date: Tue, 30 Jun 2015 22:55:24 +0200 Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks, Richard gcc/ * defaults.h (HAVE_epilogue,

Re: update gthr-tpf.h

2015-07-01 Thread DJ Delorie
Ping? https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00149.html This patch updates gthr-tpf.h to the current gthr.h API and TPF API. Ok? * gthr-tpf.h (__GTHREADS_CXX0X): Define. (__gthread_t): Define. (__gthread_cond_t): Define. (__gthread_time_t): Define.

Re: [PATCH] config: fix AM_ICONV for in-tree libiconv

2015-07-01 Thread Yaakov Selkowitz
On Wed, 2015-07-01 at 15:46 -0500, Yaakov Selkowitz wrote: On Fri, 2015-05-29 at 16:25 -0500, Yaakov Selkowitz wrote: This is the second in a series of patches to make a build with an in-tree GNU libiconv work as designed. Currently GDB is the only toolchain component which actually uses

[PATCH] PR fortran/66545 -- Fix NULL pointer dereference

2015-07-01 Thread Steve Kargl
The attached patch avoid the dereference of a NULL pointer, and thereby avoids an ICE. Regression tested on trunk. OK to commit? 2015-07-01 Steven g. Kargl ka...@gcc.gnu.org PR fortran/66545 * primary.c (match_sym_complex_part): Do not dereference NULL pointer. 2015-07-01

Re: [PATCH] config: fix AM_ICONV for in-tree libiconv

2015-07-01 Thread Yaakov Selkowitz
On Wed, 2015-07-01 at 15:46 -0500, Yaakov Selkowitz wrote: On Fri, 2015-05-29 at 16:25 -0500, Yaakov Selkowitz wrote: This is the second in a series of patches to make a build with an in-tree GNU libiconv work as designed. Currently GDB is the only toolchain component which actually uses

Re: [C++/preprocessor Patch] PR c++/53690

2015-07-01 Thread Paolo Carlini
Hi again, On 07/01/2015 10:54 PM, Paolo Carlini wrote: Hi, On 07/01/2015 10:45 PM, Andreas Schwab wrote: Paolo Carlini paolo.carl...@oracle.com writes: @@ -972,21 +972,20 @@ ucn_valid_in_identifier (cpp_reader *pfile, cppcha or 0060 (`), nor one in the range D800 through DFFF

[gomp4] Allow parameter declarations with deviceptr

2015-07-01 Thread James Norris
Hi, This patch allows parameter declarations to be used as arguments to deviceptr for C and C++. Committed to gomp-4_0-branch. Jim diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c index 88e68ae..dc244ce 100644 --- a/gcc/c/c-parser.c +++ b/gcc/c/c-parser.c @@ -10749,7 +10749,7 @@

Re: [patch, wwwdocs, committed] Document -Winteger-division for Fortran

2015-07-01 Thread Gerald Pfeifer
On Wed, 1 Jul 2015, Thomas Koenig wrote: I just committed the following patch to the WWW docs. Thanks. In the description, are the integers constants truncated or (the result of) the division? An error was reported by Gerald's script, but that seems to have been spurious. At least, there

Re: [patch, wwwdocs, committed] Document -Winteger-division for Fortran

2015-07-01 Thread Steve Kargl
On Wed, Jul 01, 2015 at 11:39:09PM +0200, Gerald Pfeifer wrote: On Wed, 1 Jul 2015, Thomas Koenig wrote: I just committed the following patch to the WWW docs. Thanks. In the description, are the integers constants truncated or (the result of) the division? The result is truncated. real

Re: [PATCH] toplevel: fixes for in-tree libiconv

2015-07-01 Thread Yaakov Selkowitz
On Fri, 2015-05-29 at 16:14 -0500, Yaakov Selkowitz wrote: This is the first in a series of patches to make a build with an in-tree GNU libiconv work as designed. This patch fixes dependencies for parallel make, and avoids failures with make targets not supported by GNU libiconv. Thanks

Re: [PATCH] toplevel: fixes for in-tree libiconv

2015-07-01 Thread Yaakov Selkowitz
On Fri, 2015-05-29 at 16:14 -0500, Yaakov Selkowitz wrote: This is the first in a series of patches to make a build with an in-tree GNU libiconv work as designed. This patch fixes dependencies for parallel make, and avoids failures with make targets not supported by GNU libiconv. Thanks

Re: [gomp4.1] Support #pragma omp target {enter,exit} data

2015-07-01 Thread Ilya Verbin
On Tue, Jun 30, 2015 at 18:10:44 +0200, Jakub Jelinek wrote: The thing is whether it is actually a good idea to allocate the enter data allocated objects together. In OpenMP 4.0, generally objects would be allocated and deallocated at the same times, except for multiple host threads trying to

Re: Regressions with [committed] Use target-insns.def for prologue epilogue insns

2015-07-01 Thread Richard Sandiford
Hans-Peter Nilsson hans-peter.nils...@axis.com writes: From: Richard Sandiford richard.sandif...@arm.com Date: Tue, 30 Jun 2015 22:55:24 +0200 Bootstrapped regression-tested on x86_64-linux-gnu and aarch64-linux-gnu. Also tested via config-list.mk. Committed as preapproved. Thanks,

Re: [PATCH] config: fix AM_ICONV for in-tree libiconv

2015-07-01 Thread Yaakov Selkowitz
On Wed, 2015-07-01 at 15:52 -0500, Yaakov Selkowitz wrote: On Wed, 2015-07-01 at 15:46 -0500, Yaakov Selkowitz wrote: On Fri, 2015-05-29 at 16:25 -0500, Yaakov Selkowitz wrote: This is the second in a series of patches to make a build with an in-tree GNU libiconv work as designed.

Re: [PATCH] config: fix AM_ICONV for in-tree libiconv

2015-07-01 Thread Yaakov Selkowitz
On Fri, 2015-05-29 at 16:25 -0500, Yaakov Selkowitz wrote: This is the second in a series of patches to make a build with an in-tree GNU libiconv work as designed. Currently GDB is the only toolchain component which actually uses an in-tree libiconv. This patch modifies the common AM_ICONV

Re: [C++/preprocessor Patch] PR c++/53690

2015-07-01 Thread Andreas Schwab
Paolo Carlini paolo.carl...@oracle.com writes: @@ -972,21 +972,20 @@ ucn_valid_in_identifier (cpp_reader *pfile, cppcha or 0060 (`), nor one in the range D800 through DFFF inclusive. *PSTR must be preceded by \u or \U; it is assumed that the - buffer end is delimited by a non-hex

Re: [C++/preprocessor Patch] PR c++/53690

2015-07-01 Thread Paolo Carlini
Hi, On 07/01/2015 10:45 PM, Andreas Schwab wrote: Paolo Carlini paolo.carl...@oracle.com writes: @@ -972,21 +972,20 @@ ucn_valid_in_identifier (cpp_reader *pfile, cppcha or 0060 (`), nor one in the range D800 through DFFF inclusive. *PSTR must be preceded by \u or \U; it is

Re: [PATCH] PR fortran/66545 -- Fix NULL pointer dereference

2015-07-01 Thread Thomas Koenig
Hi Steve, The attached patch avoid the dereference of a NULL pointer, and thereby avoids an ICE. Regression tested on trunk. OK to commit? OK. Thanks for the patch! Thomas

Re: PR 66685: parallel returns being misclassified

2015-07-01 Thread Jeff Law
On 07/01/2015 09:25 AM, Richard Sandiford wrote: Richard Biener richard.guent...@gmail.com writes: On Wed, Jul 1, 2015 at 10:27 AM, Richard Sandiford richard.sandif...@arm.com wrote: This patch should restore bootstrap on hppa (and probably other targets besides). The change to use

Re: [patch] fix regrename pass to ensure renamings produce valid insns

2015-07-01 Thread Jeff Law
On 06/28/2015 03:08 PM, Sandra Loosemore wrote: Re the testcase, this fixed 16 FAILs on existing tests in the gcc testsuite with the forthcoming nios2 load/store multiple instruction support, all assembler errors due to the bad instructions being generated. There's nothing I can do on nios2

Re: [RFA] Factor conversion out of COND_EXPR using match.pd pattern

2015-07-01 Thread Jeff Law
On 06/30/2015 01:45 AM, Richard Biener wrote: On Mon, Jun 29, 2015 at 7:51 PM, Jeff Law l...@redhat.com wrote: So do we want to restrict the new pattern on GENERIC, then rely on phiopt to get smarter and catch this stuff for GIMPLE? Or drop the pattern totally and do everything in phiopt on

Re: [PATCH 1/2] Add gcc/typed-splay-tree.h

2015-07-01 Thread Jeff Law
On 06/29/2015 11:36 AM, David Malcolm wrote: On Thu, 2015-06-25 at 13:18 -0600, Jeff Law wrote: On 06/25/2015 01:13 PM, David Malcolm wrote: I found when implementing switch statements for the jit that it was much easier to work with libiberty's splay-tree.h by first wrapping it in a C++

[PATCH] PR fortran/56520 -- Special case unary minus/plus

2015-07-01 Thread Steve Kargl
The attached patch special cases unary minus/plus when gfotran tries to match a function keyword in a mangled statement. The testcase contains examples, but for context consider 'c = exp(-a) )' where the final ')' is erronous. gfortran is trying to match a keyword such as 'c = exp(x=-a)' so it

Re: [PATCH, PR66432] Handle PARM_DECL in remap_gimple_op_r

2015-07-01 Thread Tom de Vries
On 01/07/15 13:58, Richard Biener wrote: On Wed, Jul 1, 2015 at 1:43 PM, Tom de Vries tom_devr...@mentor.com wrote: Hi, I. When running test libgomp.c/appendix-a/a.29.1.c with '--target_board unix/-O2/-g', we run into this failure: ... FAIL: libgomp.c/appendix-a/a.29.1.c (test for excess

[testsuite, committed, PR66723] Use scan-rtl-dump in i386/vararg-loc.c

2015-07-01 Thread Tom de Vries
Hi, I've committed this patch, which rewrites the scan lines in test-case i386/vararg-loc.c from scan-assembler to scan-rtl-dump, to fix PR66723. Thanks, - Tom Use scan-rtl-dump in i386/vararg-loc.c 2015-07-01 Tom de Vries t...@codesourcery.com PR testsuite/66723 *

[gomp4, committed, PR66716] Fix compilation libgomp.oacc-c-c++-common/kernels-loop.c -g

2015-07-01 Thread Tom de Vries
Hi, when compiling libgomp.oacc-c-c++-common/kernels-loop at -O2 -g, we run into: ... FAIL: libgomp.oacc-c/../libgomp.oacc-c-c++-common/kernels-loop.c -DACC_DEVICE_TYPE_host=1 -DACC_MEM_SHARED=1 (internal compiler error) ... The ICE in more detail: ...

[patch, wwwdocs, committed] Document -Winteger-division for Fortran

2015-07-01 Thread Thomas Koenig
Hello world, I just committed the following patch to the WWW docs. An error was reported by Gerald's script, but that seems to have been spurious. At least, there was no error on revalidation. Regards Thomas 2015-07-01 Thomas Koenig tkoe...@netcologne.de *

C++ PATCH for c++/65945 (nullptr_t alignment)

2015-07-01 Thread Jason Merrill
It's trivial to fix the alignment of nullptr_t, but I was concerned about ABI impact. On further research it seems that it won't cause any trouble with argument alignment, since that doesn't seem to rely on TYPE_ALIGN at all; I think the only ABI breakage would come from unaligned nullptr_t

Re: [PATCH GCC][refacor]Manage allocation of struct iv in obstack.

2015-07-01 Thread Richard Biener
On Tue, Jun 30, 2015 at 9:49 AM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Jun 30, 2015 at 4:31 AM, Bin.Cheng amker.ch...@gmail.com wrote: On Sat, Jun 27, 2015 at 5:13 AM, Jeff Law l...@redhat.com wrote: On 06/26/2015 03:02 AM, Bin Cheng wrote: Hi, GCC avoids

Re: [patch] Fix PR middle-end/66633

2015-07-01 Thread Jakub Jelinek
On Wed, Jul 01, 2015 at 09:52:04AM +0200, Eric Botcazou wrote: Jakub, 2015-06-29 Eric Botcazou ebotca...@adacore.com PR middle-end/66633 * tree-nested.c (convert_nonlocal_omp_clauses): Initialize need_chain to true if the function is nested and if not optimizing.

Re: [patch] PR debug/66653: avoid late_global_decl on decl_type_context()s

2015-07-01 Thread Richard Biener
On Tue, Jun 30, 2015 at 4:30 PM, Jason Merrill ja...@redhat.com wrote: On 06/29/2015 06:32 PM, Jason Merrill wrote: On 06/29/2015 05:07 AM, Richard Biener wrote: On Fri, Jun 26, 2015 at 11:59 PM, Jason Merrill ja...@redhat.com wrote: On 06/26/2015 05:37 AM, Richard Biener wrote: Can we

Re: [patch] Fix ICE during SCC hashing in LTO mode

2015-07-01 Thread Richard Biener
On Tue, Jun 30, 2015 at 11:03 PM, Eric Botcazou ebotca...@adacore.com wrote: Ok (I suppose this also affects the GCC 5 branch?) Thanks. Yes, it does, but I'm not sure if we need to backport it immediately. Given there is no PR to track this I'll count on you remembering to backport it then ;)

Re: [patch] Fix ICE during SCC hashing in LTO mode

2015-07-01 Thread Eric Botcazou
Given there is no PR to track this I'll count on you remembering to backport it then ;) OK. If you'd want it in 5.2 then you'd need to backport it by this Friday morning. No thanks. :-) -- Eric Botcazou

Re: [patch] Fix PR middle-end/66633

2015-07-01 Thread Eric Botcazou
Jakub, 2015-06-29 Eric Botcazou ebotca...@adacore.com PR middle-end/66633 * tree-nested.c (convert_nonlocal_omp_clauses): Initialize need_chain to true if the function is nested and if not optimizing. (convert_local_omp_clauses): Initialize need_frame to true if

Re: [PATCH, CHKP, PR middle-end/66568] Fix internal compiler error: in expand_expr_addr_expr_1

2015-07-01 Thread H.J. Lu
On Thu, Jun 18, 2015 at 3:16 AM, Ilya Enkovich enkovich@gmail.com wrote: Hi, Currently return statement with no bounds in instrumented function causes ICE. This patch uses zero bounds in such cases instead. Bootstrapped and regtested for x86_64-unknown-linux-gnu. Applied to trunk.

Re: C++ PATCH to change default dialect to C++14

2015-07-01 Thread Jim Wilson
On 07/01/2015 11:26 AM, Jason Merrill wrote: I've been threatening to do this for a couple of months, and now that the regressions are under control I think it's time. This patch changes the default C++ dialect to C++14. Tested x86_64-pc-linux-gnu, applying to trunk. This causes a build

RE: [Patch,tree-optimization]: Add new path Splitting pass on tree ssa representation

2015-07-01 Thread Ajit Kumar Agarwal
-Original Message- From: Joseph Myers [mailto:jos...@codesourcery.com] Sent: Wednesday, July 01, 2015 3:48 AM To: Ajit Kumar Agarwal Cc: l...@redhat.com; GCC Patches; Vinod Kathail; Shail Aditya Gupta; Vidhumouli Hunsigida; Nagaraju Mekala Subject: Re: [Patch,tree-optimization]: Add

Re: C++ PATCH to change default dialect to C++14

2015-07-01 Thread Jason Merrill
On 07/02/2015 12:10 AM, Jim Wilson wrote: This is a known gmp problem, documented in PR56019 and in https://gcc.gnu.org/gcc-4.9/porting_to.html near the bottom where it discusses cstddef.h changes. This document also says that A workaround until libraries get updated is to include

Re: [PATCH] PR fortran/56520 -- Special case unary minus/plus

2015-07-01 Thread Jerry DeLisle
On 07/01/2015 10:53 AM, Steve Kargl wrote: The attached patch special cases unary minus/plus when gfotran tries to match a function keyword in a mangled statement. The testcase contains examples, but for context consider 'c = exp(-a) )' where the final ')' is erronous. gfortran is trying to

Re: [gomp4] Allow parameter declarations with deviceptr

2015-07-01 Thread Cesar Philippidis
On 07/01/2015 02:25 PM, James Norris wrote: This patch allows parameter declarations to be used as arguments to deviceptr for C and C++. Does this fix an existing failure? If not, can you please add a new test case? Thanks, Cesar

  1   2   >