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 marc.gli...@inria.fr 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.

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

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 tejas.bela...@arm.com 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

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 Of

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

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 l...@redhat.com wrote: On 09/30/14 03:22, Bin Cheng wrote: 2014-09-30 Bin Chengbin.ch...@arm.com Mike Stumpmikest...@comcast.net * timevar.def (TV_SCHED_FUSION): New time var. * passes.def (pass_sched_fusion): New pass.

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_NULL

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 ubiz...@gmail.com: On Tue, Jul 15, 2014 at 1:01 PM, Ilya Enkovich enkovich@gmail.com

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

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 enkovich@gmail.com 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

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

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 hale.w...@arm.com 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

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

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

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 prachi.godb...@imgtec.com * 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

[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(),

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 nice to

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) 0) +

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 infrastructure.

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

2014-11-05 Thread Marcus Shawcroft
On 31 October 2014 14:37, Renlin Li renlin...@arm.com 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 renlin...@arm.com *

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 ubiz...@gmail.com wrote: Hello! I would like to backport recent alias

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 ja...@redhat.com 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;

[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

[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.

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 ja...@redhat.com 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

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

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 ramana@googlemail.com wrote: On Wed, Oct 29, 2014 at 1:22 PM, Christophe Lyon christophe.l...@linaro.org wrote: Hi, Following discussions after Thomas's patches improving bswap support https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01279.html

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 christophe.l...@linaro.org wrote: On 24 October 2014 10:07, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 21 October 2014 14:02, Christophe Lyon christophe.l...@linaro.org wrote: This patch series is an updated version of the series I

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 christophe.l...@linaro.org wrote: On 31 October 2014 18:15, Ramana Radhakrishnan ramana@googlemail.com wrote: On Wed, Oct 29, 2014 at 1:22 PM, Christophe Lyon christophe.l...@linaro.org wrote: Hi, Following discussions after Thomas's patches

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

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 wrote:

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

2014-11-05 Thread Eric Botcazou
2014-11-03 Alex Velenko alex.vele...@arm.com * 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):

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) if

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 series--I

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 l...@redhat.com 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

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 mikest...@comcast.net wrote: On Nov 1, 2014, at 5:39 AM, Evgeny Stupachenko evstu...@gmail.com wrote: When PIC

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 christophe.l...@linaro.org wrote: On 24 October 2014 10:07, Marcus Shawcroft marcus.shawcr...@gmail.com wrote: On 21 October 2014 14:02, Christophe Lyon christophe.l...@linaro.org wrote: This patch series is an updated version of the series I

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 christophe.l...@linaro.org 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 {

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

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 kyrylo.tkac...@arm.com 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:

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

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

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

[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:

[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/pr60002.C:

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); + break; As

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

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? ChangeLog:

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. Used

[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 ce...@codesourcery.com 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,

[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 ce...@codesourcery.com wrote: 2014-10-20 Cesar Philippidis ce...@codesourcery.com gcc/ * gimplify.c [...] (gimplify_expr): Remove OACC_WAIT, since it handled directly by the front ends. In r217136, I have

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 v...@linux.vnet.ibm.com 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

[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 the C library to give us

[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 tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Nov 5 15:54:45 2014 + OpenACC Fortran testsuite: Expect some things to work by now. gcc/testsuite/

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 only

Re: [gomp4] OpenACC / C++

2014-11-05 Thread Thomas Schwinge
Hi! On Wed, 15 Oct 2014 11:21:05 -0500, James Norris jnor...@codesourcery.com 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

[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 tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Nov 5 16:02:30 2014 + libgomp OpenACC testsuite: Remove two obsolete test cases. libgomp/ *

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-reporting

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 al...@redhat.com 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

[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 ilya.to...@intel.com * gcc.target/i386/avx512vl-vandnpd-2.c: Fix

[gomp4] OpenACC documentation updates.

2014-11-05 Thread Thomas Schwinge
Hi! Applied to gomp-4_0-branch in r217142: commit 0c5178ff5207bf1ede83070629c7d76fbbdf1afb Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Nov 5 16:12:51 2014 + OpenACC documentation updates. gcc/ * invoke.texi: Update for OpenACC.

[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 tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Nov 5 16:16:14 2014 + libgomp testsuite: OpenACC C++ testing. libgomp/ *

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 tocarip.in...@gmail.com 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 ilya.to...@intel.com

Enable -fextended-identifiers by default

2014-11-05 Thread Joseph Myers
As proposed at https://gcc.gnu.org/ml/gcc/2014-11/msg00014.html, 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 tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Nov 5 16:26:47 2014 + OpenACC cache directive for C. gcc/c-family/ * c-pragma.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 i.usma...@samsung.com 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

[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 tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed Nov 5 16:35:30 2014 + OpenACC cache

[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

[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

[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 ce...@codesourcery.com 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

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

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 4 November 2014 23:40, Jeff Law l...@redhat.com 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 anthony.bran...@gmail.com PR driver/36312 *

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 fail

[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

[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

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

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

2014-11-05 Thread Eric Botcazou
2014-10-31 Eric Botcazou ebotca...@adacore.com * 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.

[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 ebotca...@adacore.com * gcc-interface/trans.c (Handled_Sequence_Of_Statements_to_gnu): Set 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,

[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

[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 ebotca...@adacore.com * gcc-interface/trans.c (Subprogram_Body_to_gnu): For

[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:

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 the C library to give us

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 rely

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.

[jit] Drop the disabled debugging code within handle_locations

2014-11-05 Thread David Malcolm
On Tue, 2014-11-04 at 15:21 -0700, Jeff Law wrote: On 10/31/14 11:02, David Malcolm wrote: This files implements the gcc::jit::playback internal API, called by the dummy frontend to replay the public API calls made to the library. A thin wrapper around trees. gcc/jit/ *

Re: Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-11-05 Thread jb999
Jeff Law l...@redhat.com: On 10/23/14 08:30, jb...@gmx.de wrote: Jeff Law l...@redhat.com: On 10/21/14 12:21, jb...@gmx.de wrote: Jeff Law l...@redhat.com: On 10/21/14 16:13, Haswell wrote: The additional source must have the same requirement crossmodule-indircall-1.c has. *

  1   2   >