Re: [C++ Patch] PR 52987

OK. Jason

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

>> 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 gcc/testsuite/ChangeLog: PR middl

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

>> 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 gcc/testsuite/ChangeLog: PR middle-end/25529 * gcc.dg

[msp430] minor optimizations and tweaks

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 lo

libstdc++: more __intN tweaks

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.

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

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 comments

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

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 s

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

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

[C++ Patch] PR 52987

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:

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

On Fri, Apr 17, 2015 at 5:36 AM, H.J. Lu wrote: > On Fri, Apr 17, 2015 at 4:59 AM, Jakub Jelinek 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 library for that. >>> > So, IMH

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

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 ab

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

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 pa

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

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

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

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.

[PATCH] Remove unused get_current_pass_name

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 x86-64-

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

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

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

Straightforward implementation. libgcc/ChangeLog: 2015-07-22 Uros Bizjak 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 PR target/66954 * config/i

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

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]

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

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 writes: > > > + /* Don't warn for e.g. > > > + HOST_WIDE_INT n; > > > + ... > > > + if (n == (long) n) ... > > > + */ > > > + if ((C

Re: [PR64164] drop copyrename, integrate into expand

On Jul 21, 2015, Richard Biener wrote: > On Sat, Jul 18, 2015 at 9:37 AM, Alexandre Oliva 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 this test for else? It turns out it'

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

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 w

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

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 i

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

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 dec

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

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 PR target/

Re: [PR64164] drop copyrename, integrate into expand

On Jul 21, 2015, Richard Biener wrote: > On Sat, Jul 18, 2015 at 9:37 AM, Alexandre Oliva wrote: >> On Jul 16, 2015, Alexandre Oliva wrote: >> + /* If we are assigning parameters for a function, rather >> +than for a call, propagate the RTL of the complex parm to >> +

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

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 the

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

[ 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

[PATCH] Fix default_binds_local_p_2 for extern protected data

The commit https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=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, fals

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

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 testsuit

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

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

Re: [WIP] OpenMP 4 NVPTX support

On Wed, Jul 22, 2015 at 06:04:20PM +0200, Thomas Schwinge wrote: > On Tue, 21 Apr 2015 17:58:39 +0200, Jakub Jelinek 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 c

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

On Fri, Jun 26, 2015 at 02:45:39PM +0100, Jiong Wang wrote: > > Marcus Shawcroft writes: > > 2015-06-26 Jiong Wang > > wwwdocs/ > * htdocs/gcc-6/changes.html (AArch64): Document -fpic for small model. > > Index: gcc-6/changes.html >

[PATCH] Check TYPE_OVERFLOW_WRAPS for parloops reductions

[ 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 wrote: On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Pop wrote: Tom de Vries wrote: Fix reduction safety checks diff --git a/gcc/tree-vect-loop.c b

[PATH PR66926,PR66951} simple fix for ICE.

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 PR tree-optim

Re: [WIP] OpenMP 4 NVPTX support

Hi! On Tue, 21 Apr 2015 17:58:39 +0200, Jakub Jelinek 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 weeks of work > a

[PATCH] Enable reductions without fassociative-math in graphite

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: [AArch64/wwwdoc] Document -fpic support for small memory model

Jiong Wang writes: > Marcus Shawcroft writes: > >> On 26 June 2015 at 10:32, Jiong Wang 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 t

[PATCH] Don't allow unsafe reductions in graphite

[ 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 wrote: >On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Pop wrote: >>Tom de Vries wrote: >>>Fix reduction safety checks >>> >>> * graphite-sese-to

PR middle-end/16351 NULL dereference warnings

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 th

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

On Wed, Jul 22, 2015 at 6:59 AM, H.J. Lu wrote: > On Wed, Jul 22, 2015 at 6:55 AM, Segher Boessenkool > 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 >>> >>> +

PR c/16351 Extend Wnonnull for returns_nonnull

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

Some additional comments for nvptx.c

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 2259

[PATCH] Document ftrapv/fwrapv interaction

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

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

On Wed, Jul 22, 2015 at 05:49:13PM +0300, Maxim Blumental wrote: > 2015-07-22 Maxim Blumenthal > > 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 corner cases in the r

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

Corrected log and patch. 2015-07-22 17:12 GMT+03:00 Jakub Jelinek : > On Wed, Jul 22, 2015 at 04:55:09PM +0300, Maxim Blumental wrote: >> 2015-07-22 Maxim Blumenthal >> >> PR libgomp/66950 >> * testsuite/libgomp.c/examples-4/simd-7.c: Lower the defined constant >> N to 30. A

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

Hi! On Wed, 22 Jul 2015 16:32:17 +0200, I wrote: > On Wed, 1 Jul 2015 13:16:14 +0200, Tom de Vries > 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 with the following c

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

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

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

On Tue, Jul 21, 2015 at 03:53:47PM -0700, Aldy Hernandez wrote: > commit 61b2d11dfa8083014b385fc6ec6564fc18c41c72 > Author: Aldy Hernandez > Date: Tue Jul 21 08:02:39 2015 -0700 > > * tree-pretty-print.c (dump_omp_clause): Pass TYPE_SIGN to > wi::neg_p. > c/ > * c-typeck.c

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

Hi Tom! On Wed, 1 Jul 2015 13:16:14 +0200, Tom de Vries 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 initialized, if

RE: [PATCH, MIPS] I6400 scheduling

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 Softwar

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

Andrew Bennett 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 > @@ -1,6 +1,7 @@ > /*

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

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 * config/arm/cortex-a

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

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

Re: Tests for libgomp based on OpenMP Examples 4.0.2.

On Wed, Jul 22, 2015 at 04:55:09PM +0300, Maxim Blumental wrote: > 2015-07-22 Maxim Blumenthal > > 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 > - fib_ref. In

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

On Wed, Jul 22, 2015 at 6:55 AM, Segher Boessenkool 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 at -WORD(AP) in t

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

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

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

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: Tests for libgomp based on OpenMP Examples 4.0.2.

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 t

[PATCH] S/390: Improve risbg usage

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 Krebb

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

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

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

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

[PATCH] [gomp] Simplify thread pool initialization

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. libgomp/

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

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: [PATCH, MIPS] Scheduling for M51xx core family

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_SPE

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

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 wrote: > > > On Mon, Jul 20, 2015 at 04:23:08PM -0400, David Edelsohn wrote: > > >> This seems to have caused a number

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

On Wed, Jul 22, 2015 at 08:35:12AM -0400, David Edelsohn wrote: > On Tue, Jul 21, 2015 at 5:59 AM, Marek Polacek 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. > > > > Sorr

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

On Tue, Jul 21, 2015 at 5:59 AM, Marek Polacek 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 commit. > > 2015

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

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 as

[PATCH] Add location to genmatch operator

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 * genmatch.c (struct operand): Add location member. (predicate, expr

Re: RFC: [PATCH] Add __builtin_ia32_stack_top

On Tue, Jul 21, 2015 at 2:45 PM, H.J. Lu 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 negative impact on perf

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

On Tue, Jul 21, 2015 at 11:16 AM, Hurugalawadi, Naveen 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. (*) > Thanks for the revie

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

On Tue, Jul 21, 2015 at 11:16 AM, Hurugalawadi, Naveen 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 modified patch as per your c

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

The 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 * config/

[PATCH] Fix canonical form of true/false conditions

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

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

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 ju

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

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,

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

On Tue, Jul 21, 2015 at 11:15 AM, Hurugalawadi, Naveen 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&(-1>>N) ==/!= 0. > a * N ==/!= 0 where N is a power of 2 to a & (-1< log2 of N. >

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

On Wed, Jul 22, 2015 at 2:40 AM, Andrew Pinski wrote: > On Tue, Jul 21, 2015 at 12:16 PM, Richard Biener > wrote: >> On July 21, 2015 11:38:31 AM GMT+02:00, Jakub Jelinek >> wrote: >>>On Tue, Jul 21, 2015 at 09:15:31AM +, Hurugalawadi, Naveen wrote: Please find attached the patch which

Re: [RFC, PR66873] Use graphite for parloops

On Wed, Jul 22, 2015 at 1:01 PM, Richard Biener wrote: > On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Pop 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.

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

On 17 July 2015 at 09:32, James Greenhalgh 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 compilation > pipeline. The pr

Re: [RFC, PR66873] Use graphite for parloops

On Tue, Jul 21, 2015 at 8:42 PM, Sebastian Pop 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 TYPE_OVERFLOW_WRAPS for

[PATCH] [AArch64] fix typo in vec_store_lanesoi_lane

Committed as obvious r226061. gcc/ChangeLog: 2015-07-22 Charles Baylis * config/aarch64/aarch64-simd.md (vec_store_lanesoi_lane): Fix typo in attribute. From 7d98f7fc82cfc3012b460e4f4f91200fedcb04db Mon Sep 17 00:00:00 2001 From: Charles Baylis Date: Tue, 21 Jul 2015 16:54:32

Re: [AArch64] PR63521 Define REG_ALLOC_ORDER/HONOR_REG_ALLOC_ORDER

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 ca

[PATCH] Fix match with result operands and conditions

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 ge

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

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

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

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 > > * config/aarch64/aarch64.c (aarch64_option_valid_

[PATCH] Fix PR66945

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 propaga

[PATCH] Fix PR66952

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 in

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

Hi! (Cesar, you had the same question.) On Thu, 14 May 2015 11:26:15 +0200, Jakub Jelinek 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 impossible since >

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

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,

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

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

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

Hi Nathan! On Tue, 21 Jul 2015 16:05:05 -0400, Nathan Sidwell wrote: > On 07/18/15 11:37, Thomas Schwinge wrote: > > On Thu, 09 Jul 2015 20:25:22 -0400, Nathan Sidwell wrote: > >> This is the patch I committed. [...] > > > > Prompted by your recent "-O0 patch" to »[f]ix PTX worker spill/fill«,

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

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

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