[testsuite]MIPS remove duplicate div-x test

2017-06-01 Thread Paul Hua
Hi, There are duplicate testcase in gcc.target/mips dir. div-5.c same as div-9.c. div-6.c same as div-10.c. div-7.c same as div-11.c. div-8.c same as div-12.c. Is this deliberate? Otherwise, the attached patch fixing this. Paul. ***ChangeLog*** 2017-06-01Chenghua Xu Remove duplica

Re: [PATCH v2, rs6000] Fold vector absolutes in GIMPLE

2017-06-01 Thread Richard Biener
On Wed, May 31, 2017 at 9:38 PM, Will Schmidt wrote: > Hi, > > Add support for early expansion of vector absolute built-ins. > > [V2] Per reviews and feedback, skip the early folding for > integral types based on a check against TYPE_OVERFLOW_WRAPS(arg0). > > Added test variants to exercise the -f

Re: [PATCH, rs6000] Fold vector shifts in GIMPLE

2017-06-01 Thread Richard Biener
On Wed, May 31, 2017 at 10:01 PM, Will Schmidt wrote: > Hi, > > Add support for early expansion of vector shifts. Including > vec_sl (shift left), vec_sr (shift right), vec_sra (shift > right algebraic), vec_rl (rotate left). > Part of this includes adding the vector shift right instructions to >

Re: [PATCH v2] Implement no_sanitize function attribute

2017-06-01 Thread Bernhard Reutner-Fischer
On 31 May 2017 14:25:09 CEST, "Martin Liška" wrote: >I've got written that on my TODO list. Will work on that some time in >the stage1. BTW.. May I ask you to put it below https://gcc.gnu.org/PR65534 (the tailcall resp. IPA-ICF thing :-) Many TIA and cheers,

Re: Handle unpropagated assignments in SLP

2017-06-01 Thread Richard Biener
On Thu, Jun 1, 2017 at 8:45 AM, Richard Sandiford wrote: > Some of the SVE patches extend SLP to predicated operations created by > ifcvt. However, ifcvt currently forces the mask into a temporary: > > mask = ifc_temp_var (TREE_TYPE (mask), mask, &gsi); > > and at the moment SLP d

Re: [PATCH, rs6000] Fold vector shifts in GIMPLE

2017-06-01 Thread Jakub Jelinek
On Thu, Jun 01, 2017 at 09:48:37AM +0200, Richard Biener wrote: > On Wed, May 31, 2017 at 10:01 PM, Will Schmidt > wrote: > > Hi, > > > > Add support for early expansion of vector shifts. Including > > vec_sl (shift left), vec_sr (shift right), vec_sra (shift > > right algebraic), vec_rl (rotate

RE: [PATCH 6/7] [ARC] Prevent moving stores to the frame before the stack adjustment.

2017-06-01 Thread Claudiu Zissulescu
> Given the description the code looks fine. It would be nice to see > more of a _why_ in the commit message. I'm guessing this is either > something related to signal handling, or debugging... I don't see why > this would be needed for functional correctness. > The issue is how we generate a f

Re: [PATCH] DWARF: for variants, produce unsigned discr. when debug type is unsigned

2017-06-01 Thread Pierre-Marie de Rodat
On 05/31/2017 10:34 PM, Jason Merrill wrote: OK. Jason Committed. Thank you! -- Pierre-Marie de Rodat

Support $SYSROOT for = in -I etc.

2017-06-01 Thread Rainer Orth
GNU ld recently gained support for a $SYSROOT token in -L to denote the sysroot prefix (similar to $ORIGIN in rpath), to be more intuitive than the current use of '=' for that purpose: PR ld/21251 Support $SYSROOT in ld -L and INPUT command https://sourceware.org/bugzilla/s

[OBVIOUS][SPARC] Fix a couple of insn alternatives missing a type attribute

2017-06-01 Thread Jose E. Marchesi
Installed the patch below to trunk as obvious. It fixes a couple of insn alternatives missing a type attribute in sparc.md. Salud! Index: gcc/ChangeLog === --- gcc/ChangeLog (revision 248773) +++ gcc/ChangeLog (working

Re: [gcn][patch] Add -mgpu option and plumb in assembler/linker

2017-06-01 Thread Thomas Schwinge
Hi! Sorry for the late reply. On Fri, 28 Apr 2017 18:06:39 +0100, Andrew Stubbs wrote: > 3. Add -mgpu option and corresponding --with-gpu. I've deliberately used > "gpu" instead of "cpu" because I want offloading compilers to be able to > say "-mcpu=foo -foffload=-mgpu=bar", or even have the h

[testsuite, committed] Require label_values for some test-cases

2017-06-01 Thread Tom de Vries
Hi, this patch adds dg-require-effective-target label_values for some test-cases. Committed as obvious. Thanks, - Tom Require label_values for some test-cases 2017-06-01 Tom de Vries * c-c++-common/pr43395.c: Add dg-require-effective-target label_values. * gcc.c-torture/compile/asmgoto

[PATCH] DWARF: add DW_AT_location for global decls with DECL_VALUE_EXPR

2017-06-01 Thread Pierre-Marie de Rodat
Hi, In GNAT, we materialize renamings that cannot be described in standard DWARF as synthetic variables that describe how to fetch the renamed object. Look for "___XR" in gcc/ada/exp_dbug.ads for more details about this convention. In order to have a location for these variables in the debug inf

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

2017-06-01 Thread Dominique d'Humières
> Le 31 mai 2017 à 21:03, Nicolas Koenig a écrit : > > Hello Dominique, > > attached is the next try, this time without stupidities (I hope). Both test > cases you posted don't ICE anymore. > > Ok for trunk? > > Nicolas > Preliminary tests look OK, full testing in progress. Thanks, Domin

RE: [PATCH 6/7] [ARC] Prevent moving stores to the frame before the stack adjustment.

2017-06-01 Thread Claudiu Zissulescu
> Given the description the code looks fine. It would be nice to see > more of a _why_ in the commit message. I'm guessing this is either > something related to signal handling, or debugging... I don't see why > this would be needed for functional correctness. Committed with additional comment

RE: [PATCH 7/7] [ARC] Test against frame_pointer_needed in arc_can_eliminate.

2017-06-01 Thread Claudiu Zissulescu
> Looks good, > Committed, thank you, Claudiu

RE: [PATCH 5/7] [ARC] Update (non)commutative_binary_comparison patterns.

2017-06-01 Thread Claudiu Zissulescu
> Looks good, thanks, > Andrew Committed, thank you, Claudiu

RE: [PATCH 4/7] [ARC] Change predicate movv2hi to avoid scaled addresses.

2017-06-01 Thread Claudiu Zissulescu
> Seems reasonable. > Committed, thank you, Claudiu

RE: [PATCH 3/7] [ARC] Allow r30 to be used by the reg-alloc.

2017-06-01 Thread Claudiu Zissulescu
> Looks good, thanks, > Committed, thank you, Claudiu

RE: [PATCH 2/7] [ARC] Avoid use of hard registers before reg-alloc.

2017-06-01 Thread Claudiu Zissulescu
> Looks good, thanks, Committed, thank you, Claudiu

RE: [PATCH 1/7] [ARC] Make mulsi for A700 pattern commutative.

2017-06-01 Thread Claudiu Zissulescu
> Looks good thanks, Committed, thank you, Claudiu

Re: [PATCH, GCC/ARM/gcc-7-branch] Backport PR71607

2017-06-01 Thread Prakhar Bahuguna
On 01/06/2017 07:15:47, Richard Sandiford wrote: > Prakhar Bahuguna writes: > > On 31/05/2017 14:11:43, Richard Sandiford wrote: > >> Prakhar Bahuguna writes: > >> > On 31/05/2017 09:19:40, Richard Sandiford wrote: > >> >> const_ints are supposed to be stored in sign-extended form, so a 32-bit >

[Ada] Fix PR ada/80921

2017-06-01 Thread Eric Botcazou
We have apparently never tried to build shared libraries in cross builds. Tested on x86_64-suse-linux, applied on mainline and 7 & 6 branches. 2017-06-01 Eric Botcazou PR ada/80921 * configure.ac (default_gnatlib_target): Remove bogus condition. (have_getipinfo): Twea

[Committed] S/390: Don't fetch the return address early with ooo

2017-06-01 Thread Andreas Krebbel
We used to load the return address slot some time in advance. This helped on older machines to resolve the data dependencies in time. However, it is pointless on out of order CPUs. Disabled with that patch. Regression tested on s390x. Committed to mainline. Bye, -Andreas- gcc/ChangeLog: 2017

[PR 80898] Propagate grp_write from disqualified SRA candidates

2017-06-01 Thread Martin Jambor
Hi, when I wrote the lazy setting of grp_write flag early next year, I made a mistake when thinking about what to do about SRA candidates that were disqualified but form a RHS of an assignment link which was to be used to set grp_write of the LHS when appropriate. The code expects that the RHS ac

Re: [PR 80898] Propagate grp_write from disqualified SRA candidates

2017-06-01 Thread Richard Biener
On Thu, 1 Jun 2017, Martin Jambor wrote: > Hi, > > when I wrote the lazy setting of grp_write flag early next year, I > made a mistake when thinking about what to do about SRA candidates > that were disqualified but form a RHS of an assignment link which was > to be used to set grp_write of the L

Re: Default std::vector default and move constructor

2017-06-01 Thread Jonathan Wakely
On 31/05/17 22:28 +0200, François Dumont wrote: Unless I made a mistake it revealed that restoring explicit call to _Bit_alloc_type() in default constructor was not enough. G++ doesn't transform it into a value-init if needed. I don't know if it is a compiler bug but I had to do just like prese

Re: [v3] Fix cross compilation to Solaris

2017-06-01 Thread Jonathan Wakely
On 30/05/17 15:10 +0200, Rainer Orth wrote: I recently tried a cross-build from sparc-sun-solaris2.12 to i386-pc-solaris2.12 (with cross-binutils and gas, but the native ld which has been a cross-linker for quite some time). The build failed in libstdc++-v3 like this: /vol/gcc/src/hg/trunk/loca

[PATCH 2/7] [ARC] Define ADDITIONAL_REGISTER_NAMES.

2017-06-01 Thread Claudiu Zissulescu
This macro is needed to be used with -ffixed- option, and inline asm. gcc/ 2017-01-09 Claudiu Zissulescu * config/arc/arc.h (ADDITIONAL_REGISTER_NAMES): Define. --- gcc/config/arc/arc.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/gcc/config/arc/arc.h b/gcc/config/arc/ar

[PATCH 0/7] [ARC] Bug fixing, add support for naked functions

2017-06-01 Thread Claudiu Zissulescu
From: claziss Hi, The first patch adds support for 'naked' functions to ARC compiler, a number of tests are also added. The second patch defines additional register names aliases to be handled by inline asm code or -ffixed register commnds. The following two patches are in the context of impro

[PATCH 1/7] [ARC] Add support for naked functions.

2017-06-01 Thread Claudiu Zissulescu
gcc/ 2016-12-13 Claudiu Zissulescu Andrew Burgess * config/arc/arc-protos.h (arc_compute_function_type): Change prototype. (arc_return_address_register): New function. * config/arc/arc.c (arc_handle_fndecl_attribute): New function. (arc_handle_fndec

[PATCH 3/7] [ARC] [LRA] Fix tests asm constraints.

2017-06-01 Thread Claudiu Zissulescu
LRA doesn't like the 'X' constraint as used in our tests, remove it. gcc/testsuite 2017-01-09 Claudiu Zissulescu * gcc.target/arc/mulsi3_highpart-1.c: Remove 'X' constraint. * gcc.target/arc/mulsi3_highpart-2.c: Likewise. --- gcc/testsuite/gcc.target/arc/mulsi3_highpart-1.c |

[PATCH 4/7] [ARC] [LRA] Avoid emitting COND_EXEC during expand.

2017-06-01 Thread Claudiu Zissulescu
Emmitting COND_EXEC rtxes during expand does not always work. gcc/ 2017-01-10 Claudiu Zissulescu * config/arc/arc.md (clzsi2): Expand to an arc_clzsi2 instruction that also clobbers the CC register. The old expand code is moved to ... (*arc_clzsi2): ... here.

[PATCH 6/7] [ARC] Deprecate mexpand-adddi option.

2017-06-01 Thread Claudiu Zissulescu
From: claziss Emitting subregs in the expand is not a good idea. Deprecate this option. gcc/ 2017-04-26 Claudiu Zissulescu * config/arc/arc.md (adddi3): Remove support for mexpand-adddi option. (subdi3): Likewise. * config/arc/arc.opt (mexpand-adddi): Deprecat

[PATCH 5/7] [ARC] Enable indexed loads for elf targers.

2017-06-01 Thread Claudiu Zissulescu
gcc/ 2017-02-28 Claudiu Zissulescu * config/arc/arc.opt (mindexed-loads): Use initial value TARGET_INDEXED_LOADS_DEFAULT. (mauto-modify-reg): Use initial value TARGET_AUTO_MODIFY_REG_DEFAULT. * config/arc/elf.h (TARGET_INDEXED_LOADS_DEFAULT): Define.

[PATCH 7/7] [ARC] Consolidate PIC implementation.

2017-06-01 Thread Claudiu Zissulescu
This patch refactors a number of functions and compiler hooks into using a single function which checks if a rtx is suited for pic or not. Removed functions are arc_legitimate_pc_offset_p and arc_legitimate_pic_operand_p beeing replaced by calls to arc_legitimate_pic_addr_p. Thus we have an unitary

Re: [PATCH] DWARF: add DW_AT_location for global decls with DECL_VALUE_EXPR

2017-06-01 Thread Richard Biener
On Thu, Jun 1, 2017 at 11:28 AM, Pierre-Marie de Rodat wrote: > Hi, > > In GNAT, we materialize renamings that cannot be described in standard > DWARF as synthetic variables that describe how to fetch the renamed > object. Look for "___XR" in gcc/ada/exp_dbug.ads for more details about > this con

Re: [PATCH][AArch64] Allow const0_rtx operand for atomic compare-exchange patterns

2017-06-01 Thread Kyrill Tkachov
Ping. Thanks, Kyrill On 08/05/17 11:59, Kyrill Tkachov wrote: Ping. Thanks, Kyrill On 24/04/17 10:37, Kyrill Tkachov wrote: Pinging this back into context so that I don't forget about it... https://gcc.gnu.org/ml/gcc-patches/2017-02/msg01648.html Thanks, Kyrill On 28/02/17 12:29, Kyrill T

Re: [PATCH] DWARF: add DW_AT_location for global decls with DECL_VALUE_EXPR

2017-06-01 Thread Pierre-Marie de Rodat
On 06/01/2017 03:53 PM, Richard Biener wrote: Ok. Richard. Committed. Thank you, Richard! -- Pierre-Marie de Rodat

Re: [PATCH,DWARF,v2] AIX dwarf2out label fix PING

2017-06-01 Thread David Edelsohn
Ping https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01440.html Thanks, David

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

2017-06-01 Thread Dominique d'Humières
> Le 1 juin 2017 à 11:30, Dominique d'Humières a écrit : > > >> Le 31 mai 2017 à 21:03, Nicolas Koenig a écrit : >> >> Hello Dominique, >> >> attached is the next try, this time without stupidities (I hope). Both test >> cases you posted don't ICE anymore. >> >> Ok for trunk? >> >> Nicola

Re: [v3] Fix cross compilation to Solaris

2017-06-01 Thread Rainer Orth
Hi Jonathan, >>This way, config.h is identical between a native build and the cross >>above, with the exception of HAVE_SETENV which is equally guarded with >>GLIBCXX_IS_NATIVE in acinclude.m4 (GLIBCXX_CONFIGURE_TESTSUITE). Maybe >>it's time to somehow refine the GLIBCXX_IS_NATIVE check to allow

Re: [Patch, fortran] PR35339 Optimize implied do loops in io statements

2017-06-01 Thread Dominique d'Humières
> Le 1 juin 2017 à 16:19, Dominique d'Humières a écrit : > > I see > > FAIL: gfortran.dg/deferred_character_2.f90 -O1 execution test > FAIL: gfortran.dg/deferred_character_2.f90 -O2 execution test > FAIL: gfortran.dg/deferred_character_2.f90 -O3 -fomit-frame-pointer > -funroll-loops -f

Re: [Patch] Forward triviality in variant

2017-06-01 Thread Jonathan Wakely
On 30/05/17 02:16 -0700, Tim Shen via libstdc++ wrote: diff --git a/libstdc++-v3/include/std/variant b/libstdc++-v3/include/std/variant index b9824a5182c..f81b815af09 100644 --- a/libstdc++-v3/include/std/variant +++ b/libstdc++-v3/include/std/variant @@ -290,6 +290,53 @@ _GLIBCXX_BEGIN_NAMESPACE

Re: [PATCH, rs6000] Fold vector shifts in GIMPLE

2017-06-01 Thread Bill Schmidt
> On Jun 1, 2017, at 2:48 AM, Richard Biener wrote: > > On Wed, May 31, 2017 at 10:01 PM, Will Schmidt > wrote: >> Hi, >> >> Add support for early expansion of vector shifts. Including >> vec_sl (shift left), vec_sr (shift right), vec_sra (shift >> right algebraic), vec_rl (rotate left). >> P

Re: [Patch] Forward triviality in variant

2017-06-01 Thread Ville Voutilainen
On 1 June 2017 at 18:13, Jonathan Wakely wrote: > On 30/05/17 02:16 -0700, Tim Shen via libstdc++ wrote: >> >> diff --git a/libstdc++-v3/include/std/variant >> b/libstdc++-v3/include/std/variant >> index b9824a5182c..f81b815af09 100644 >> --- a/libstdc++-v3/include/std/variant >> +++ b/libstdc++-v

Re: [Patch] Forward triviality in variant

2017-06-01 Thread Jonathan Wakely
On 01/06/17 18:21 +0300, Ville Voutilainen wrote: On 1 June 2017 at 18:13, Jonathan Wakely wrote: On 30/05/17 02:16 -0700, Tim Shen via libstdc++ wrote: diff --git a/libstdc++-v3/include/std/variant b/libstdc++-v3/include/std/variant index b9824a5182c..f81b815af09 100644 --- a/libstdc++-v3/in

Re: [Patch] Forward triviality in variant

2017-06-01 Thread Ville Voutilainen
On 1 June 2017 at 18:29, Jonathan Wakely wrote: >> They all seem to be shortcuts for something::value, so it seems to me >> logical to have >> them all be _v. > > > The _v suffixes in the standard are there to distinguish std::foo from > std::foo_v, but we don't have that problem. Wouldn't necess

[PING**3] [PATCH] Force use of absolute path names for gcov

2017-06-01 Thread Bernd Edlinger
Ping... On 05/12/17 18:47, Bernd Edlinger wrote: > Ping... > > On 04/28/17 19:41, Bernd Edlinger wrote: >> Ping... >> >> I attached a rebased patch file, with the doc changes and >> merge conflicts with trunk of today fixed, but otherwise >> identical. >> >> >> Thanks >> Bernd. >> >> On 04/21/17

[PING**2][PATCH][ PR rtl-optimization/79286] Drop may_trap_p exception to testing dominance in update_equiv_regs

2017-06-01 Thread Bernd Edlinger
Ping... On 05/12/17 18:48, Bernd Edlinger wrote: > Ping... > > On 04/29/17 09:06, Bernd Edlinger wrote: >> On 04/28/17 20:46, Jeff Law wrote: >>> On 04/28/2017 11:27 AM, Bernd Edlinger wrote: > Yes I agree, that is probably not worth it. So I could try to remove the special ha

[PING**4] [PATCH, ARM] correctly encode the CC reg data flow

2017-06-01 Thread Bernd Edlinger
Ping... On 05/12/17 18:49, Bernd Edlinger wrote: > Ping... > > On 04/29/17 19:21, Bernd Edlinger wrote: >> Ping... >> >> On 04/20/17 20:11, Bernd Edlinger wrote: >>> Ping... >>> >>> for this patch: >>> https://gcc.gnu.org/ml/gcc-patches/2017-01/msg01351.html >>> >>> On 01/18/17 16:36, Bernd Edlin

[PING**4] [PATCH, ARM] Further improve stack usage on sha512 (PR 77308)

2017-06-01 Thread Bernd Edlinger
Ping... On 05/12/17 18:49, Bernd Edlinger wrote: > Ping... > > On 04/29/17 19:45, Bernd Edlinger wrote: >> Ping... >> >> I attached a rebased version since there was a merge conflict in >> the xordi3 pattern, otherwise the patch is still identical. >> It splits adddi3, subdi3, anddi3, iordi3, xor

[PING**3] [PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308)

2017-06-01 Thread Bernd Edlinger
Ping... On 05/12/17 18:50, Bernd Edlinger wrote: > Ping... > > On 04/29/17 19:52, Bernd Edlinger wrote: >> Ping... >> >> I attached the latest version of my patch. >> >> >> Thanks >> Bernd. >> >> On 12/18/16 14:14, Bernd Edlinger wrote: >>> Hi, >>> >>> this splits the *arm_negdi2, *arm_cmpdi_insn

[arm-embedded] [PATCH, GCC, ARM/embedded-6/7-branch] Set mode for success result of atomic compare and swap

2017-06-01 Thread Thomas Preudhomme
Hi, We have decided to apply the following patch to the embedded-6-branch and embedded-7-branch to fix a genrecog warning when processing sync.md. ChangeLog entry is as follows: 2017-05-03 Thomas Preud'homme Backport from mainline 2017-05-03 Thomas Preud'homme gcc/ * c

[PING**2] [PATCH] Implement a warning for bogus sizeof(pointer) / sizeof(pointer[0])

2017-06-01 Thread Bernd Edlinger
Ping... On 05/12/17 18:55, Bernd Edlinger wrote: > Ping for the C changes. > > Thanks > Bernd. > > On 05/03/17 15:14, Jason Merrill wrote: >> On Tue, May 2, 2017 at 9:26 AM, Bernd Edlinger >> wrote: >>> On 05/01/17 17:54, Jason Merrill wrote: On Fri, Apr 28, 2017 at 1:05 PM, Bernd Edlinger

Re: [Patch] Forward triviality in variant

2017-06-01 Thread Jonathan Wakely
On 01/06/17 18:43 +0300, Ville Voutilainen wrote: On 1 June 2017 at 18:29, Jonathan Wakely wrote: They all seem to be shortcuts for something::value, so it seems to me logical to have them all be _v. The _v suffixes in the standard are there to distinguish std::foo from std::foo_v, but we do

[PATCH, GCC, ARM/embedded-6-branch] Add mode to probe_stack set operands

2017-06-01 Thread Thomas Preudhomme
Hi, We have decided to apply the following patch to the embedded-6-branch to fix a genrecog warning when processing arm.md. 2017-06-01 Thomas Preud'homme Backport from gcc-7-branch 2016-05-09 Kyrylo Tkachov * config/arm/arm.md (probe_stack): Add modes to set source and

[PING][PATCH] [i386] Recompute the frame layout less often

2017-06-01 Thread Bernd Edlinger
Ping... the latest version of this patch was posted here: https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01758.html Thanks Bernd. On 05/23/17 16:31, Bernd Edlinger wrote: > Hi, > > this is the latest version of my patch. > > As already said, it attempts to compute > the frame layout only when

Re: [Patch] Forward triviality in variant

2017-06-01 Thread Ville Voutilainen
On 1 June 2017 at 19:03, Jonathan Wakely wrote: > On 01/06/17 18:43 +0300, Ville Voutilainen wrote: >> >> On 1 June 2017 at 18:29, Jonathan Wakely wrote: They all seem to be shortcuts for something::value, so it seems to me logical to have them all be _v. >>> The _v suffixes i

Re: [Patch] Forward triviality in variant

2017-06-01 Thread Jonathan Wakely
On 01/06/17 19:07 +0300, Ville Voutilainen wrote: On 1 June 2017 at 19:03, Jonathan Wakely wrote: On 01/06/17 18:43 +0300, Ville Voutilainen wrote: On 1 June 2017 at 18:29, Jonathan Wakely wrote: They all seem to be shortcuts for something::value, so it seems to me logical to have them all

C/C++ PATCH to implement -Wmultiline-expansion (PR c/80116)

2017-06-01 Thread Marek Polacek
A motivating example for this warning can be found e.g. in PRE10-C. Wrap multistatement macros in a do-while loop https://www.securecoding.cert.org/confluence/x/jgL7 i.e., #define SWAP(x, y) \ tmp = x; \ x = y; \ y = tmp used like this [1] int x, y, z, tmp; if (z == 0) SWAP(x, y);

Re: [PATCH v2, rs6000] Fold vector absolutes in GIMPLE

2017-06-01 Thread Segher Boessenkool
On Wed, May 31, 2017 at 02:38:15PM -0500, Will Schmidt wrote: > Add support for early expansion of vector absolute built-ins. > > [V2] Per reviews and feedback, skip the early folding for > integral types based on a check against TYPE_OVERFLOW_WRAPS(arg0). > > Added test variants to exercise the

Re: [PATCH, rs6000] fold vector min/max in GIMPLE

2017-06-01 Thread Segher Boessenkool
On Wed, May 31, 2017 at 03:00:15PM -0500, Will Schmidt wrote: > OK for trunk? Looks good, please commit. Segher

Re: [PATCH] Fix PR80721

2017-06-01 Thread Jonathan Wakely
On 12/05/17 12:10 +0200, Richard Biener wrote: It was pointed out by Markus that the EH emergency pool is not kept sorted and fully merged properly for the cases of freeing an entry before the first free entry and for the cases where merging with the immediate successor and for the case with mer

RE: [PING][PATCH][Aarch64] Add support for overflow add and sub operations

2017-06-01 Thread Michael Collison
Ping. Testsuite issue resolved. Okay for trunk? -Original Message- From: Christophe Lyon [mailto:christophe.l...@linaro.org] Sent: Friday, May 19, 2017 3:59 AM To: Michael Collison Cc: gcc-patches@gcc.gnu.org; nd Subject: Re: [PATCH][Aarch64] Add support for overflow add and sub operati

Re: [PING**3] [PATCH] Force use of absolute path names for gcov

2017-06-01 Thread Nathan Sidwell
On 06/01/2017 11:59 AM, Bernd Edlinger wrote: Ping... What are you asking to be reviewed by who? nathan -- Nathan Sidwell

Re: C/C++ PATCH to implement -Wmultiline-expansion (PR c/80116)

2017-06-01 Thread David Malcolm
On Thu, 2017-06-01 at 18:45 +0200, Marek Polacek wrote: > A motivating example for this warning can be found e.g. in > > PRE10-C. Wrap multistatement macros in a do-while loop > https://www.securecoding.cert.org/confluence/x/jgL7 > > i.e., > > #define SWAP(x, y) \ > tmp = x; \ > x = y;

Re: [PATCH] [i386] Recompute the frame layout less often

2017-06-01 Thread Uros Bizjak
On Tue, May 23, 2017 at 4:31 PM, Bernd Edlinger wrote: > Hi, > > this is the latest version of my patch. > > As already said, it attempts to compute > the frame layout only when relevant data have > changed. > > Apologies for doing more clean-up on Daniel's > patch than absolutely necessary, but .

Re: [PATCH, rs6000] Fold vector logicals (eqv) in GIMPLE

2017-06-01 Thread Segher Boessenkool
On Wed, May 31, 2017 at 03:00:54PM -0500, Will Schmidt wrote: > Add support for early expansion of vector eqv built-ins. > OK for trunk? Yup, looks fine. Thanks, Segher > 2017-05-26 Will Schmidt > > * config/rs6000/rs6000.c (rs6000_gimple_fold_builtin): Add handling >

[PATCH] testsuite: ensure GCC_COLORS is unset

2017-06-01 Thread David Malcolm
Dominique noted on IRC that the new test show-template-tree-color.C (r248698) fails when GCC_COLORS is set in the environment. The following patch unsets GCC_COLORS within gcc-dg.exp, fixing this issue. Successfully regrtested on x86_64-pc-linux-gnu; I also verified the fix of the failing test by

Re: C/C++ PATCH to implement -Wmultiline-expansion (PR c/80116)

2017-06-01 Thread Trevor Saunders
On Thu, Jun 01, 2017 at 06:45:17PM +0200, Marek Polacek wrote: > A motivating example for this warning can be found e.g. in > > PRE10-C. Wrap multistatement macros in a do-while loop > https://www.securecoding.cert.org/confluence/x/jgL7 > > i.e., > > #define SWAP(x, y) \ > tmp = x; \ >

Re: [PATCH,DWARF,v2] AIX dwarf2out label fix

2017-06-01 Thread Jason Merrill
On 05/18/2017 06:00 AM, David Edelsohn wrote: This version adds a macro DWARF_INITIAL_LENGTH_SIZE_STR based on DWARF_OFFSET_SIZE to define the string expression to append to the label to correct the offset. Because AIX Assembler inserts the section length, the section label generated by GCC poin

Re: [PING**3] [PATCH] Force use of absolute path names for gcov

2017-06-01 Thread Bernd Edlinger
On 06/01/17 19:52, Nathan Sidwell wrote: > On 06/01/2017 11:59 AM, Bernd Edlinger wrote: >> Ping... > > What are you asking to be reviewed by who? > > nathan > Aehm, sorry. This is a gcc option that converts relative path names to absolute ones, so that gcov can properly merge the line numbers

Re: [PATCH,DWARF,v2] AIX dwarf2out label fix

2017-06-01 Thread David Edelsohn
On Thu, Jun 1, 2017 at 3:03 PM, Jason Merrill wrote: > On 05/18/2017 06:00 AM, David Edelsohn wrote: >> >> This version adds a macro DWARF_INITIAL_LENGTH_SIZE_STR based on >> DWARF_OFFSET_SIZE to define the string expression to append to the >> label to correct the offset. >> >> Because AIX Assemb

Re: [PATCH] testsuite: ensure GCC_COLORS is unset

2017-06-01 Thread Mike Stump
On Jun 1, 2017, at 11:59 AM, David Malcolm wrote: > > The following patch unsets GCC_COLORS within gcc-dg.exp, > fixing this issue. > OK for trunk? Ok.

[PATCH] Backport x86 intrin parameter/var uglification patch to 6.x

2017-06-01 Thread Jakub Jelinek
Hi! Jon mentioned on IRC somebody complained about namespace pollution of the STL headers (that include x86 intrin headers). This patch backports r239617 to 6.x, bootstrapped/regtested on x86_64-linux and i686-linux, committed to gcc-6-branch. The patch applies cleanly to gcc-5-branch if the pku

Re: [PATCH,DWARF,v2] AIX dwarf2out label fix

2017-06-01 Thread Jason Merrill
On Thu, Jun 1, 2017 at 3:27 PM, David Edelsohn wrote: > On Thu, Jun 1, 2017 at 3:03 PM, Jason Merrill wrote: >> On 05/18/2017 06:00 AM, David Edelsohn wrote: >>> >>> This version adds a macro DWARF_INITIAL_LENGTH_SIZE_STR based on >>> DWARF_OFFSET_SIZE to define the string expression to append to

Re: [PATCH] Add attribute((target_clone(...))) to PowerPC

2017-06-01 Thread Segher Boessenkool
Hi Mike, On Wed, May 31, 2017 at 06:33:37PM -0400, Michael Meissner wrote: > +/* On PowerPC, we have a limited number of target clones that we care about > + which means we can use an array to hold the options, rather than having > more > + elaborate data structures to identify each possible

Re: Default std::vector default and move constructor

2017-06-01 Thread François Dumont
On 01/06/2017 15:34, Jonathan Wakely wrote: On 31/05/17 22:28 +0200, François Dumont wrote: Unless I made a mistake it revealed that restoring explicit call to _Bit_alloc_type() in default constructor was not enough. G++ doesn't transform it into a value-init if needed. I don't know if it is a

[PATCH] handle bzero/bcopy in DSE and aliasing (PR 80933, 80934)

2017-06-01 Thread Martin Sebor
While testing some otherwise unrelated enhancements in these areas I noticed that calls to bzero and bcopy are not being handled as efficiently as equivalent calls to memset and memcpy. Specifically, redundant calls are not eliminated and the functions appear to be treated as if they allowed thei

Re: [PATCH, AArch64] Add x86 intrinsic headers to GCC AArch64 taget

2017-06-01 Thread Joseph Myers
On Mon, 29 May 2017, Hurugalawadi, Naveen wrote: > Hi, > > Please find attached the patch that adds first set of X86 instrinsic > headers to AArch64 target. > The implementation is based on similar work targeted at PPC64LE. > https://gcc.gnu.org/ml/gcc-patches/2017-05/msg00550.html > > We are us

[PATCH v3, rs6000] gcc mainline, add builtin support for vec_doublee, vec_doubleo, vec_doublel builtins

2017-06-01 Thread Carl E. Love
GCC Maintainers: This is version 3 of the patch to add support for the various vec_doublee, vec_doubleo, vec_doublel, vec_doubleh builtin-ins. I have addressed the formatting comments from Segher on version 2. I have run the patch through contrib/check_GNU_style.sh to check for obvious formattin

Re: [PING**2] [PATCH] Implement a warning for bogus sizeof(pointer) / sizeof(pointer[0])

2017-06-01 Thread Joseph Myers
The C changes are OK. -- Joseph S. Myers jos...@codesourcery.com

Re: C/C++ PATCH to implement -Wmultiline-expansion (PR c/80116)

2017-06-01 Thread Joseph Myers
On Thu, 1 Jun 2017, David Malcolm wrote: > The patch appears to only consider "if" and "else" clauses. Shouldn't > it also cover "for", "while" and "do/while"? do/while would normally get a syntax error in the problem cases. -- Joseph S. Myers jos...@codesourcery.com

Re: C/C++ PATCH to implement -Wmultiline-expansion (PR c/80116)

2017-06-01 Thread Martin Sebor
On 06/01/2017 10:45 AM, Marek Polacek wrote: A motivating example for this warning can be found e.g. in PRE10-C. Wrap multistatement macros in a do-while loop https://www.securecoding.cert.org/confluence/x/jgL7 i.e., #define SWAP(x, y) \ tmp = x; \ x = y; \ y = tmp used like this [1

Re: [PATCH v3, rs6000] gcc mainline, add builtin support for vec_doublee, vec_doubleo, vec_doublel builtins

2017-06-01 Thread Segher Boessenkool
Hi Carl, On Thu, Jun 01, 2017 at 02:55:45PM -0700, Carl E. Love wrote: > Add vec_doublee, vec_doubleo, vec_doublel, vec_doubleh builtin-ins > > gcc/ChangeLog: > > 2017-06-01 Carl Love > >* config/rs6000/rs6000-c: Add support for built-in functions >vector double vec_doublee (vector s

[PATCH][Aarch64] Add vectorized mersenne twister

2017-06-01 Thread Michael Collison
This patch adds an vectorized implementation of the mersenne twister random number generator. This implementation is approximately 2.6 times faster than the non-vectorized implementation. This implementation includes "arm_neon.h" when including the optimized . This has the effect of polluting

[PATCH][Aarch64] Relational compare zero not merged into subtract

2017-06-01 Thread Michael Collison
This patch improves code generation for relational compares against zero that are not merged into a subtract instruction. This patch improves the >= and < cases. An example of the '<' case: int lt (int x, int y) { if ((x - y) < 0) return 10; return 0; } Trunk generates: lt: s

Re: [PATCH] use the right conversion warning option (PR c/80892)

2017-06-01 Thread Eric Gallager
I tested this patch; it fixes the warnings that caused me to open the bug in the first place. Thank you! Eric On 5/30/17, Martin Sebor wrote: > The conversion enhancements I committed in r248431 introduced > an unintended change in which warning option is used to issue > certain integer convers