Re: [PATCH][RX] v2 instructions support.

2015-12-23 Thread Yoshinori Sato
Ping. On Thu, 10 Dec 2015 23:11:33 +0900, Yoshinori Sato wrote: > > Add RX v2 enhancement instructions support. > > gcc/ChangeLog > 2015/12/10 Yoshinori Sato > > * config/rx/constraints.md: Add constraint "q". > * doc/md.texi: Likewise. > * config/rx/rx-opts.h(rx_cpu_types)

PR ipa/67811, part 2

2015-12-23 Thread Richard Henderson
Mea culpa. Somehow I diffed the wrong trees and missed a regression in libitm.c/cancel.c. Re-tested and committed. r~ * tree-cfg.c (make_edges_bb): Add abort edge for outer transactions. diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c index 436ea14..3b2fab5 100644 --- a/gcc/tree-cfg.c +

Re: [PATCH, RFC, gfortran] Multi-threaded random number generator

2015-12-23 Thread Steve Kargl
On Thu, Dec 24, 2015 at 12:29:02AM +0200, Janne Blomqvist wrote: > > the GFortran random number generator (RANDOM_NUMBER and RANDOM_SEED > intrinsics) has a number of issues that the attached preliminary patch > tries to address. > First, I think your effort to retool random numbers is a nice en

[PATCH, RFC, gfortran] Multi-threaded random number generator

2015-12-23 Thread Janne Blomqvist
Hi, the GFortran random number generator (RANDOM_NUMBER and RANDOM_SEED intrinsics) has a number of issues that the attached preliminary patch tries to address. - 64-bit integers are available on all targets GFortran supports, and the vast majority of users nowadays use targets with native 64-b

Re: [Patch, libstdc++/68877] Reimplement __is_[nothrow_]swappable

2015-12-23 Thread Daniel Krügler
This is a second try for a patch for libstdc++ bug 68877. See below for responses. 2015-12-22 22:42 GMT+01:00 Jonathan Wakely : > On 21/12/15 12:45 +0100, Daniel Krügler wrote: >> >> 2015-12-14 21:48 GMT+01:00 Daniel Krügler : >>> >>> This is a reimplementation of __is_swappable and >>> __is_nothr

Re: [PATCH] add test for c++/69023 - bitset whose name is used in constant-expression rejected

2015-12-23 Thread Jason Merrill
OK, thanks. Jason

Re: [PATCH][WIP] libstdc++: Make certain exceptions transaction_safe.

2015-12-23 Thread Jason Merrill
On 12/23/2015 12:35 PM, Torvald Riegel wrote: +//#if !defined (HAVE_ELF_STYLE_WEAKREF) Can I assume weak refs to be supported, or how do I check for whether they are? What's your preference? G++ does support targets without weak symbols, but I imagine we can decide not to support libitm on su

Re: [wwwdocs] Document how to add a new SSH key

2015-12-23 Thread Toon Moene
On 12/23/2015 08:57 PM, Marc Glisse wrote: On Wed, 23 Dec 2015, Toon Moene wrote: On 12/22/2015 02:59 PM, Gerald Pfeifer wrote: +ssh username@gcc.gnu.org append-key < KEYFILE Hmm, I get: toon@moene:~$ ssh t...@gcc.gnu.org append-key < .ssh/id_dsa.pub /home/toon/.ssh/config line 3: Bad pr

Re: [wwwdocs] Document how to add a new SSH key

2015-12-23 Thread Marc Glisse
On Wed, 23 Dec 2015, Toon Moene wrote: On 12/22/2015 02:59 PM, Gerald Pfeifer wrote: Jan (Beulich) ran into this, and indeed I could not find it documented. So I added it. ;-) +ssh username@gcc.gnu.org append-key < KEYFILE Hmm, I get: toon@moene:~$ ssh t...@gcc.gnu.org append-key < .ssh

Re: [wwwdocs] Document how to add a new SSH key

2015-12-23 Thread Toon Moene
On 12/22/2015 02:59 PM, Gerald Pfeifer wrote: Jan (Beulich) ran into this, and indeed I could not find it documented. So I added it. ;-) +ssh username@gcc.gnu.org append-key < KEYFILE Hmm, I get: toon@moene:~$ ssh t...@gcc.gnu.org append-key < .ssh/id_dsa.pub /home/toon/.ssh/config line 3

Re: [patch] ARM FreeBSD fix bootstrap

2015-12-23 Thread Andreas Tobler
On 23.12.15 11:22, Richard Earnshaw (lists) wrote: On 22/12/15 19:53, Andreas Tobler wrote: Hi all, the commit for PR68617 broke boostrap on armv6*-*-freebsd*. We still have unaligned_access = 0 on armv6 here on FreeBSD. The commit from the above PR overrides my SUBTARGET_OVERRIDE_OPTIONS I c

Re: [patch] ARM FreeBSD fix bootstrap

2015-12-23 Thread Andreas Tobler
Hi Kyrill, On 23.12.15 10:40, Kyrill Tkachov wrote: On 22/12/15 19:53, Andreas Tobler wrote: Hi all, the commit for PR68617 broke boostrap on armv6*-*-freebsd*. We still have unaligned_access = 0 on armv6 here on FreeBSD. The commit from the above PR overrides my SUBTARGET_OVERRIDE_OPTIONS

Re: [PATCH, middle-end]: Fix PR68999, gfortran.fortran-torture/execute/save_1.f90 execution failure on alpha

2015-12-23 Thread Richard Biener
On December 23, 2015 5:58:07 PM GMT+01:00, Uros Bizjak wrote: >On Wed, Dec 23, 2015 at 2:39 PM, Richard Biener > wrote: >> On December 23, 2015 10:39:17 AM GMT+01:00, Uros Bizjak > wrote: >>>Hello! >>> >>>There is a logic error in Honza's patch "Transparent alias suport >part >>>10" [1]. The part

Re: varpool/constpool bug

2015-12-23 Thread Nathan Sidwell
On 12/23/15 09:48, Nathan Sidwell wrote: On 12/21/15 14:45, Jeff Law wrote: With some kind of comment in decl_in_symtab_p indicating why we need to check and filter on !DECL_IN_CONSTANT_POOL this is OK. Done, thanks. (I half guessed HPPA might be such a port :) sigh, there are other route

[PATCH] add test for c++/69023 - bitset whose name is used in constant-expression rejected

2015-12-23 Thread Martin Sebor
Attached is a testsuite-only patch for the correct handling of the (now resolved as invalid) c++/69023 - bitset whose name is used in constant expression rejected. GCC does the right thing but I didn't see the case in the PR exercised by the test suite, so while experimenting to make sure I under

Re: [PATCH][WIP] libstdc++: Make certain exceptions transaction_safe.

2015-12-23 Thread Torvald Riegel
On Wed, 2015-12-16 at 21:05 +, Jonathan Wakely wrote: > Sorry for the delay finishing this review, some of the code kept > melting my brain ;-) I know what you mean :) Thanks for the review! > On 14/11/15 20:45 +0100, Torvald Riegel wrote: > >diff --git a/libstdc++-v3/config/abi/pre/gnu.ver

Re: [PATCH, middle-end]: Fix PR68999, gfortran.fortran-torture/execute/save_1.f90 execution failure on alpha

2015-12-23 Thread Uros Bizjak
On Wed, Dec 23, 2015 at 2:39 PM, Richard Biener wrote: > On December 23, 2015 10:39:17 AM GMT+01:00, Uros Bizjak > wrote: >>Hello! >> >>There is a logic error in Honza's patch "Transparent alias suport part >>10" [1]. The part in memrefs_conflict_p should be changed to: >> >>- /* If decls a

[PTX] remove some predicates

2015-12-23 Thread Nathan Sidwell
This patch removes some now unused predicates, removes the inspection of a mov insn's source operand along with explict address conversion instruction. the noting of use of a function address is done at emission time. nathan 2015-12-23 Nathan Sidwell * config/nvptx/nvptx-protos.h (nvptx_

Re: varpool/constpool bug

2015-12-23 Thread Nathan Sidwell
On 12/21/15 14:45, Jeff Law wrote: With some kind of comment in decl_in_symtab_p indicating why we need to check and filter on !DECL_IN_CONSTANT_POOL this is OK. Done, thanks. (I half guessed HPPA might be such a port :) nathan 2015-12-23 Nathan Sidwell gcc/ * alias.c (compare_base_de

Re: [patch] ARM FreeBSD fix bootstrap

2015-12-23 Thread Nathan Sidwell
On 12/23/15 05:22, Richard Earnshaw (lists) wrote: 2015-12-22 Andreas Tobler * config/arm/freebsd.h (SUBTARGET_OVERRIDE_OPTIONS): Adjust to check unaligned_access on the gcc_options set. * config/arm/arm.c (arm_option_override): Move SUBTARGET_OVERRIDE_OPTIONS from here t

Re: [PATCH, middle-end]: Fix PR68999, gfortran.fortran-torture/execute/save_1.f90 execution failure on alpha

2015-12-23 Thread Uros Bizjak
On Wed, Dec 23, 2015 at 2:39 PM, Richard Biener wrote: > On December 23, 2015 10:39:17 AM GMT+01:00, Uros Bizjak > wrote: >>Hello! >> >>There is a logic error in Honza's patch "Transparent alias suport part >>10" [1]. The part in memrefs_conflict_p should be changed to: >> >>- /* If decls a

Re: [PATCH, middle-end]: Fix PR68999, gfortran.fortran-torture/execute/save_1.f90 execution failure on alpha

2015-12-23 Thread Richard Biener
On December 23, 2015 10:39:17 AM GMT+01:00, Uros Bizjak wrote: >Hello! > >There is a logic error in Honza's patch "Transparent alias suport part >10" [1]. The part in memrefs_conflict_p should be changed to: > >- /* If decls are different or we know by offsets that there is no >overlap, >- w

Re: [PATCH, i386] Introduce support for PKU instructions.

2015-12-23 Thread Uros Bizjak
On Tue, Dec 22, 2015 at 4:43 PM, Kirill Yukhin wrote: > Hello Uroš, > I (hopefully fixed all of inputs, thanks! > > Updated patch for i386.md in the bottom, > rest patch is the same. > > Bootstrap in progress. New tests pass. > > Is it ok for trunk if bootstrap will pass? > > On 20 Dec 11:56, Uros

Re: [ARM] Fix ICE on Ada code with -mbig-endian -mhard-float

2015-12-23 Thread Richard Earnshaw (lists)
On 21/12/15 16:24, Eric Botcazou wrote: > Hi, > > the attached Ada testcase triggers an ICE with -mbig-endian -mhard-float: > > eric@polaris:~/build/gcc/arm-linux-gnueabi> gcc/xgcc -Bgcc -S p.adb -I > gcc/ada/rts -mbig-endian -mhard-float > +===GNAT BUG DETECTED==

Re: [RFC][PATCH] Fix broken handling of LABEL_REF in genrecog + genpreds.

2015-12-23 Thread Andreas Krebbel
On 12/22/2015 12:12 PM, Dominik Vogt wrote: > All right, in the mean time the patch has passed the test suite on > x86_64, s390x and s390. See attached version 2 of the patch > (added just an additional comment in genpreds.c). Applied. Thanks! -Andreas-

Re: gomp_target_fini

2015-12-23 Thread Thomas Schwinge
Hi! Ping. On Wed, 16 Dec 2015 13:30:21 +0100, I wrote: > On Mon, 14 Dec 2015 19:47:36 +0300, Ilya Verbin wrote: > > On Fri, Dec 11, 2015 at 18:27:13 +0100, Jakub Jelinek wrote: > > > On Tue, Dec 08, 2015 at 05:45:59PM +0300, Ilya Verbin wrote: > > > > +/* This function finalizes all initialized

Re: [PATCH] Fix up reduction-1{1,2} testcases (PR middle-end/68221)

2015-12-23 Thread Thomas Schwinge
Hi! Ping. On Thu, 26 Nov 2015 14:31:56 +0100, I wrote: > On Mon, 23 Nov 2015 12:13:07 +0100 (CET), Richard Biener > wrote: > > On Fri, 20 Nov 2015, Jakub Jelinek wrote: > > > If C/C++ array section reductions have non-zero (positive) bias, it is > > > implemented by declaring a smaller private

Re: [Bulk] [OpenACC 0/7] host_data construct

2015-12-23 Thread Thomas Schwinge
Hi! On Mon, 26 Oct 2015 19:34:22 +0100, Jakub Jelinek wrote: > Your use_device sounds very similar to use_device_ptr clause in OpenMP, > which is allowed on #pragma omp target data construct and is implemented > quite a bit differently from this; it is unclear if the OpenACC standard > requires t

Don't run OpenACC C++ test twice (was: [gomp4] OpenACC / C++)

2015-12-23 Thread Thomas Schwinge
Hi! On Mon, 20 Oct 2014 12:17:41 +0200, I wrote: > I have just applied in r216456: »Enable compiler testing > for OpenACC/C++«. > * g++.dg/goacc-gomp/goacc-gomp.exp: New file. > * g++.dg/goacc/goacc.exp: Likewise. I failed to realize that g++.dg/dg.exp has a "catch all" rule, and to

Re: [patch] ARM FreeBSD fix bootstrap

2015-12-23 Thread Richard Earnshaw (lists)
On 22/12/15 19:53, Andreas Tobler wrote: > Hi all, > > the commit for PR68617 broke boostrap on armv6*-*-freebsd*. > > We still have unaligned_access = 0 on armv6 here on FreeBSD. > > The commit from the above PR overrides my SUBTARGET_OVERRIDE_OPTIONS I > called in arm_option_override. And it s

Re: [patch] ARM FreeBSD fix bootstrap

2015-12-23 Thread Kyrill Tkachov
Hi Andreas, On 22/12/15 19:53, Andreas Tobler wrote: Hi all, the commit for PR68617 broke boostrap on armv6*-*-freebsd*. We still have unaligned_access = 0 on armv6 here on FreeBSD. The commit from the above PR overrides my SUBTARGET_OVERRIDE_OPTIONS I called in arm_option_override. And it s

[PATCH, middle-end]: Fix PR68999, gfortran.fortran-torture/execute/save_1.f90 execution failure on alpha

2015-12-23 Thread Uros Bizjak
Hello! There is a logic error in Honza's patch "Transparent alias suport part 10" [1]. The part in memrefs_conflict_p should be changed to: - /* If decls are different or we know by offsets that there is no overlap, - we win. */ - if (!cmp || !offset_overlap_p (c, xsize, ysize)) +

Re: [PING][Patch 3/3][Arm] Add support for IEEE-conformant versions of scalar fmin* and fmax*

2015-12-23 Thread Kyrill Tkachov
Hi David, On 22/12/15 17:07, David Sherwood wrote: Hi Kyrill, Thanks for the reply, I think this latest patch addresses your comments. I have added a test for the scalar forms of fmax/fmin. Is this ok now? Tested: arm-none-eabi: no regressions I've also bootstrapped this patch on arm-none-

[PATCH, PR target/69010] Fix expand for constant boolean vectors with a scalar mode

2015-12-23 Thread Ilya Enkovich
Hi, Currently boolean vector constant is expanded incorrectly when it has a scalar mode. Generated constant is incorrect and also its mode (which is VOIDmode) is used for roce_reg which causes ICE. This patch fixes both problems. Bootstrapped and tested on x86_64-pc-linux-gnu. OK for trunk? T

[PATCH] Avoid ifcvt ICE on conditional return followed by trap (PR target/69015)

2015-12-23 Thread Jakub Jelinek
Hi! The find_cond_trap code isn't really prepared to turn conditional return into unconditional when converting non-conditional trap into conditional, without RTL checking it generates (label_ref (return)), the edges updates are also wrong. This patch just disables that transformation, IMHO tradin