Re: [x86, 2/n] Replace builtins with vector extensions

2014-11-05 Thread Uros Bizjak
On Tue, Nov 4, 2014 at 9:31 PM, Marc Glisse wrote: > Ping? Uh, yes, LGTM. (I was under impression that I already OK'd this relatively non-controversial patch. The effect of having too much open tasks in parallel, I guess.) Thanks, Uros. > https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01808.htm

RE: [PATCH][6/n] Merge from match-and-simplify, make forwprop fold all stmts

2014-11-05 Thread Zhenqiang Chen
The patch leads to big regression for float operators on target without hard fpu support due to register shuffle. Please refer https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63743 for more detail. Thanks! -Zhenqiang > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-pat

Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2).

2014-11-05 Thread Tejas Belagod
On 03/11/14 16:49, Marcus Shawcroft wrote: On 1 October 2014 09:26, Tejas Belagod wrote: Hi, Returning to this old thread, https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02285.html here is a patch after a few rounds of review comments, specifically: https://gcc.gnu.org/ml/gcc-patches/2014-06

Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2).

2014-11-05 Thread Tejas Belagod
On 05/11/14 08:28, Tejas Belagod wrote: On 03/11/14 16:49, Marcus Shawcroft wrote: On 1 October 2014 09:26, Tejas Belagod wrote: Hi, Returning to this old thread, https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02285.html here is a patch after a few rounds of review comments, specifically: h

RE: [Ping] [PATCH, 9/10] aarch64: generate conditional compare instructions

2014-11-05 Thread Zhenqiang Chen
I had retested all the ccmp patches. Bootstrap and no make check regression on X86-64. Bootstrap and no make check regression on AARCH64 qemu. OK for trunk? Thanks! -Zhenqiang > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/03/2014 05:27 PM, Marek Polacek wrote: Another shot at optimizing redundant UBSAN_NULL statements. This time we walk the dominator tree - that should result in more effective optimization - and keep a list of UBSAN_NULL statements that dominate the current block, see the comment before san

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass

2014-11-05 Thread Bin.Cheng
On Sat, Nov 1, 2014 at 4:29 AM, Jeff Law wrote: > On 09/30/14 03:22, Bin Cheng wrote: > >> >> 2014-09-30 Bin Cheng >> Mike Stump >> >> * timevar.def (TV_SCHED_FUSION): New time var. >> * passes.def (pass_sched_fusion): New pass. >> * config/arm/arm.c (TARGET_SC

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 12:19:22PM +0300, Yury Gribov wrote: > On 11/03/2014 05:27 PM, Marek Polacek wrote: > >Another shot at optimizing redundant UBSAN_NULL statements. > > > >This time we walk the dominator tree - that should result in > >more effective optimization - and keep a list of UBSAN_NU

Re: [PATCH, i386] Add prefixes avoidance tuning for silvermont target

2014-11-05 Thread Ilya Enkovich
Hi, Having stage1 close to end, may we make some decision regarding this patch? Having a couple of working variants, may we choose and use one of them? Thanks, Ilya 2014-07-15 17:38 GMT+04:00 Uros Bizjak : > On Tue, Jul 15, 2014 at 1:01 PM, Ilya Enkovich wrote: >> On 15 Jul 10:42, Uros Bizjak w

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 12:33 PM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 12:19:22PM +0300, Yury Gribov wrote: On 11/03/2014 05:27 PM, Marek Polacek wrote: Another shot at optimizing redundant UBSAN_NULL statements. This time we walk the dominator tree - that should result in more effective optimiz

Re: [PATCH, i386] Add prefixes avoidance tuning for silvermont target

2014-11-05 Thread Uros Bizjak
On Wed, Nov 5, 2014 at 10:35 AM, Ilya Enkovich wrote: > Hi, > > Having stage1 close to end, may we make some decision regarding this > patch? Having a couple of working variants, may we choose and use one > of them? I propose to wait for Vlad for an update about his plans on register preference a

Re: [gofrontend-dev] [PATCH 4/4] Gccgo port to s390[x] -- part II

2014-11-05 Thread Dominik Vogt
On Tue, Nov 04, 2014 at 08:16:51PM -0800, Ian Taylor wrote: > I committed the change to go-test.exp. Thanks. > > The other changes are not OK. As described in > gcc/testsuite/go.test/test/README.gcc, the files in > gcc/testsuite/go.test/test are an exact copy of the master Go > testsuite. Any c

Re: Small multiplier support in Cortex-M0/1/+

2014-11-05 Thread Ramana Radhakrishnan
On Tue, Oct 21, 2014 at 11:01 AM, Hale Wang wrote: > Hi, > > Some configurations of the Cortex-M0 and Cortex-M1 come with a high latency > multiplier. This patch adds support for such configurations. > > Small multiplier means using add/sub/shift instructions to replace the mul > instruction for t

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Marek Polacek
On Wed, Nov 05, 2014 at 12:54:37PM +0300, Yury Gribov wrote: > Are you going to work on ASan soon? I could rebase my patches on top of > Marek's infrastructure. I'm not going to work on ASan today or tomorrow, but it'd be nice to get this ASan opt in in this stage1. So if you can rebase your pat

Re: [gofrontend-dev] [PATCH 1/4] Gccgo port to s390[x] -- part II

2014-11-05 Thread Dominik Vogt
On Tue, Nov 04, 2014 at 02:39:34PM -0800, Ian Taylor wrote: > Note that libgo/runtime/runtime.c now refers to S390_HAVE_STCKF. It's > not obvious to me that that is defined anywhere. Perhaps it is in a > later patch in this series--I haven't looked. This chunk is broken but harmless (because S39

RE: [PATCH][MIPS] Fix P5600 memory cost

2014-11-05 Thread Matthew Fortune
> The patch below fixes the memory cost for P5600. > > ChangeLog: > 2014-11-05 Prachi Godbole > > * config/mips/mips.c (mips_rtx_cost_data): Fix memory_letency cost for > p5600. Please follow these instructions to add yourself to MAINTAINERS in the write-after-approval section now that you have

[PATCH, libfortran] PR 47007, 61847 Locale failures in libgfortran

2014-11-05 Thread Janne Blomqvist
Hi, the attached patch fixes a few locale related failures in libgfortran, in the case where the POSIX 2008 extended locale functionality and extensions strto{f,d,ld}_l are present. These failures typically occur when libgfortran is used from a program which has set the locale with setlocale(), a

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 11:29:19AM +0100, Marek Polacek wrote: > On Wed, Nov 05, 2014 at 12:54:37PM +0300, Yury Gribov wrote: > > Are you going to work on ASan soon? I could rebase my patches on top of > > Marek's infrastructure. > > I'm not going to work on ASan today or tomorrow, but it'd be ni

Re: [PATCH, PR63742][ARM] Fix arm *movhi_insn_arch4 pattern for big-endian

2014-11-05 Thread Ramana Radhakrishnan
On 05/11/14 07:09, Yangfei (Felix) wrote: Hi, This patch fixes PR63742 by improving arm *movhi_insn_arch4 pattern to make it works under big-endian. The idea is simple: Use movw for certain const source operand instead of ldrh. And exclude the const values which cannot be handled

Re: [PATCH, libfortran] PR 47007, 61847 Locale failures in libgfortran

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 12:48:01PM +0200, Janne Blomqvist wrote: > @@ -3528,6 +3533,11 @@ finalize_transfer (st_parameter_dt *dtp) >if ((dtp->common.flags & IOPARM_DT_HAS_SIZE) != 0) > *dtp->size = dtp->u.p.size_used; > > +#ifdef HAVE_USELOCALE > + if (dtp->u.p.old_locale != (locale_t)

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Marek Polacek
On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote: > On Wed, Nov 05, 2014 at 11:29:19AM +0100, Marek Polacek wrote: > > On Wed, Nov 05, 2014 at 12:54:37PM +0300, Yury Gribov wrote: > > > Are you going to work on ASan soon? I could rebase my patches on top of > > > Marek's infrastructur

Re: [PATCH][AARCH64]Add ACLE arch-related predefined macros

2014-11-05 Thread Marcus Shawcroft
On 31 October 2014 14:37, Renlin Li wrote: > Hi all, > > This is a simple patch to add arch-related macros defined ACLE 2.0. > > aarch64-none-elf target is tested on the model, no new issues. > > Is this Okay for trunk? > > gcc/ChangeLog: > > 2014-10-31 Renlin Li > > * config/aarch64/aarch6

Re: [4.9 RFA PATCH, RTL-optimization]: Backport recent AND-alignment alias fixes to 4.9 branch

2014-11-05 Thread Richard Biener
On Wed, 5 Nov 2014, Uros Bizjak wrote: > Ping for [1], quoted below. Ok. Thanks, Richard. > [1] https://gcc.gnu.org/ml/gcc-patches/2014-10/msg03189.html > > Thanks, > Uros. > > On Thu, Oct 30, 2014 at 11:38 AM, Uros Bizjak wrote: > > Hello! > > > > I would like to backport recent alias fixes

Re: [PATCH, libfortran] PR 47007, 61847 Locale failures in libgfortran

2014-11-05 Thread Janne Blomqvist
On Wed, Nov 5, 2014 at 1:07 PM, Jakub Jelinek wrote: > On Wed, Nov 05, 2014 at 12:48:01PM +0200, Janne Blomqvist wrote: >> @@ -3528,6 +3533,11 @@ finalize_transfer (st_parameter_dt *dtp) >>if ((dtp->common.flags & IOPARM_DT_HAS_SIZE) != 0) >> *dtp->size = dtp->u.p.size_used; >> >> +#ifdef

[ARM] RFA: Use new rtl iterators in arm_find_sub_rtx_with_code

2014-11-05 Thread Richard Sandiford
I think these functions only want to iterate over instruction patterns rather than whole instructions (which would include things like REG_EQUAL notes), since only the patterns are relevant for finding dependencies. There's then no need to check for null rtxes. Tested by making sure there were no

[ARM] RFA: Use new rtl iterators in arm_cannot_copy_insn

2014-11-05 Thread Richard Sandiford
Tested in the same way as the aarch-common.c patch. OK to install? Thanks, Richard gcc/ * config/arm/arm.c (arm_note_pic_base): Delete. (arm_cannot_copy_insn_p): Use FOR_EACH_SUBRTX. Index: gcc/config/arm/arm.c ===

[ARM] RFA: Use new rtl iterators in arm_tls_referenced_p

2014-11-05 Thread Richard Sandiford
Tested in the same way as the aarch-common.c patch. OK to install? Thanks, Richard gcc/ * config/arm/arm.c: Include rtl-iter.h. (arm_tls_referenced_p_1): Delete. (arm_tls_referenced_p): Use FOR_EACH_SUBRTX. Index: gcc/config/arm/arm.c ===

[AArch64] RFA: Use new rtl iterators in arm_cannot_copy_insn

2014-11-05 Thread Richard Sandiford
This is part of a series to remove uses of for_each_rtx from the ports. Tested by making sure there were no code changes for gcc.dg, gcc.c-torture and g++.dg for aarch64-linux-gnu. OK to install? Thanks, Richard gcc/ * config/aarch64/aarch64.c: Include rtl-iter.h. (aarch64_tls_

Re: [PATCH, libfortran] PR 47007, 61847 Locale failures in libgfortran

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 01:48:32PM +0200, Janne Blomqvist wrote: > On Wed, Nov 5, 2014 at 1:07 PM, Jakub Jelinek wrote: > > On Wed, Nov 05, 2014 at 12:48:01PM +0200, Janne Blomqvist wrote: > >> @@ -3528,6 +3533,11 @@ finalize_transfer (st_parameter_dt *dtp) > >>if ((dtp->common.flags & IOPARM_

Re: [PARCH 1/2, x86, PR63534] Fix darwin bootstrap

2014-11-05 Thread Eric Botcazou
> Now if your argument is that IRA/LRA handle this, that's fine, a pointer > to that code would be appreciated so that it can be quickly audited. > Certainly the old local-alloc/global-alloc had magic for setjmp/longjmp > and maybe IRA/LRA does too, but it's better to be sure than just assume. See

Re: The nvptx port [8/11+] Write undefined decls.

2014-11-05 Thread Bernd Schmidt
On 10/22/2014 08:11 PM, Jeff Law wrote: I'm not going to insist you do this in the same way as the PA. That was a different era -- we had significant motivation to make things work in such a way that everything could be buried in the pa specific files. That sometimes led to less than optimal app

Re: [Patch, testsuite] [AArch64,ARM] support bswap tests on aarch64_be

2014-11-05 Thread Christophe Lyon
On 31 October 2014 18:15, Ramana Radhakrishnan wrote: > On Wed, Oct 29, 2014 at 1:22 PM, Christophe Lyon > wrote: >> Hi, >> >> Following discussions after Thomas's patches improving bswap support >> https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01279.html >> >> I noticed that: >> * the associated

Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests

2014-11-05 Thread Christophe Lyon
ping? On 26 October 2014 17:50, Christophe Lyon wrote: > On 24 October 2014 10:07, Marcus Shawcroft wrote: >> On 21 October 2014 14:02, Christophe Lyon wrote: >>> This patch series is an updated version of the series I sent here: >>> https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00022.html >>>

Re: [Patch, testsuite] [AArch64,ARM] support bswap tests on aarch64_be

2014-11-05 Thread Marcus Shawcroft
On 5 November 2014 12:08, Christophe Lyon wrote: > On 31 October 2014 18:15, Ramana Radhakrishnan > wrote: >> On Wed, Oct 29, 2014 at 1:22 PM, Christophe Lyon >> wrote: >>> Hi, >>> >>> Following discussions after Thomas's patches improving bswap support >>> https://gcc.gnu.org/ml/gcc-patches/201

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 02:23 PM, Marek Polacek wrote: On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 11:29:19AM +0100, Marek Polacek wrote: On Wed, Nov 05, 2014 at 12:54:37PM +0300, Yury Gribov wrote: Are you going to work on ASan soon? I could rebase my patches

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 03:16:49PM +0300, Yury Gribov wrote: > On 11/05/2014 02:23 PM, Marek Polacek wrote: > >On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote: > >>On Wed, Nov 05, 2014 at 11:29:19AM +0100, Marek Polacek wrote: > >>>On Wed, Nov 05, 2014 at 12:54:37PM +0300, Yury Gribov

Re: [RTL, Patch] Int div by constant compilation enhancement

2014-11-05 Thread Eric Botcazou
> 2014-11-03 Alex Velenko > > * simplify-rtx.c (simplify_binary_operation_1): Div check added. > * rtl.h (SUBREG_P): New macro added. Present tense in Change entries: * rtl.h (SUBREG_P): New macro. * simplify-rtx.c (simplify_binary_operation_1): Simplify consecutive

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 03:21 PM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 03:16:49PM +0300, Yury Gribov wrote: On 11/05/2014 02:23 PM, Marek Polacek wrote: On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 11:29:19AM +0100, Marek Polacek wrote: On Wed, Nov 05,

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-11-05 Thread Ilya Verbin
On 03 Nov 10:24, Jakub Jelinek wrote: > On Tue, Oct 28, 2014 at 10:30:47PM +0300, Ilya Verbin wrote: > > @@ -474,6 +475,13 @@ cgraph_node::create (tree decl) > >gcc_assert (TREE_CODE (decl) == FUNCTION_DECL); > > > >node->decl = decl; > > + > > + if (lookup_attribute ("omp declare target

Re: [PATCH 2/n] OpenMP 4.0 offloading infrastructure: LTO streaming

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 03:46:55PM +0300, Ilya Verbin wrote: > Maybe also with this change? > > diff --git a/gcc/omp-low.c b/gcc/omp-low.c > index 4e9ed25..beae5b5 100644 > --- a/gcc/omp-low.c > +++ b/gcc/omp-low.c > @@ -1653,8 +1653,7 @@ scan_sharing_clauses (tree clauses, omp_context *ctx) >

Re: [gofrontend-dev] [PATCH 1/4] Gccgo port to s390[x] -- part II

2014-11-05 Thread Dominik Vogt
On Wed, Nov 05, 2014 at 11:31:28AM +0100, Dominik Vogt wrote: > On Tue, Nov 04, 2014 at 02:39:34PM -0800, Ian Taylor wrote: > > Note that libgo/runtime/runtime.c now refers to S390_HAVE_STCKF. It's > > not obvious to me that that is defined anywhere. Perhaps it is in a > > later patch in this ser

Re: [PARCH 1/2, x86, PR63534] Fix darwin bootstrap

2014-11-05 Thread Evgeny Stupachenko
On Tue, Nov 4, 2014 at 1:40 AM, Jeff Law wrote: > On 11/01/14 06:39, Evgeny Stupachenko wrote: >> >> When PIC register is pseudo there is nothing special about it's value >> that setjmp can hurt. So if the pseudo register lives across >> setjmp_receiver RA should care about correct allocation (in

Re: [PARCH 1/2, x86, PR63534] Fix darwin bootstrap

2014-11-05 Thread Evgeny Stupachenko
We don't emit extra SET_GOT. That is beneficial. As for stack usage, that is RA to decide which register is more beneficial to put on stack. On Sat, Nov 1, 2014 at 8:33 PM, Mike Stump wrote: > On Nov 1, 2014, at 5:39 AM, Evgeny Stupachenko wrote: >> When PIC register is pseudo there is nothing s

Re: The nvptx port [10/11+] Target files

2014-11-05 Thread Bernd Schmidt
On 11/04/2014 05:51 PM, Bernd Schmidt wrote: On 11/04/2014 05:48 PM, Richard Henderson wrote: On 10/28/2014 03:56 PM, Bernd Schmidt wrote: +nvptx_ptx_type_from_mode (enum machine_mode mode, bool promote) +{ + switch (mode) +{ +case BLKmode: + return ".b8"; +case BImode: +

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 03:34 PM, Yury Gribov wrote: On 11/05/2014 03:21 PM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 03:16:49PM +0300, Yury Gribov wrote: On 11/05/2014 02:23 PM, Marek Polacek wrote: On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 11:29:19AM +

Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests

2014-11-05 Thread Ramana Radhakrishnan
On Sun, Oct 26, 2014 at 4:50 PM, Christophe Lyon wrote: > On 24 October 2014 10:07, Marcus Shawcroft wrote: >> On 21 October 2014 14:02, Christophe Lyon wrote: >>> This patch series is an updated version of the series I sent here: >>> https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00022.html >>>

Re: [Patch ARM-AArch64/testsuite v3 00/21] Neon intrinsics executable tests

2014-11-05 Thread Marcus Shawcroft
On 26 October 2014 16:50, Christophe Lyon wrote: > I've just realized afterwards that the tests aren't guarded against > targets not supporting Neon. > > How about adding the attached small patch? +if {[istarget arm*-*-*] +&& ![check_effective_target_arm_neon_ok]} then { + return +} + Umm,

Re: [PATCH 3/n] OpenMP 4.0 offloading infrastructure: offload tables

2014-11-05 Thread Ilya Verbin
On 08 Oct 11:23, Jakub Jelinek wrote: > LGTM, with the requested var/section renames. > Would like if Honza and/or Richard had a look at the cgraph/LTO stuff > in the patch though. Since patch 2 was updated, this patch also should be updated. Now the offload_vars array is filled in varpool_node::g

Re: [PATCH][AArch64][4.8] LINK_SPEC changes for Cortex-A53 erratum 835769 workaround

2014-11-05 Thread Marcus Shawcroft
On 22 October 2014 15:20, Kyrill Tkachov wrote: > Hi all, > > This is the 4.8 backport of the LINK_SPEC changes to pass down the linker > option > --fix-cortex-a53-835769 > > Bootstrapped and tested on aarch64-none-linux-gnu. > This depends on the patches under review at: > https://gcc.gnu.org/ml/

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Jakub Jelinek
On Wed, Nov 05, 2014 at 04:13:01PM +0300, Yury Gribov wrote: > >>Wouldn't it break most uses of __asan_poison_memory_region ? > > > >Most probably but I wonder if we should ask people to simply do asm > >volatile with memory clobber in this case? And we probably shouldn't > >call the whole thing i

Re: [Ping] [PATCH, 9/10] aarch64: generate conditional compare instructions

2014-11-05 Thread Richard Henderson
On 11/05/2014 10:05 AM, Zhenqiang Chen wrote: > I had retested all the ccmp patches. > > Bootstrap and no make check regression on X86-64. > Bootstrap and no make check regression on AARCH64 qemu. > > OK for trunk? No patch? Or what is it that you're wanting approval for? r~

Re: [PATCH] Optimize UBSAN_NULL checks, add sanopt.c

2014-11-05 Thread Yury Gribov
On 11/05/2014 04:23 PM, Jakub Jelinek wrote: On Wed, Nov 05, 2014 at 04:13:01PM +0300, Yury Gribov wrote: Wouldn't it break most uses of __asan_poison_memory_region ? Most probably but I wonder if we should ask people to simply do asm volatile with memory clobber in this case? And we probably

Re: [libcc1, build] Enable libcc1 on Solaris

2014-11-05 Thread Phil Muldoon
On 03/11/14 16:54, Rainer Orth wrote: > I noticed that the new libcc1 wasn't built on Solaris. This happens > because socketpair doesn't live in libc, but in libsocket instead. To > deal with this, I've copied the libgo (and libjava) code to detect the > need for libsocket and libnsl. Once the b

[PATCH] PR 63721 IPA ICF cause atomic-comp-swap-release-acquire.c ICE

2014-11-05 Thread Jiong Wang
the same ICE will happen on x86-64, if compile with -O2 -fPIC. the reason is for the following two functions, they are identical, so IPA-ICF pass try to transform the second function to call the first one directly. int atomic_compare_exchange_STRONG_RELEASE_ACQUIRE (int a, int b) { return __

libstdc++ new deque failures

2014-11-05 Thread David Edelsohn
Jonathan, I still am seeing new failures in the libstdc++ deque testsuite as of last night. I don't know if you still are working through the fallout from the earlier patches, but I wanted to make you aware. AIX defaults to 32 bit. A template was not initialized for int? FAIL: 23_containers/deq

[PATCH, committed] Fix AIX testsuite failures

2014-11-05 Thread David Edelsohn
The appended patch XFAILs or adjusts testcases to avoid spurious warnings on AIX. Bootstrapped on powerpc-ibm-aix7.1.0.0 Thanks, David * gcc.dg/torture/pr59166.c: XFAIL on AIX. * g++.dg/ext/visitibility/anon1.C: XFAIL on AIX. * g++.dg/opt/pr60002.C: XFAIL on AIX.

Re: [PATCH, committed] Fix AIX testsuite failures

2014-11-05 Thread Rainer Orth
Hi David, > The appended patch XFAILs or adjusts testcases to avoid spurious > warnings on AIX. > > Bootstrapped on powerpc-ibm-aix7.1.0.0 > > Thanks, David > > * gcc.dg/torture/pr59166.c: XFAIL on AIX. > * g++.dg/ext/visitibility/anon1.C: XFAIL on AIX. > * g++.dg/opt/pr600

Re: [PATCHv5][Kasan] Allow to override Asan shadow offset from command line

2014-11-05 Thread Kirill Yukhin
Hello, On 24 Oct 17:56, Yury Gribov wrote: ... > +const struct test_data_t test_data[] = { > + { STRTOL, "-0x8000", 0, -0x8000L, 0 }, ... > + switch (test_data[i].fun) > + { > + case STRTOL: > + res = strtol (test_data[i].nptr, 0, test_data[i].base); > + bre

Re: nvptx offloading patches [1/n]

2014-11-05 Thread Michael Matz
Hi, On Tue, 4 Nov 2014, Jeff Law wrote: > They still need to agree on the layout of the structure. And assuming > it'll always be memcpy perhaps isn't wise. Consider the possibility > that one day (perhaps soon) the host and GPU may share address space & > memory. Not only soon, there is al

RE: [PATCH, i686] Fix for asan test failures with -m32 happened after EBX enabling in PIC mode

2014-11-05 Thread Zamyatin, Igor
> > Hi! > > > > Following patch (moving initialization of pic_offset_table_rtx > earlier) fixes failures for asan tests on 32 bits in PIC mode mentioned here - > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63534#c48 > > > > > > Bootstrapped/regtested on x86_64, i686 > > > > Is it ok for trunk? > >

Re: [patch,gomp-4_0-branch] acc nested function support

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 16:45 -0800, Cesar Philippidis wrote: > Here's an updated version of my nested function patch. > > David, I tweaked the gimple class hierarchy a little bit. Here's what > the updated class diagram looks like: > > + gimple_statement_omp > | |layout: GSS_OMP.

[gomp4] Remove unused BUILT_IN_OMP_SET_NUM_THREADS (was: various OpenACC/PTX built-ins and a reduction tweak)

2014-11-05 Thread Thomas Schwinge
Hi! On Thu, 18 Sep 2014 20:43:20 +0200, I wrote: > On Tue, 16 Sep 2014 17:32:54 -0700, Cesar Philippidis > wrote: > > The patch [...] > > --- a/gcc/omp-builtins.def > > +++ b/gcc/omp-builtins.def > > @@ -236,6 +236,3 @@ DEF_GOMP_BUILTIN (BUILT_IN_GOMP_TARGET_UPDATE, > > "GOMP_target_update", >

[gomp4] Remove unused OACC_WAIT (was: acc dealloc map)

2014-11-05 Thread Thomas Schwinge
Hi! On Mon, 20 Oct 2014 13:30:23 -0700, Cesar Philippidis wrote: > 2014-10-20 Cesar Philippidis > > gcc/ > * gimplify.c [...] > (gimplify_expr): Remove OACC_WAIT, since it handled directly by the > front ends. In r217136, I have now completely removed it: commit 1f7

Re: [gofrontend-dev] [PATCH 4/4] Gccgo port to s390[x] -- part II

2014-11-05 Thread Ian Taylor
On Wed, Nov 5, 2014 at 2:05 AM, Dominik Vogt wrote: > On Tue, Nov 04, 2014 at 08:16:51PM -0800, Ian Taylor wrote: >> I committed the change to go-test.exp. Thanks. >> >> The other changes are not OK. As described in >> gcc/testsuite/go.test/test/README.gcc, the files in >> gcc/testsuite/go.test/

[jit] Use ISALPHA and ISALNUM rather than writing our own

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 14:39 -0700, Jeff Law wrote: > On 11/04/14 09:57, David Malcolm wrote: > >>> +#define IS_ASCII_DIGIT(CHAR) \ > >>> + ((CHAR) >= '0' && (CHAR) <='9') > >>> + > >>> +#define IS_ASCII_ALNUM(CHAR) \ > >>> + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) > >> Can't we rely on t

[gomp4] OpenACC Fortran testsuite: Expect some things to work by now.

2014-11-05 Thread Thomas Schwinge
Hi! Committed to gomp-4_0-branch in r217137: commit 83c3ae92fb16c23a782f012a49dd7aa1fcd01287 Author: tschwinge Date: Wed Nov 5 15:54:45 2014 + OpenACC Fortran testsuite: Expect some things to work by now. gcc/testsuite/ * gfortran.dg/goacc/data-tree.f95: Remove dg

Re: [PATCH AVX512] Fix dg.torture tests with avx512

2014-11-05 Thread Ilya Tocar
On 03 Nov 11:21, Jakub Jelinek wrote: > On Fri, Oct 31, 2014 at 11:17:07AM +0100, Uros Bizjak wrote: > > I'd like to ask Jakub for a review of the above two parts, other parts > > are OK with a rename (as mentioned above). > > Looks ok to me. Where the ICEs discovered just by normal make check or

Re: [gomp4] OpenACC / C++

2014-11-05 Thread Thomas Schwinge
Hi! On Wed, 15 Oct 2014 11:21:05 -0500, James Norris wrote: > This patch adds OpenACC support to C++ in the gomp4 branch. I found a few missing pieces; applied to gomp-4_0-branch in r217139: commit 09b8ef34550c377610c7a01aa2057fc8297e1b0a Author: tschwinge Date: Wed Nov 5 15:58:09 2014 +000

[gomp4] libgomp OpenACC testsuite: Remove two obsolete test cases.

2014-11-05 Thread Thomas Schwinge
Hi! Applied to gomp-4_0-branch in r217140: commit 3838e13dedee9217b067cf2ab4b3fb5bb7d5cf68 Author: tschwinge Date: Wed Nov 5 16:02:30 2014 + libgomp OpenACC testsuite: Remove two obsolete test cases. libgomp/ * testsuite/libgomp.oacc-c/goacc_kernels.c: Remove file

Re: [PING][PATCH] Don't call fatal_error before error reporting has been initialized.

2014-11-05 Thread Ilya Tocar
Ping. On 20 Oct 19:25, Ilya Tocar wrote: > Same in collect2. > > On 09 Oct 15:40, Ilya Tocar wrote: > > Ping. > > > > On 29 Sep 18:02, Ilya Tocar wrote: > > > Hi, > > > > > > Currently if call to atexit (lto_wrapper_cleanup) fails we > > > won't report error as we haven't initialized error-repo

Re: [debug-early] emit locals early patchset

2014-11-05 Thread Martin Jambor
On Tue, Oct 28, 2014 at 03:57:43PM +0100, Richard Biener wrote: > On Tue, Oct 28, 2014 at 1:00 AM, Aldy Hernandez wrote: > > Gentlemen! > > > > My apologies for the big patch. In getting locals emitted early (parameters > > and locally scoped variables), I ran into many things which were in need

Re: [gomp4] Rationalise thread-local variables in libgomp OpenACC support

2014-11-05 Thread Thomas Schwinge
Hi! On Wed, 29 Oct 2014 15:20:17 +, Julian Brown wrote: > On Tue, 28 Oct 2014 11:16:19 + > Julian Brown wrote: > > This patch rationalises TLS support by moving all thread-local > > variables into a single structure. Because this meant interfering with > > how per-thread/per-device init

[PATCH] Correctly check dg-require-effective-target in avx512 tests.

2014-11-05 Thread Ilya Tocar
Hi, Currently we only check for dg-require-effective-target avx512vl in avx512vl tests. We should also check for avx512dq/avx512bw. Patch bwllow does this. Ok for trunk? 2014-11-05 Ilya Tocar * gcc.target/i386/avx512vl-vandnpd-2.c: Fix dg-require-effective-target cehck.

[gomp4] OpenACC documentation updates.

2014-11-05 Thread Thomas Schwinge
Hi! Applied to gomp-4_0-branch in r217142: commit 0c5178ff5207bf1ede83070629c7d76fbbdf1afb Author: tschwinge Date: Wed Nov 5 16:12:51 2014 + OpenACC documentation updates. gcc/ * invoke.texi: Update for OpenACC. * sourcebuild.texi: Likewise. gcc/fo

[gomp4] libgomp testsuite: OpenACC C++ testing (was: [2/3] OpenACC 2.0 support for libgomp - new tests)

2014-11-05 Thread Thomas Schwinge
Hi! Applied to gomp-4_0-branch in r217143: commit a78a06124f4047ec46a85e539e83640cc973aec1 Author: tschwinge Date: Wed Nov 5 16:16:14 2014 + libgomp testsuite: OpenACC C++ testing. libgomp/ * testsuite/libgomp.oacc-c++/c++.exp: Enable libgomp.oacc-c-c++-co

Re: [PATCH] Correctly check dg-require-effective-target in avx512 tests.

2014-11-05 Thread Uros Bizjak
On Wed, Nov 5, 2014 at 5:14 PM, Ilya Tocar wrote: > Hi, > > Currently we only check for dg-require-effective-target avx512vl in > avx512vl tests. We should also check for avx512dq/avx512bw. > Patch bwllow does this. > Ok for trunk? > > 2014-11-05 Ilya Tocar > > * gcc.target/i386/avx512v

Enable -fextended-identifiers by default

2014-11-05 Thread Joseph Myers
As proposed at , this patch enables -fextended-identifiers by default for all standard versions including this feature (all C++ versions, C99 and above for C, but not C90 / C94 / gnu89 / preprocessing assembler). It adds a couple of tests for areas

[gomp4] OpenACC cache directive for C.

2014-11-05 Thread Thomas Schwinge
Hi! In r217145, I applied Jim's patch to gomp-4_0-branch: commit 4361f9b6b2c74c2961c3a5290a4945abe2d7a444 Author: tschwinge Date: Wed Nov 5 16:26:47 2014 + OpenACC cache directive for C. gcc/c-family/ * c-pragma.c (oacc_pragmas): Add "cache". gcc/c/

[gomp4] OpenACC cache directive maintenance (was: [PATCH 4/6] [GOMP4] OpenACC 1.0+ support in fortran front-end)

2014-11-05 Thread Thomas Schwinge
Hi! On Fri, 24 Jan 2014 20:33:35 +0100, I wrote: > On Thu, 23 Jan 2014 22:04:45 +0400, Ilmir Usmanov > wrote: > > Subject: [PATCH 4/6] OpenACC GENERIC nodes > > --- a/gcc/tree-core.h > > +++ b/gcc/tree-core.h > > @@ -216,12 +216,18 @@ enum omp_clause_code { > > + /* Internal structure to hold

[PING 2] Enhance array types debug info. for Ada

2014-11-05 Thread Pierre-Marie de Rodat
Hello, Ping for https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00694.html Thanks in advance! Regards, -- Pierre-Marie de Rodat

Re: [gomp4] OpenACC cache directive maintenance

2014-11-05 Thread Thomas Schwinge
Hi Cesar! On Wed, 05 Nov 2014 17:36:46 +0100, I wrote: > In r217146, I applied the following to gomp-4_0-branch: > > commit e8e44b733808997d06c0cdf9bf5756ce03530f42 > Author: tschwinge > Date: Wed Nov 5 16:35:30 2014 + > > OpenACC cache directive maintenance. > > gcc/c/ >

[PATCH] Fix libbacktrace and libiberty tests fail on sanitized GCC due to wrong link options.

2014-11-05 Thread Maxim Ostapenko
Hi, When I ran Asan tests under Asan-bootstrapped GCC 5.0, I've noted, that tests for libiberty and libbacktrace fail to link with sanitized libbacktrace.a and libiberty.a because of missing -static-libasan -fsanitize=address linker flags. This patch adds necessary flags to provide a linkage o

[gomp4] Testing of C/C++ OpenACC cache directive (was: OpenACC cache directive for C)

2014-11-05 Thread Thomas Schwinge
Hi! On Wed, 05 Nov 2014 17:29:19 +0100, I wrote: > In r217145, I applied Jim's patch to gomp-4_0-branch: > * testsuite/libgomp.oacc-c/cache-1.c: New file. > * testsuite/libgomp.oacc-c++/cache-1.C: Likewise. Applied to gomp-4_0-branch in r217147: commit 267cffc4105255a8372fb5788fdcbb

[gomp4] OpenACC update host/self maintenance (was: acc enter/exit data)

2014-11-05 Thread Thomas Schwinge
Hi! On Thu, 30 Oct 2014 17:11:04 -0700, Cesar Philippidis wrote: > gcc/fortran/ > * gfortran.h (enum OMP_LIST_HOST): Remove. > (enum OMP_LIST_DEVICE, OMP_LIST_DEVICE): Remove. > * dump-parse-tree.c (show_omp_clauses): Remove OMP_LIST_HOST and > OMP_LIST_DEVICE from

RE: [PATCHv3][MIPS] Implement O32 ABI extensions (GCC)

2014-11-05 Thread Matthew Fortune
Hi Catherine, The full patch is attached and the delta from v2 is inline below. Testing (O32): MIPS I - FP32, MIPS II - FP32, MIPS II - FPXX MIPS32 - FP32, MIPS32 - FPXX, MIPS32 - FPXX ODDSPREG, MIPS32R2 - FP32, MIPS32R2 - FPXX, MIPS32R2 - FPXX ODDSPREG, MIPS32R2 - FP64, MIPS32R2 - FP64A One of

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 4 November 2014 23:40, Jeff Law wrote: > On 10/25/14 04:20, Anthony Brandon wrote: >> >> Hi, >> >> Sorry for the delay. Here are the updated diff and changelog. >> >> gcc/testsuite/ChangeLog: >> >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * gcc.misc-tests/output.exp: New

Re: libstdc++ new deque failures

2014-11-05 Thread Jonathan Wakely
On 5 November 2014 14:14, David Edelsohn wrote: > Jonathan, > > I still am seeing new failures in the libstdc++ deque testsuite as of > last night. I don't know if you still are working through the fallout > from the earlier patches, but I wanted to make you aware. Yes, those tests are meant to f

[gomp4] Use GOMP_OFFLOAD_ prefix for (OpenACC) plugin hooks

2014-11-05 Thread Julian Brown
Hi, Mirroring changes in Ilya Verbin's libgomp offloading pieces posted to trunk, this patch adds a prefix of GOMP_OFFLOAD_ to the OpenACC plugin hooks. Some of these bits will not be needed for a trunk version of the patch once Ilya's patch is approved (I'm hoping other incompatibilities haven't

[gomp4] Move libgomp plugins into subdirectory

2014-11-05 Thread Julian Brown
Hi, This patch moves plugin-nvptx.c and plugin-host.c (from oacc-host.c) into a new "plugin" subdirectory, as requested by Jakub, and to match more closely the layout of the Intel MIC pieces. This also moves the autotools bits to enable the NVPTX plugin and locate CUDA libraries into the plugin di

[jit] Verify enum values earlier

2014-11-05 Thread David Malcolm
It wasn't clear to me that all of these enum values were being fully validated by the internals, and it's better to fail early (so we can report which function was at fault), so explicitly validate enum values at the API entrypoints. The new testcases bring the # of expected passes in jit.sum from

Add uniform_inside_sphere_distribution

2014-11-05 Thread Ed Smith-Rowland
This distribution has come in handy for me. It relies on uniform_on_sphere_distribution and like it, min and max have no real meaning. Unlike uniform_on_sphere_distribution which really is a random multidimensional unit vector users often want to pick the radius of the distribution. Unit radiu

Re: [Ada] Changes related to back-end inlining

2014-11-05 Thread Eric Botcazou
> 2014-10-31 Eric Botcazou > > * inline.adb (Has_Excluded_Declaration): With back-end inlining, > only return true for nested packages. > (Cannot_Inline): Issue errors/warnings whatever the optimization level > for back-end inlining and remove assertion. Here is a follo

[Ada] Fix location information of exception block

2014-11-05 Thread Eric Botcazou
This makes it so that an exception block doesn't inherit a bogus location information in SJLJ mode. Tested on x86_64-suse-linux, applied on the mainline. 2014-11-05 Eric Botcazou * gcc-interface/trans.c (Handled_Sequence_Of_Statements_to_gnu): Set the SLOC of the node on the

Re: [gomp4] Use GOMP_OFFLOAD_ prefix for (OpenACC) plugin hooks

2014-11-05 Thread Ilya Verbin
Hi, On 05 Nov 17:56, Julian Brown wrote: > +GOMP_OFFLOAD_register_image (void *host_table, void *target_data) > +GOMP_OFFLOAD_get_table (struct mapping_table **table) FYI, these interfaces may change in the near future. Currently GOMP_OFFLOAD_get_table returns a joint table for all images, offloa

[Ada] Fix ICE on type derived from private discriminated type

2014-11-05 Thread Eric Botcazou
The compiler aborts on a record type derived from a private discriminated record type without discriminant contraints, if the private discriminated record type is itself derived from another discriminated record type. Tested on x86_64-suse-linux, applied on the mainline. 2014-11-05 Eric Botca

[Ada] Fix crash on function with In-Out param returning discriminated type

2014-11-05 Thread Eric Botcazou
The compiler crashes on a function with an In-Out parameter which returns a discriminated record type with default discriminant. Tested on x86_64-suse-linux, applied on the mainline. 2014-11-05 Eric Botcazou * gcc-interface/trans.c (Subprogram_Body_to_gnu): For a function with

[jit] Add comments throughout libgccjit.c, and in libgccjit.h

2014-11-05 Thread David Malcolm
Committed to branch dmalcolm/jit: Also, add checking to ensure that gcc_jit_context_new_array_type fails with an error if given a negative size. gcc/jit/ChangeLog.jit: * docs/topics/expressions.rst (Type-coercion): Casts between pointer types are valid. * libgccjit.c: Docu

Re: [PATCH 10/27] New file: gcc/jit/libgccjit.c

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 14:39 -0700, Jeff Law wrote: > On 11/04/14 09:57, David Malcolm wrote: > >>> +#define IS_ASCII_DIGIT(CHAR) \ > >>> + ((CHAR) >= '0' && (CHAR) <='9') > >>> + > >>> +#define IS_ASCII_ALNUM(CHAR) \ > >>> + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) > >> Can't we rely on t

Re: [jit] Use ISALPHA and ISALNUM rather than writing our own

2014-11-05 Thread Jeff Law
On 11/05/14 08:48, David Malcolm wrote: On Tue, 2014-11-04 at 14:39 -0700, Jeff Law wrote: On 11/04/14 09:57, David Malcolm wrote: +#define IS_ASCII_DIGIT(CHAR) \ + ((CHAR) >= '0' && (CHAR) <='9') + +#define IS_ASCII_ALNUM(CHAR) \ + (IS_ASCII_ALPHA (CHAR) || IS_ASCII_DIGIT (CHAR)) Can't we r

Re: The nvptx port [8/11+] Write undefined decls.

2014-11-05 Thread Jeff Law
On 11/05/14 05:01, Bernd Schmidt wrote: On 10/22/2014 08:11 PM, Jeff Law wrote: I'm not going to insist you do this in the same way as the PA. That was a different era -- we had significant motivation to make things work in such a way that everything could be buried in the pa specific files. Th

  1   2   >