PR middle-end/16351 NULL dereference warnings

2015-07-22 Thread Manuel López-Ibáñez
I took the patch in https://gcc.gnu.org/ml/gcc-patches/2014-01/msg01715.html and removed the Wnull-attribute part, since most of it can be done from the FE as shown in https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01857.html and also to make the patch smaller and easier to review. I also fixed

Re: [AArch64/wwwdoc] Document -fpic support for small memory model

2015-07-22 Thread Jiong Wang
Jiong Wang writes: Marcus Shawcroft writes: On 26 June 2015 at 10:32, Jiong Wang jiong.w...@arm.com wrote: This patch respin https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01804.html. A new symbol classification SYMBOL_SMALL_GOT_28K added to represent symbol which needs go through GOT

Re: [WIP] OpenMP 4 NVPTX support

2015-07-22 Thread Jakub Jelinek
On Wed, Jul 22, 2015 at 06:04:20PM +0200, Thomas Schwinge wrote: On Tue, 21 Apr 2015 17:58:39 +0200, Jakub Jelinek ja...@redhat.com wrote: Attached is a minimal patch to get at least a trivial OpenMP 4.0 testcase offloading to NVPTX (the first patch). The second patch is WIP, just first

[PATH PR66926,PR66951} simple fix for ICE.

2015-07-22 Thread Yuri Rumyantsev
Hi All, Here is simple fix which fixes PR66926 and PR66951 - fix condition for renaming virtual operands to determine that statement is outside of loop. Bootstrap and regression testing did not show any new failures. Is it OK for trunk? gcc/ChangeLog 2015-07-22 Yuri Rumyantsev

[PATCH] Check TYPE_OVERFLOW_WRAPS for parloops reductions

2015-07-22 Thread Tom de Vries
[ was: Re: [RFC, PR66873] Use graphite for parloops ] On 22/07/15 13:02, Richard Biener wrote: On Wed, Jul 22, 2015 at 1:01 PM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Pop seb...@gmail.com wrote: Tom de Vries wrote: Fix reduction safety

Re: [C/C++ PATCH] Implement -Wshift-overflow (PR c++/55095) (take 3)

2015-07-22 Thread Marek Polacek
On Wed, Jul 22, 2015 at 07:48:47AM -0500, Segher Boessenkool wrote: vmx.exp sets a bunch of options and the test overrides that now. Options like -maltivec are pretty important for this test to work -- it #includes altivec.h, which does #error unless -maltivec is set, and things go downhill

[PATCH] Document ftrapv/fwrapv interaction

2015-07-22 Thread Tom de Vries
[ Re: [RFC, PR66873] Use graphite for parloops ] On 22/07/15 13:01, Richard Biener wrote: why only scalar floats? Please use FLOAT_TYPE_P. + if (INTEGRAL_TYPE_P (type)) +return (!TYPE_OVERFLOW_TRAPS (type) +TYPE_OVERFLOW_WRAPS (type)); it cannot both wrap and trap thus

Some additional comments for nvptx.c

2015-07-22 Thread Bernd Schmidt
Nathan asked me to go through nvptx.c and update some comments for things that aren't completely obvious. I've committed the following to trunk. Bernd Index: gcc/config/nvptx/nvptx.c === --- gcc/config/nvptx/nvptx.c (revision

PR c/16351 Extend Wnonnull for returns_nonnull

2015-07-22 Thread Manuel López-Ibáñez
While looking at PR c/16351, I noticed that all tests proposed for -Wnull-attribute (https://gcc.gnu.org/ml/gcc-patches/2014-01/msg01715.html) could be warned from the FEs by simply extending the existing Wnonnull. Bootstrapped and regression tested on x86_64-linux-gnu. OK? gcc/ChangeLog:

[PATCH] Don't allow unsafe reductions in graphite

2015-07-22 Thread Tom de Vries
[ was: Re: [RFC, PR66873] Use graphite for parloops ] On 22/07/15 13:02, Richard Biener wrote: On Wed, Jul 22, 2015 at 1:01 PM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Popseb...@gmail.com wrote: Tom de Vries wrote: Fix reduction safety

[PATCH] Enable reductions without fassociative-math in graphite

2015-07-22 Thread Tom de Vries
Hi, this patch allows non-float reductions to be detected by graphite, independent of whether fassociative-math (which only has effect for float operations) is set. Currently bootstrapping and reg-testing on x86_64. OK for trunk? Thanks, - Tom Enable reductions without fassociative-math in

Re: [WIP] OpenMP 4 NVPTX support

2015-07-22 Thread Thomas Schwinge
Hi! On Tue, 21 Apr 2015 17:58:39 +0200, Jakub Jelinek ja...@redhat.com wrote: Attached is a minimal patch to get at least a trivial OpenMP 4.0 testcase offloading to NVPTX (the first patch). The second patch is WIP, just first few needed changes to make libgomp to build for NVPTX (several

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

2015-07-22 Thread H.J. Lu
On Wed, Jul 22, 2015 at 6:59 AM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Jul 22, 2015 at 6:55 AM, Segher Boessenkool seg...@kernel.crashing.org wrote: On Wed, Jul 22, 2015 at 05:10:04AM -0700, H.J. Lu wrote: I got a feedback, suggesting __builtin_stack_top, instead of

Re: [AArch64/wwwdoc] Document -fpic support for small memory model

2015-07-22 Thread James Greenhalgh
On Fri, Jun 26, 2015 at 02:45:39PM +0100, Jiong Wang wrote: Marcus Shawcroft writes: 2015-06-26 Jiong Wang jiong.w...@arm.com wwwdocs/ * htdocs/gcc-6/changes.html (AArch64): Document -fpic for small model. Index: gcc-6/changes.html

[PING][PATCH, PR46193] Handle mix/max pointer reductions in parloops

2015-07-22 Thread Tom de Vries
On 13/07/15 13:02, Tom de Vries wrote: Hi, this patch fixes PR46193. It handles min and max reductions of pointer type in parloops. Bootstrapped and reg-tested on x86_64. OK for trunk? Ping. Thanks, - Tom 0001-Handle-mix-max-pointer-reductions-in-parloops.patch Handle mix/max pointer

Re: [PATCH] Fix ubsan tree sharing (PR sanitizer/66908)

2015-07-22 Thread Jakub Jelinek
On Wed, Jul 22, 2015 at 07:26:22PM +0200, Marek Polacek wrote: In this testcase we were generating an uninitialized variable when doing -fsanitize=shift,bounds sanitization. The shift instrumentation is done first; after that, the IR looks like res[i] = (m 31) ? __ubsan (... tab[i] ...)

[gomp4, committed] Set safelen to INT_MAX for oacc independent pragma

2015-07-22 Thread Tom de Vries
[ was; Re: [PATCH, gomp4] Propagate independent clause for OpenACC kernels pass ] On 14/07/15 11:48, Jakub Jelinek wrote: On Tue, Jul 14, 2015 at 05:35:28PM +0800, Chung-Lin Tang wrote: The wording of OpenACC independent is more simple: ... the independent clause tells the implementation that

Re: [PR64164] drop copyrename, integrate into expand

2015-07-22 Thread Alexandre Oliva
On Jul 21, 2015, Richard Biener richard.guent...@gmail.com wrote: On Sat, Jul 18, 2015 at 9:37 AM, Alexandre Oliva aol...@redhat.com wrote: On Jul 16, 2015, Alexandre Oliva aol...@redhat.com wrote: + /* If we are assigning parameters for a function, rather +than for a

Re: [PR64164] drop copyrename, integrate into expand

2015-07-22 Thread Alexandre Oliva
On Jul 21, 2015, Richard Biener richard.guent...@gmail.com wrote: On Sat, Jul 18, 2015 at 9:37 AM, Alexandre Oliva aol...@redhat.com wrote: + if (cfun-gimple_df) If the cfun-gimple_df check is to decide whether this is a call or a function then no, this can't work reliably. What is

[Ping] Re: [C++ Patch/RFC] PR 53184

2015-07-22 Thread Paolo Carlini
Hi, On 05/05/2015 11:24 PM, Paolo Carlini wrote: Hi, per the audit trail, this issue appears to boil down to two separate issues: - The warning doesn't appear universally useful, thus it would be nice to give it a name in order to enable disabling it. - As shown by the testcase, sometimes

[PATCH] Fix ubsan tree sharing (PR sanitizer/66908)

2015-07-22 Thread Marek Polacek
In this testcase we were generating an uninitialized variable when doing -fsanitize=shift,bounds sanitization. The shift instrumentation is done first; after that, the IR looks like res[i] = (m 31) ? __ubsan (... tab[i] ...) ? 0, ... tab[i] ...; where tab[i] are identical. That means that

Re: [gomp] Move openacc vector worker single handling to RTL

2015-07-22 Thread Nathan Sidwell
On 07/20/15 11:08, Nathan Sidwell wrote: On 07/20/15 09:01, Nathan Sidwell wrote: On 07/18/15 11:37, Thomas Schwinge wrote: Hi Nathan! For OpenACC nvptx offloading, there must still be something wrong; here's a count of the (non-deterministic!) regressions of ten runs of the libgomp

Re: [Patch, fortran] PR 37131, inline matmul

2015-07-22 Thread Thomas Koenig
Hi Mikael, However, it introduces regressions on matmul_bounds_{2,4,5}. It seems the incorrect extent runtime errors are completely optimized away (even at -O0). Any ideas? This is seriously wierd. It seems that the call to gfortran_error is really optimized away, because the middle-end

[PATCH][AArch64] Fix LINUX_TARGET_LINK_SPEC to be consistent with ARM

2015-07-22 Thread Szabolcs Nagy
Same as https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01387.html but for AArch64. -dynamic-linker is only passed to the linker if !static !shared. -rdynamic handling is changed too to be consistent with arm: only pass -export-dynamic if !static. 2015-07-22 Szabolcs Nagy szabolcs.n...@arm.com

Re: [C/C++ PATCH] Implement -Wtautological-compare (PR c++/66555, c/54979)

2015-07-22 Thread Marek Polacek
On Tue, Jul 14, 2015 at 11:42:21PM +0200, Marek Polacek wrote: On Tue, Jul 14, 2015 at 10:30:06PM +0100, Richard Sandiford wrote: Marek Polacek pola...@redhat.com writes: + /* Don't warn for e.g. + HOST_WIDE_INT n; + ... + if (n == (long) n) ... + */ + if

[PATCH] Fix default_binds_local_p_2 for extern protected data

2015-07-22 Thread Szabolcs Nagy
The commit https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=222184 changed a true to false in varasm.c: bool default_binds_local_p_2 (const_tree exp) { - return default_binds_local_p_3 (exp, flag_shlib != 0, true, true); + return default_binds_local_p_3 (exp, flag_shlib != 0, true,

[PATCH, i386]: Fix PR 66954, function multiversioning fails for target aes

2015-07-22 Thread Uros Bizjak
Straightforward implementation. libgcc/ChangeLog: 2015-07-22 Uros Bizjak ubiz...@gmail.com PR target/66954 * config/i386/cpuinfo.c (enum processor_features): Add FEATURE_AES. (get_available_features): Handle FEATURE_AES. gcc/ChangeLog: 2015-07-22 Uros Bizjak ubiz...@gmail.com

Re: [C/C++ PATCH] Implement -Wtautological-compare (PR c++/66555, c/54979)

2015-07-22 Thread Marek Polacek
On Wed, Jul 22, 2015 at 12:43:53PM -0600, Martin Sebor wrote: On 07/14/2015 09:18 AM, Marek Polacek wrote: Code such as if (i == i) is hardly ever desirable, so we should be able to warn about this to prevent dumb mistakes. I haven't tried the patch or even studied it very carefully but I

Re: [C/C++ PATCH] Implement -Wtautological-compare (PR c++/66555, c/54979)

2015-07-22 Thread Martin Sebor
On 07/14/2015 09:18 AM, Marek Polacek wrote: Code such as if (i == i) is hardly ever desirable, so we should be able to warn about this to prevent dumb mistakes. I haven't tried the patch or even studied it very carefully but I wonder if this is also the case when i is declared volatile. I.e.,

Re: [C/C++ PATCH] Implement -Wtautological-compare (PR c++/66555, c/54979)

2015-07-22 Thread Marek Polacek
On Wed, Jul 22, 2015 at 01:48:03PM -0600, Martin Sebor wrote: On 07/22/2015 01:06 PM, Marek Polacek wrote: On Wed, Jul 22, 2015 at 12:43:53PM -0600, Martin Sebor wrote: On 07/14/2015 09:18 AM, Marek Polacek wrote: Code such as if (i == i) is hardly ever desirable, so we should be able to warn

[PATCH][AArch64] elf toolchain does not pass -shared linker option

2015-07-22 Thread Szabolcs Nagy
Valid linker options should be treated for elf consistently with the linux-gnu target. I'm not sure about the undocumented -h option (blindly copied LINUX_TARGET_LINK_SPEC from aarch64-linux without the dynamic-linker flag). (Not passing -shared can cause broken vdso.so in the linux kernel when

[PATCH] Remove unused get_current_pass_name

2015-07-22 Thread Bernd Edlinger
Hi, I noticed recently that tree-pass.h contains a declaration of get_current_pass_name, but this function is not defined, and where ever we need the current pass name, we simply use current_pass-name.  So I would like to remove that declaration. Boot-strapped and regression-tested on

Re: [gomp4] libgomp: Cope with DejaGnu having no mechanism to transfer environment variables to remote boards

2015-07-22 Thread Thomas Schwinge
Hi! This is about communicating environment variables to the target for use in (libgomp) execution testing. In particular, the ACC_DEVICE_TYPE environment variable to select at runtime which offloading device to use. I had an ugly hack for that particular case,

[PATCH] Fix PR66952

2015-07-22 Thread Richard Biener
This PR shows an issue with ifcombine which ends up executing stmts producing range info under a different condition than before (always true). With a twisted enough maze we end up miscompiling the testcase for this reason. Thus the following patch which resets all flow-sensitive info on defs

[PATCH] Fix PR66945

2015-07-22 Thread Richard Biener
The following fixes issues that arise when a SSA propagator ends up deciding only a single outgoing edge is executable but the folder at substitute-and-fold time decides the other one is executable. This can of course only happen with undefined behavior (or with bugs...). In this case the

Re: [PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion

2015-07-22 Thread James Greenhalgh
On Tue, Jul 21, 2015 at 05:59:39PM +0100, Kyrill Tkachov wrote: Sorry, here's the correct version, which uses initialized instead of inited in one of the variable names. Some nits below. Kyrill 2015-07-21 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/aarch64/aarch64.c

Re: [PATCH] [PATCH][ARM] Fix split-live-ranges-for-shrink-wrap.c testcase.

2015-07-22 Thread Ramana Radhakrishnan
Committed to trunk r226036. Is patch ok for fsf-5? OK for all release branches where affected as this is a testism. Ramana kind regards, Alex

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

2015-07-22 Thread Tom de Vries
On 01/07/15 13:16, Tom de Vries wrote: 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

[PATCH] Fix match with result operands and conditions

2015-07-22 Thread Richard Biener
Currently the code generated for the following (stupid example) (match (integer_zerop @0) INTEGER_CST@0 (if (integer_zerop (@0 is wrong in not assigning anything to the result @0. The following obvious patch fixes that. We don't have a match pattern like the above so it doesn't affect

[PATCH] [AArch64] fix typo in vec_store_lanesoi_lanemode

2015-07-22 Thread Charles Baylis
Committed as obvious r226061. gcc/ChangeLog: 2015-07-22 Charles Baylis charles.bay...@linaro.org * config/aarch64/aarch64-simd.md (vec_store_lanesoi_lanemode): Fix typo in attribute. From 7d98f7fc82cfc3012b460e4f4f91200fedcb04db Mon Sep 17 00:00:00 2001 From: Charles Baylis

Re: [RFC, PR66873] Use graphite for parloops

2015-07-22 Thread Richard Biener
On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Pop seb...@gmail.com wrote: Tom de Vries wrote: Fix reduction safety checks * graphite-sese-to-poly.c (is_reduction_operation_p): Limit flag_associative_math to SCALAR_FLOAT_TYPE_P. Honour TYPE_OVERFLOW_TRAPS and

Re: [PATCH v3] [AArch64] PR63870 Improve error messages for NEON single lane memory access intrinsics

2015-07-22 Thread Charles Baylis
On 17 July 2015 at 09:32, James Greenhalgh james.greenha...@arm.com wrote: This seems an odd limitation, presumably this is a side effect of waiting until expand time to throw an error... It does suggest that we're tackling the problem in the wrong way by pushing this to so late in the

[PATCH] Fix canonical form of true/false conditions

2015-07-22 Thread Richard Biener
Currently fold_stmt via gimple_cond_set_condition_from_tree and gimple_cond_get_ops_from_tree and gimple_cond_make_false and gimple_cond_make_true do not agree on the canonical form of if (true) and if (false) resulting in spurious foldings. The following makes gimple_cond_make_false/true follow

[PATCH, nios2] Remove unused header from libgcc linux-atomic.c

2015-07-22 Thread Chung-Lin Tang
The asm/unistd.h header was used back when Nios II Linux used a syscall cmpxchg, long since removed and actually never got into the FSF trunk. Patch removes the #include, and the following error code #defines which are all no longer used. Committed. Chung-Lin 2015-07-22 Chung-Lin Tang

Re: Fold some equal to and not equal to patterns in match.pd

2015-07-22 Thread Richard Biener
On Tue, Jul 21, 2015 at 11:15 AM, Hurugalawadi, Naveen naveen.hurugalaw...@caviumnetworks.com wrote: Hi, Please find attached the patch which performs following patterns folding in match.pd:- a ==/!= a p+ b to b ==/!= 0. a N ==/!= 0 to a(-1N) ==/!= 0. a * N ==/!= 0 where N is a power of 2

Re: [AArch64] PR 63521. define REG_ALLOC_ORDER/HONOR_REG_ALLOC_ORDER

2015-07-22 Thread James Greenhalgh
On Wed, May 20, 2015 at 01:35:41PM +0100, Jiong Wang wrote: Current IRA still use both target macros in a few places. Tell IRA to use the order we defined rather than with it's own cost calculation. Allocate caller saved first, then callee saved. This is especially useful for LR/x30, as

Re: [PATCH 3/4] S390 -march=native related fixes

2015-07-22 Thread Dominik Vogt
On Tue, Jul 21, 2015 at 07:05:27PM +0200, Ulrich Weigand wrote: Dominik Vogt wrote: * config/s390/driver-native.c (s390_host_detect_local_cpu): Handle processor capabilities with -march=native. * config/s390/s390.h (MARCH_MTUNE_NATIVE_SPECS): Likewise. (DRIVER_SELF_SPECS):

Re: Fold some equal to and not equal to patterns in match.pd

2015-07-22 Thread Richard Biener
On Wed, Jul 22, 2015 at 2:40 AM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: On Tue, Jul 21, 2015 at 12:16 PM, Richard Biener richard.guent...@gmail.com wrote: On July 21, 2015 11:38:31 AM GMT+02:00, Jakub Jelinek ja...@redhat.com wrote: On Tue, Jul 21, 2015 at 09:15:31AM +,

Re: [RFC, PR66873] Use graphite for parloops

2015-07-22 Thread Richard Biener
On Wed, Jul 22, 2015 at 1:01 PM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Pop seb...@gmail.com wrote: Tom de Vries wrote: Fix reduction safety checks * graphite-sese-to-poly.c (is_reduction_operation_p): Limit

Re: [AArch64] PR63521 Define REG_ALLOC_ORDER/HONOR_REG_ALLOC_ORDER

2015-07-22 Thread Jiong Wang
Jiong Wang writes: Current IRA still use both target macros in a few places. Tell IRA to use the order we defined rather than with it's own cost calculation. Allocate caller saved first, then callee saved. This is especially useful for LR/x30, as it's free to allocate and is pure caller

New Ukrainian PO file for 'gcc' (version 5.2.0)

2015-07-22 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Ukrainian team of translators. The file is available at: http://translationproject.org/latest/gcc/uk.po (This file, 'gcc-5.2.0.uk.po', has

Re: [PR25530] Convert (unsigned t / 2) * 2 into (unsigned t ~1)

2015-07-22 Thread Richard Biener
On Tue, Jul 21, 2015 at 11:16 AM, Hurugalawadi, Naveen naveen.hurugalaw...@caviumnetworks.com wrote: Hi, handle exact_div differently, like fold-const.c does. Then expressing ~1 with the result expression is really excessive - you should simply build this with @1 - 1 if @1 is a power of two.

Re: [C/C++ PATCH] Implement -Wshift-overflow (PR c++/55095) (take 3)

2015-07-22 Thread Marek Polacek
On Wed, Jul 22, 2015 at 08:35:12AM -0400, David Edelsohn wrote: On Tue, Jul 21, 2015 at 5:59 AM, Marek Polacek pola...@redhat.com wrote: On Mon, Jul 20, 2015 at 04:23:08PM -0400, David Edelsohn wrote: This seems to have caused a number of new failures in the PPC testsuite for vmx/unpack.

Re: [C/C++ PATCH] Implement -Wshift-overflow (PR c++/55095) (take 3)

2015-07-22 Thread Segher Boessenkool
On Wed, Jul 22, 2015 at 02:39:14PM +0200, Marek Polacek wrote: On Wed, Jul 22, 2015 at 08:35:12AM -0400, David Edelsohn wrote: On Tue, Jul 21, 2015 at 5:59 AM, Marek Polacek pola...@redhat.com wrote: On Mon, Jul 20, 2015 at 04:23:08PM -0400, David Edelsohn wrote: This seems to have caused

RE: [PATCH, MIPS] Add -march=interaptiv

2015-07-22 Thread Robert Suchanek
Hi Catherine, gcc/ * config/mips/mips-cpus.def (interaptiv): Define. * config/mips/mips-tables.opt: Regenerate. * config/mips/mips.h (MIPS_ISA_LEVEL_SPEC): Map - march=interaptiv to -mips32r2. (BASE_DRIVER_SELF_SPECS): Likewise but map to -mdsp. *

Re: Fold some equal to and not equal to patterns in match.pd

2015-07-22 Thread Segher Boessenkool
On Tue, Jul 21, 2015 at 05:40:07PM -0700, Andrew Pinski wrote: The biggest question now becomes which way is the canonical form for gimple and we can decide to optimize it on the RTL level (combine) instead if it produces better code in those cases. combine does not do instruction selection in

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

2015-07-22 Thread Maxim Blumental
Changed simd-7 test for C and Fortran to lower the execution time to ~0.6 and ~1 second respectively while keeping the test representative. Measured the time like: $ time make check-target-libgomp RUNTESTFLAGS=c.exp=simd-7.c on a Linux x86-64 machine. (Without the patch I also had a very long

[PATCH] S/390: Fix cfi for GPR 2 FPR saves

2015-07-22 Thread Andreas Krebbel
Hi, GCC currently does not emit register save cfi information for the stack pointer register. Instead dwarf2cfi considers the load into an FPR as using a new CFA register from now on. Adding a CFA_REGISTER note prevent dwarf2cfi from interpreting the insn itself. This fixes the Glibc testcases

[PATCH] Add location to 'two conversions in a row' error

2015-07-22 Thread Richard Biener
... and improve wording. Committed as obvious. Richard. 2015-07-22 Richard Biener rguent...@suse.de * genmatch.c (expr::gen_transform): Clarify error message and display location. Index: gcc/genmatch.c === ---

[PATCH] S/390: Improve risbg usage

2015-07-22 Thread Andreas Krebbel
Hi, with the attached patch we use risbg in more situations. This especially helps the SpecCPU 400.perlbench testcase. Bootstrapped on s390 and s390x. No regressions. I'll commit the patch after waiting a few days for review comments. Bye, -Andreas- gcc/ChangeLog: 2015-07-22 Andreas

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

2015-07-22 Thread Segher Boessenkool
On Wed, Jul 22, 2015 at 05:10:04AM -0700, H.J. Lu wrote: I got a feedback, suggesting __builtin_stack_top, instead of __builtin_ia32_stack_top. But I don't know if + /* After the prologue, stack top is at -WORD(AP) in the current +frame. */ + emit_insn (gen_rtx_SET

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

2015-07-22 Thread H.J. Lu
On Wed, Jul 22, 2015 at 6:55 AM, Segher Boessenkool seg...@kernel.crashing.org wrote: On Wed, Jul 22, 2015 at 05:10:04AM -0700, H.J. Lu wrote: I got a feedback, suggesting __builtin_stack_top, instead of __builtin_ia32_stack_top. But I don't know if + /* After the prologue, stack top is

Re: [gomp] Move openacc vector worker single handling to RTL

2015-07-22 Thread Thomas Schwinge
Hi Nathan! On Tue, 21 Jul 2015 16:05:05 -0400, Nathan Sidwell nat...@codesourcery.com wrote: On 07/18/15 11:37, Thomas Schwinge wrote: On Thu, 09 Jul 2015 20:25:22 -0400, Nathan Sidwell nat...@acm.org wrote: This is the patch I committed. [...] Prompted by your recent -O0 patch to

RE: [Patch] Add support for IEEE-conformant versions of scalar fmin* and fmax*

2015-07-22 Thread David Sherwood
Hi, Sorry to bother people again. Is this OK to go now? Thanks! David. -Original Message- From: David Sherwood [mailto:david.sherw...@arm.com] Sent: 15 July 2015 11:29 To: 'Joseph Myers' Cc: gcc-patches@gcc.gnu.org Subject: RE: [Patch] Add support for IEEE-conformant versions of scalar

libgomp testing, RUNTESTFLAGS (was: [gomp4] libgomp: Cope with DejaGnu having no mechanism to transfer environment variables to remote boards)

2015-07-22 Thread Thomas Schwinge
Hi! (Cesar, you had the same question.) On Thu, 14 May 2015 11:26:15 +0200, Jakub Jelinek ja...@redhat.com wrote: Talking about the libgomp testsuite, can we rename the libgomp/testsuite/libgomp-oacc-*/*.exp files to something unique? I mean, trying to run say just the OpenMP C tests is

Re: [PR25529] Convert (unsigned t * 2)/2 into unsigned (t 0x7FFFFFFF)

2015-07-22 Thread Richard Biener
On Tue, Jul 21, 2015 at 11:16 AM, Hurugalawadi, Naveen naveen.hurugalaw...@caviumnetworks.com wrote: Hi, For signed types with TYPE_OVERFLOW_UNDEFINED you can simply cancel the operation (even for non-power-of-two multipliers). Thanks for the review and comments. Please find attached the

[PATCH] Add location to genmatch operator

2015-07-22 Thread Richard Biener
This simplifies code by adding a location to each operator. This also fixes bogus locations in the current generated files. Bootstrap running on x86_64-unknown-linux-gnu. Richard. 2015-07-22 Richard Biener rguent...@suse.de * genmatch.c (struct operand): Add location member.

Re: [Patch, fortran] PR 37131, inline matmul

2015-07-22 Thread Mikael Morin
Le 21/07/2015 21:49, Thomas Koenig a écrit : Am 21.07.2015 um 19:26 schrieb Mikael Morin: I would like to avoid the hack in iresolve. So let's reuse the frontend-passes.c part of my patch (set resolved_isym) I would much prefer if that was put into gfc_resolve_fe_runtime_error, next to the

[PATCH] [gomp] Simplify thread pool initialization

2015-07-22 Thread Sebastian Huber
Move the thread pool initialization from the team start to the team creation. This eliminates one conditional expression. In addition this is a preparation patch to enable shared thread pools which I would like to use for RTEMS later. No unexpected failures on x86_64-unknown-linux-gnu.

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

2015-07-22 Thread H.J. Lu
On Tue, Jul 21, 2015 at 2:45 PM, H.J. Lu hongjiu...@intel.com wrote: When __builtin_frame_address is used to retrieve the address of the function stack frame, the frame pointer is always kept, which wastes one register and 2 instructions. For x86-32, one less register means significant

Re: [C/C++ PATCH] Implement -Wshift-overflow (PR c++/55095) (take 3)

2015-07-22 Thread David Edelsohn
On Tue, Jul 21, 2015 at 5:59 AM, Marek Polacek pola...@redhat.com wrote: On Mon, Jul 20, 2015 at 04:23:08PM -0400, David Edelsohn wrote: This seems to have caused a number of new failures in the PPC testsuite for vmx/unpack. Sorry about that. Should be fixed with this patch I'm about to

RE: [PATCH, MIPS] Scheduling for M51xx core family

2015-07-22 Thread Robert Suchanek
Hi Matthew, gcc/ * config/mips/m5100.md: New file. * config/mips/mips-cpus.def (m5100, m5101): Define. * config/mips/mips-tables.opt: Regenerate. * config/mips/mips.c (mips_rtx_cost_data): Add costs for m5100. * config/mips/mips.h (MIPS_ISA_LEVEL_SPEC): Map

Re: [C/C++ PATCH] Implement -Wtautological-compare (PR c++/66555, c/54979)

2015-07-22 Thread Martin Sebor
On 07/22/2015 01:06 PM, Marek Polacek wrote: On Wed, Jul 22, 2015 at 12:43:53PM -0600, Martin Sebor wrote: On 07/14/2015 09:18 AM, Marek Polacek wrote: Code such as if (i == i) is hardly ever desirable, so we should be able to warn about this to prevent dumb mistakes. I haven't tried the

libstdc++: more __intN tweaks

2015-07-22 Thread DJ Delorie
Another place where a list of all types are explicitly listed, and the __intN types need to be included, and elsewhere protection against errors [-Wnarrowing] on targets that have small size_t. Ok? * include/bits/functional_hash.h: Add specializations for __intN types.

[msp430] minor optimizations and tweaks

2015-07-22 Thread DJ Delorie
As indicated. Committed. * config/msp430/t-msp430 (MULTILIB_DIRNAMES): Remove trailing slashes. * config/msp430/msp430.md (ashlhi3): Optimize shifts of subregs. (ashrhi3): Likewise. (lshrhi3): Likewise. (movhi): Take advantage of zero-extend to

Re: [PR25530] Convert (unsigned t / 2) * 2 into (unsigned t ~1)

2015-07-22 Thread Hurugalawadi, Naveen
Your previous patch correctly restricted this to unsigned types. Thanks for your review and comments. Please find attached the modified patch as per your comments. Please let me know if this version is okay? Thanks, Naveen 2015-07-22 Naveen H.S naveen.hurugalaw...@caviumnetworks.com

Re: [PR25529] Convert (unsigned t * 2)/2 into unsigned (t 0x7FFFFFFF)

2015-07-22 Thread Hurugalawadi, Naveen
so using wi::mask is prefered here. Thanks for your review and comments. Please find attached the modified patch as per your comments. Please let me know if this version is okay? Thanks, Naveen 2015-07-22 Naveen H.S naveen.hurugalaw...@caviumnetworks.com gcc/testsuite/ChangeLog:

Re: [C++ Patch] PR 52987

2015-07-22 Thread Jason Merrill
OK. Jason

Re: [gomp4.1] Initial support for some OpenMP 4.1 construct parsing

2015-07-22 Thread Jakub Jelinek
On Mon, Jul 20, 2015 at 08:10:41PM +0200, Jakub Jelinek wrote: And here is untested incremental libgomp side of the proposed GOMP_MAP_FIRSTPRIVATE_POINTER. Actually, that seems unnecessary, for the array section maps we already have there a pointer, so we can easily implement that just on the

Re: PATCH] PR target/65612: Multiversioning doesn't work with DSO nor PIE

2015-07-22 Thread Sriraman Tallam
On Fri, Apr 17, 2015 at 5:36 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Apr 17, 2015 at 4:59 AM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Apr 17, 2015 at 04:48:48AM -0700, H.J. Lu wrote: I don't like it. Nonshared libgcc is libgcc.a, period. No sense in creating yet another

[C++ Patch] PR 52987

2015-07-22 Thread Paolo Carlini
Hi, this bug is purely about error recovery. A while ago I fixed the first half, but for, eg: int foo(x a) { } we still emit the pointless: 52987_2.C:1:14: error: expected ‘,’ or ‘;’ before ‘{’ token In fact, we *already* have code helping error recovery in cp_parser_simple_declaration:

[PATCH, MIPS] Compact branch support for MIPS32R6/MIPS64R6

2015-07-22 Thread Matthew Fortune
A full range of 'compact' branch instructions were introduced to MIPS as part of Release 6. The compact term is used to identify the fact that these do not have a delay slot. http://imgtec.com/mips/architectures/mips64/ The one subtlety of compact branches is that while they do not have a delay

[patch] Include reduction on libackend.a and language source files

2015-07-22 Thread Andrew MacLeod
This is the result of running include reduction on all the files which make up libbackend.a, as well as most of the language files found in subdirectories lto, c ,cp, java, go, fortran, jit, ada. well, some of ada. :-) I looked at the output and hand tweaked a few things... removing

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

2015-07-22 Thread Maxim Blumental
Corrected log and patch. 2015-07-22 17:12 GMT+03:00 Jakub Jelinek ja...@redhat.com: On Wed, Jul 22, 2015 at 04:55:09PM +0300, Maxim Blumental wrote: 2015-07-22 Maxim Blumenthal maxim.blument...@intel.com PR libgomp/66950 * testsuite/libgomp.c/examples-4/simd-7.c: Lower the

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

2015-07-22 Thread Jakub Jelinek
On Wed, Jul 22, 2015 at 05:49:13PM +0300, Maxim Blumental wrote: 2015-07-22 Maxim Blumenthal maxim.blument...@intel.com PR libgomp/66950 * testsuite/libgomp.c/examples-4/simd-7.c(N): Change to 30 from 45. Space before (N): (fib_ref): New function. (fib): Correct

[PATCH] MIPS: Prevent the p5600-bonding.c test from being run for the n32 and 64 ABIs

2015-07-22 Thread Andrew Bennett
Hi, The MIPS p5600-bonding.c test is currently failing for the n32 and n64 ABIs. The test is checking if the load/store bonding patterns correctly match sequences of load/store instructions. There are currently no load/store bonding patterns to match DI mode values. For the n32 and n64 ABIs

[Patch ARM/AArch64 obvious] Fix typo: Rename insn_reservation cortex_53_advsimd to cortex_a53_advsimd

2015-07-22 Thread James Greenhalgh
Hi, As subject. This makes the naming scheme for insn_reservations consistent in config/arm/cortex-a53.md. Checked that we still build a compiler after this cosmetic change, and committed as obvious as revision 226069. Thanks, James 2015-07-22 James Greenhalgh james.greenha...@arm.com

RE: [PATCH] MIPS: Prevent the p5600-bonding.c test from being run for the n32 and 64 ABIs

2015-07-22 Thread Matthew Fortune
Andrew Bennett andrew.benn...@imgtec.com writes: diff --git a/gcc/testsuite/gcc.target/mips/p5600-bonding.c b/gcc/testsuite/gcc.target/mips/p5600-bonding.c index 0890ffa..20c26ca 100644 --- a/gcc/testsuite/gcc.target/mips/p5600-bonding.c +++ b/gcc/testsuite/gcc.target/mips/p5600-bonding.c @@

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

2015-07-22 Thread Thomas Schwinge
Hi Tom! On Wed, 1 Jul 2015 13:16:14 +0200, Tom de Vries tom_devr...@mentor.com wrote: 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

[gomp4] libgomp testsuite: Remove some explicit acc_device_nvidia usage (was: [gomp4, PATCH] Fix libgomp.oacc-c-c++-common/lib-3.c)

2015-07-22 Thread Thomas Schwinge
Hi! On Wed, 22 Jul 2015 16:32:17 +0200, I wrote: On Wed, 1 Jul 2015 13:16:14 +0200, Tom de Vries tom_devr...@mentor.com wrote: testcase libgomp.oacc-c-c++-common/lib-3.c is supposed to fail. libgomp: Resolve XFAIL in libgomp.oacc-c-c++-common/lib-3.c Working on this, I also came up

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

2015-07-22 Thread Jakub Jelinek
On Wed, Jul 22, 2015 at 04:55:09PM +0300, Maxim Blumental wrote: 2015-07-22 Maxim Blumenthal maxim.blument...@intel.com PR libgomp/66950 * testsuite/libgomp.c/examples-4/simd-7.c: Lower the defined constant N to 30. Add iterative reference function for Fibonacci numbers

RE: [PATCH, MIPS] I6400 scheduling

2015-07-22 Thread Robert Suchanek
Hi, diff --git a/gcc/config/mips/i6400.md b/gcc/config/mips/i6400.md new file mode 100644 index 000..101a20c --- /dev/null +++ b/gcc/config/mips/i6400.md @@ -0,0 +1,142 @@ +;; DFA-based pipeline description for I6400. +;; +;; Copyright (C) 2007-2015 Free Software Foundation,

Re: [gomp4.1] calculate pointer offsets for depend(sink)

2015-07-22 Thread Jakub Jelinek
On Tue, Jul 21, 2015 at 03:53:47PM -0700, Aldy Hernandez wrote: commit 61b2d11dfa8083014b385fc6ec6564fc18c41c72 Author: Aldy Hernandez al...@redhat.com Date: Tue Jul 21 08:02:39 2015 -0700 * tree-pretty-print.c (dump_omp_clause): Pass TYPE_SIGN to wi::neg_p. c/ *

[gomp4] libgomp: Additional acc_shutdown bug fixing and testing (was: [gomp4, PATCH] Fix libgomp.oacc-c-c++-common/lib-3.c)

2015-07-22 Thread Thomas Schwinge
Hi! On Wed, 22 Jul 2015 16:32:17 +0200, I wrote: On Wed, 1 Jul 2015 13:16:14 +0200, Tom de Vries tom_devr...@mentor.com wrote: testcase libgomp.oacc-c-c++-common/lib-3.c is supposed to fail. --- libgomp/testsuite/libgomp.oacc-c-c++-common/lib-3.c +++