Re: [PATCH] Workaround glibc <= 2.23 nextafterl/nexttowardl bug (PR tree-optimization/85699)

On May 9, 2018 10:52:05 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >glibc <= 2.23 has buggy nextafterl/nexttowardl as can be seen on the >nextafter-2.c testcase. > >Do we want to workaround this bug, e.g. with the following patch? Works for me. Was the reason to test the target

Fix PR85726 (div-div suboptimization) and a rant on match.pd :s-flag

Replacing a division feeding a division helps only when the second division is the only user, and "fusing" the divisions is downright bad if another user of the result of first division is a modulus of the same value as the second division, forming a divmod pair. See the test-case, where for the

Re: [PATCH, rs6000] Map dcbtst, dcbtt to n2=0 for __builtin_prefetch builtin.

Hi, On Tue, May 08, 2018 at 05:04:33PM -0700, Carl Love wrote: > On Tue, 2018-05-08 at 11:24 -0500, Segher Boessenkool wrote: > > What ISA version is required for the TH field to do anything?  Will > > it work on older machines too (just ignored)?  What assembler version > > is required? > > I

Re: [PATCH] PowerPC address support clean, patch 3 of 4

On Thu, May 03, 2018 at 01:22:10PM -0400, Michael Meissner wrote: > 2018-05-03 Michael Meissner > > * config/rs6000/rs6000.c (mode_supports_d_form): Rename > mode_supports_vmx_dform to mode_supports_d_form. Add an optional > argument to say which

Re: [PATCH] PowerPC address support clean, patch 2 of 4

On Thu, May 03, 2018 at 01:20:32PM -0400, Michael Meissner wrote: > 2018-05-03 Michael Meissner > > * config/rs6000/rs6000.c (mode_supports_vmx_dform): Move these > functions to be next to the other mode_supports functions. >

Re: [PATCH] PowerPC address support clean, patch 1 of 4

Hi Mike, On Thu, May 03, 2018 at 01:17:03PM -0400, Michael Meissner wrote: > 2018-05-03 Michael Meissner > > * config/rs6000/rs6000.c (mode_supports_dq_form): Rename > mode_supports_vsx_dform_quad to mode_supports_dq_form. >

Re: [PATCH, aarch64] Patch to update pipeline descriptions in thunderx2t99.md

On Fri, 2018-05-04 at 14:05 -0700, Andrew Pinski wrote: >  > >    (thunderx2t99_loadpair): Fix cpu unit ordering. > I think the original ordering was correct.  The address calculation > happens before the actual load. > thunderx2t99_asimd_load1_ldp would have a similar issue. > > Thanks, >

[PATCH] PR fortran/70870 -- Reject data object with default initialization

I plan to commit the attach patch on Saturday unless someone objects. 2018-05-09 Steven G. Kargl PR fortran/70870 * data.c (gfc_assign_data_value): Check that a data object does not also have default initialization. 2018-05-09 Steven G. Kargl

[PATCH] PR fortran/85521 -- Zero length substrings in array aconstructors

I paln to commit the attached patch on Saturday unless someone objects. 2018-05-09 Steven G. Kargl PR fortran/85521 * array.c (gfc_resolve_character_array_constructor): Substrings with upper bound smaller than lower bound are zero length strings.

[PATCH] PR fortran/85687 -- Check argument of RANK.

I plan to commit the attached patch on Saturday unless someone voices an objection. 2018-05-09 Steven G. Kargl PR fortran/85687 * check.c (gfc_check_rank): Check that the argument is a data object. 2018-05-09 Steven G. Kargl

libgo patch committed: Update go tool to match recent upstream changes

Several recent changes to the master version of cmd/go improve the gofrontend support. These changes are partially copies of existing gofrontend differences, and partially new code. This libgo patch makes the gofrontend match the upstream code. The changes included here come from:

Re: [PATCH 1/2, expr.c] Optimize switch with sign-extended index.

On Wed, May 2, 2018 at 3:05 PM, Jim Wilson wrote: > This improves the code for a switch statement on targets that sign-extend > function arguments, such as RISC-V. Given a simple testcase > ... > gcc/ > * expr.c (do_tablejump): When converting index to Pmode, if

Re: [PATCH] RISC-V: Add with-multilib-list support.

On Tue, May 1, 2018 at 11:52 AM, Jim Wilson wrote: > gcc/ > PR target/84797 > * config.gcc (riscv*-*-*): Handle --with-multilib-list. > * config/riscv/t-withmultilib: New. > * config/riscv/withmultilib.h: New. > * doc/install.texi:

[PATCH] Add constant folding for x86 shift builtins by vector

Hi! The following patch on top of the earlier ix86_*fold_builtin patch adds folding also for the *s{ll,rl,ra}v* builtins. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2018-05-09 Jakub Jelinek PR target/85323 * config/i386/i386.c

[PATCH] Workaround glibc <= 2.23 nextafterl/nexttowardl bug (PR tree-optimization/85699)

Hi! glibc <= 2.23 has buggy nextafterl/nexttowardl as can be seen on the nextafter-2.c testcase. Do we want to workaround this bug, e.g. with the following patch? Regtested on x86_64-linux (with glibc 2.26). Ok for trunk? 2018-05-09 Jakub Jelinek PR

[C++ PATCH] Fix offsetof constexpr handling (PR c++/85662, take 4)

On Wed, May 09, 2018 at 11:01:18AM -0400, Jason Merrill wrote: > On Wed, May 9, 2018 at 10:47 AM, Jakub Jelinek wrote: > > On Wed, May 09, 2018 at 10:40:26AM -0400, Jason Merrill wrote: > >> On Wed, May 9, 2018 at 4:55 AM, Jakub Jelinek wrote: > >> > On Tue,

Re: [PATCH] Define DW_FORM_strx* and DW_FORM_addrx*.

OK, thanks. On Wed, May 9, 2018 at 1:19 PM, Thomas Rix wrote: > This patch defines the dwarf 5 forms. > DW_FORM_strx1, DW_FORM_strx2, DW_FORM_strx3, DW_FORM_strx4 > And similar for addrx. > > Tom > > >

Re: [PATCH, rs6000] Add missing vec_expte, vec_loge, vec_re

Hi! On Wed, May 09, 2018 at 09:07:49AM -0700, Carl Love wrote: > 2018-05-09 Carl Love > * gcc.target/powerpc/builtins-8-runnable.c: New builtin test file. > --- /dev/null > +++ b/gcc/testsuite/gcc.target/powerpc/builtins-8-runnable.c > @@ -0,0 +1,98 @@ > +/* { dg-do run

[PATCH v2, rs6000] Improve Documentation of Built-In Functions Part 1

This is the first of several planned patches to address shortcomings in existing documentation of PowerPC built-in functions. The focus of this particular patch is to improve documentation of basic built-in functions that do not require inclusion of special header files. A summary of this patch

[PATCH] Define DW_FORM_strx* and DW_FORM_addrx*.

This patch defines the dwarf 5 forms. DW_FORM_strx1, DW_FORM_strx2, DW_FORM_strx3, DW_FORM_strx4 And similar for addrx. Tom 0001-Define-DW_FORM_strx-and-DW_FORM_addrx.patch Description: 0001-Define-DW_FORM_strx-and-DW_FORM_addrx.patch

[PATCH, rs6000] Add missing vec_expte, vec_loge, vec_re

GCC Maintainers: The following patch adds tests for the vec_expte, vec_loge, and vec_re builtins. The patch for the test case was tested on powerpc64le-unknown-linux-gnu (Power 8 LE) powerpc64-unknown-linux-gnu (Power 8 BE) powerpc64le-unknown-linux-gnu (Power 9 LE).  Please let

Re: [PATCH, libgomp, openacc] Use GOMP_ASYNC_SYNC in GOACC_declare

On 11/17/2017 09:45 AM, Tom de Vries wrote: Hi, GOACC_enter_exit_data has this prototype: ... void GOACC_enter_exit_data (int device, size_t mapnum,    void **hostaddrs, size_t *sizes,    unsigned short *kinds,    int async, int

Re: [og7] Update deviceptr handling in Fortran

On 05/09/2018 03:50 AM, Thomas Schwinge wrote: >> In addition to XPASS'ing devicetpr-1.f90, this patch [...] > > Apart from one remaining XFAIL for "-Os" (see PR80995), I now too see the > following XPASSes on my main development machine: > > PASS: libgomp.oacc-fortran/deviceptr-1.f90

Re: [PATCH] Add ax_pthread.m4 for use in binutils-gdb

Applied. On Tue, May 8, 2018 at 7:47 PM, Joshua Watt wrote: > On Wed, Apr 18, 2018, 05:20 Pedro Alves wrote: > >> On 04/17/2018 11:10 PM, Joshua Watt wrote: >> > On Tue, 2018-04-17 at 22:50 +0100, Pedro Alves wrote: >> >> On 04/17/2018 06:24 PM, Joshua

Re: [C++ PATCH] Fix offsetof constexpr handling (PR c++/85662)

On Wed, May 9, 2018 at 10:47 AM, Jakub Jelinek wrote: > On Wed, May 09, 2018 at 10:40:26AM -0400, Jason Merrill wrote: >> On Wed, May 9, 2018 at 4:55 AM, Jakub Jelinek wrote: >> > On Tue, May 08, 2018 at 11:28:18PM -0400, Jason Merrill wrote: >> >> Maybe add a

Re: [Patch] Use two source permute for vector initialization (PR 85692)

On Mittwoch, 9. Mai 2018 11:08:02 CEST Jakub Jelinek wrote: > On Tue, May 08, 2018 at 01:25:35PM +0200, Allan Sandfeld Jensen wrote: > > 2018-05-08 Allan Sandfeld Jensen > > 2 spaces between date and name and two spaces between name and email > address. > > > gcc/ >

Re: [C++ PATCH] Fix offsetof constexpr handling (PR c++/85662)

On Wed, May 09, 2018 at 10:40:26AM -0400, Jason Merrill wrote: > On Wed, May 9, 2018 at 4:55 AM, Jakub Jelinek wrote: > > On Tue, May 08, 2018 at 11:28:18PM -0400, Jason Merrill wrote: > >> Maybe add a type parameter that defaults to size_type_node... > >> > >> > + ret =

Re: [C++ PATCH] Fix offsetof constexpr handling (PR c++/85662)

On Wed, May 9, 2018 at 4:55 AM, Jakub Jelinek wrote: > On Tue, May 08, 2018 at 11:28:18PM -0400, Jason Merrill wrote: >> Maybe add a type parameter that defaults to size_type_node... >> >> > + ret = fold_convert_loc (loc, TREE_TYPE (expr), >> > +

Re: [PATCH] Fix PR c++/85400

OK. On Wed, May 9, 2018 at 6:05 AM, Eric Botcazou wrote: >> So it isn't clear to me if a cxx_make_decl_one_only is the way to go. Maybe >> doing the recalculation in comdat_linkage and maybe_make_one_only only >> would be sufficient. > > Patch to that effect attached,

Re: [committed][AArch64] Predicated SVE comparison folds

* gcc.target/aarch64/sve/vcond_6.c: Add missing brace. diff --git a/gcc/testsuite/gcc.target/aarch64/sve/vcond_6.c b/gcc/testsuite/gcc.target/aarch64/sve/vcond_6.c index a59f08d553..f41c94c400 100644 --- a/gcc/testsuite/gcc.target/aarch64/sve/vcond_6.c +++

Re: [PING] [PATCH, libgomp, openacc] Factor out async argument utility functions

On 05/01/2018 10:50 PM, Tom de Vries wrote: On 11/17/2017 02:18 PM, Tom de Vries wrote: Hi, I've factored out 3 new functions to test properties of enum acc_async_t: ... typedef enum acc_async_t {    /* Keep in sync with include/gomp-constants.h.  */    acc_async_noval = -1,    acc_async_sync 

Re: [PATCH, v2] Recognize a missed usage of a sbfiz instruction

On 09/05/18 13:30, Luis Machado wrote: Hi Kyrill, On 05/08/2018 11:09 AM, Kyrill Tkachov wrote: Hi Luis, On 07/05/18 15:28, Luis Machado wrote: Hi, On 02/08/2018 10:45 AM, Luis Machado wrote: Hi Kyrill, On 02/08/2018 09:48 AM, Kyrill Tkachov wrote: Hi Luis, On 06/02/18 15:04, Luis

[wwwdocs] about.html - simplify, update, add a bit

I promised Martin to look into adding more information to gcc.gnu.org/about.html for new contributors. This isn't the actual meat, but a number of changes I found while preparing for that. Applied (last weekend actually). Gerald Split a long sentence. Adjust the intro. Add a note to add

[PATCH] Make std::function tolerate semantically non-CopyConstructible objects

To satisfy the CopyConstructible requirement a callable object stored in a std::function must behave the same when copied from a const or non-const source. If copying a non-const object doesn't produce an equivalent copy then the behaviour is undefined. But we can make our std::function more

[PATCH] Fix BB scalar costing

The following fixes the same issue with scalar BB costing as I fixed earlier this year with the loop scalar costing. We are currently comparing apples and oranges on x86 where the add_stmt_cost hook uses costs dependent on the stmt operation while the old hook does not (cannot). Fixed as

Re: [PATCH 3/4] shrink-wrap: Improve spread_components (PR85645)

On Wed, May 09, 2018 at 09:33:30AM +0200, Eric Botcazou wrote: > > Now, neither of the two branches needs to have LR restored at all, > > because both of the branches end up in an infinite loop. > > > > This patch makes spread_component return a boolean saying if anything > > was changed, and if

Re: [PATCH, v2] Recognize a missed usage of a sbfiz instruction

Hi Kyrill, On 05/08/2018 11:09 AM, Kyrill Tkachov wrote: Hi Luis, On 07/05/18 15:28, Luis Machado wrote: Hi, On 02/08/2018 10:45 AM, Luis Machado wrote: Hi Kyrill, On 02/08/2018 09:48 AM, Kyrill Tkachov wrote: Hi Luis, On 06/02/18 15:04, Luis Machado wrote: Thanks for the feedback

Re: Handle vector boolean types when calculating the SLP unroll factor

On Wed, May 9, 2018 at 1:29 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Wed, May 9, 2018 at 12:34 PM, Richard Sandiford >> wrote: >>> The SLP unrolling factor is calculated by finding the

Re: [PATCH 1/3] Add PTWRITE builtins for x86

On Wed, May 9, 2018 at 1:23 PM, Peryt, Sebastian wrote: > I have rebased this patch to the latest trunk and addressed comments. Also, > there was a test in changelog, > but not in the patch itself - this has been added. > > Is it ok for trunk and backport to GCC-8

Re: Incremental LTO linking part 2: lto-plugin support

On Wed, May 9, 2018 at 1:25 AM, Jan Hubicka wrote: >> On Tue, 8 May 2018, Jan Hubicka wrote: >> >> > > On Tue, May 8, 2018 at 8:14 AM, Jan Hubicka wrote: >> > > > Hi, >> > > > with lto, incremental linking can be meaninfuly done in three ways: >> > > > 1) read

Re: [PATCH][i386] Adding CLDEMOTE instruction

On Tue, May 8, 2018 at 1:58 PM, Peryt, Sebastian wrote: > Sorry, forgot attachment. > > Sebastian > > > -Original Message- > From: Peryt, Sebastian > Sent: Tuesday, May 8, 2018 1:56 PM > To: gcc-patches@gcc.gnu.org > Cc: Uros Bizjak ; Kirill

Re: [PATCH][i386] Adding WAITPKG instructions

On Tue, May 8, 2018 at 1:34 PM, Peryt, Sebastian wrote: > Hi, > > This patch adds support for WAITPKG instructions. > > Is it ok for trunk and after few day for backport to GCC-8? > > 2018-05-08 Sebastian Peryt > > gcc/ > > *

Re: Handle vector boolean types when calculating the SLP unroll factor

Richard Biener writes: > On Wed, May 9, 2018 at 12:34 PM, Richard Sandiford > wrote: >> The SLP unrolling factor is calculated by finding the smallest >> scalar type for each SLP statement and taking the number of required >> lanes from

RE: [PATCH 1/3] Add PTWRITE builtins for x86

I have rebased this patch to the latest trunk and addressed comments. Also, there was a test in changelog, but not in the patch itself - this has been added. Is it ok for trunk and backport to GCC-8 after few days? gcc/ * common/config/i386/i386-common.c (OPTION_MASK_ISA_PTWRITE_SET,

[PATCH, i386]: Implement usadv64qi

This patch adds usadv64qi expander, so the compiler is able to vectorize with 512bit vpsadbw insn. 2017-05-09 Uros Bizjak PR target/85693 * config/i386/sse.md (usadv64qi): New expander. Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}. OK for

Re: Handle vector boolean types when calculating the SLP unroll factor

On Wed, May 9, 2018 at 12:34 PM, Richard Sandiford wrote: > The SLP unrolling factor is calculated by finding the smallest > scalar type for each SLP statement and taking the number of required > lanes from the vector versions of those scalar types. E.g. for an >

Re: [og7] Update deviceptr handling in Fortran

Hi Cesar! On Mon, 7 May 2018 08:49:26 -0700, Cesar Philippidis wrote: > This patch teaches both the Fortran FE and the gimplifier how to only > utilize one data mapping for OpenACC deviceptr clauses. [...] Thanks! (I didn't verify your code changes.) > In addition

[nvptx, PR85626, committed] Make trap insn noreturn

Hi, the nvptx trap* define_insns are implemented using the ptx insn 'trap' . The ptx insn 'trap' may however return, and therefore the ptx insn 'exit' is needed after the 'trap'. Fixed by attached patch. Build x86_64 with nvptx accelerator. Committed to trunk. Thanks, - Tom [nvptx] Make

Handle vector boolean types when calculating the SLP unroll factor

The SLP unrolling factor is calculated by finding the smallest scalar type for each SLP statement and taking the number of required lanes from the vector versions of those scalar types. E.g. for an int32->int64 conversion, it's the vector of int32s rather than the vector of int64s that determines

Re: Incremental LTO linking part 5: symtab and compilation driver support

On Tue, 8 May 2018, Jan Hubicka wrote: > Hi, > this patch adds the symtab support for LTO incremental linking. Most of the > code path is same for both modes of incremental link except hat we want to > produce LTO object file rather than compile down to assembly. > > Only non-obvious changes are

Re: Incremental LTO linking part 4: lto-opts support

On Tue, 8 May 2018, Jan Hubicka wrote: > Hi, > this patch prevents lto-option to store some flags that does not make snese > to store, > in partiuclar dumpdir and -fresolution. These definitly should not be > preserved from > compile time to link time and in case of incremental linking they

Re: [PATCH] Fix PR c++/85400

> So it isn't clear to me if a cxx_make_decl_one_only is the way to go. Maybe > doing the recalculation in comdat_linkage and maybe_make_one_only only > would be sufficient. Patch to that effect attached, tested on x86-64/Linux, OK for mainline? 2018-05-09 Eric Botcazou

Re: Incremental LTO linking part 3: lto-wrapper support

On Tue, 8 May 2018, Jan Hubicka wrote: > Hi, > this patch makes lto-wrapper to look for -flinker-output=rel and in this > case confiugre lto1 in non-WHOPR mode + disable section renaming. > > Bootstrapped/regtested x86_64-linux with rest of incremental link patchset. > OK? > > *

Re: [RFA] Incremental LTO linking part 1: simple-object bits

On Tue, 8 May 2018, Jan Hubicka wrote: > Hi, > for incremental linking of LTO objects we need to copy debug sections from > source object files into destination without renaming them from .gnu.debuglto > into the standard debug section (because they will again be LTO debug section > in the

Re: [gomp5] simd if/nontemporal clauses parsing and cancel if modifier

On Fri, May 4, 2018 at 8:37 PM, Jakub Jelinek wrote: > Hi! > > This patch adds parsing of if and nontemporal clauses for simd construct > and also adds parsing of (optional) cancel modifier for if clause on cancel > directive. > > While nontemporal clause is just an optimization

Re: Add clobbers around IFN_LOAD/STORE_LANES

On Tue, May 8, 2018 at 5:56 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Tue, May 8, 2018 at 3:25 PM, Richard Sandiford >> wrote: >>> We build up the input to IFN_STORE_LANES one vector at a

Re: [libstdc++, PATCH] PR libstdc++/83140 - assoc_legendre returns negated value when m is odd.

On 07/05/18 12:39 -0400, Ed Smith-Rowland wrote: All, We were using a different convention for P_l^m assoc_legendre(int l, int m, FloatTp x)  - the so-called Condon-Shortley convention which includes (-1)^m.  This unfortunately is common. This factor is taken out to match the standard. 

Re: [Patch] Use two source permute for vector initialization (PR 85692)

On Tue, May 08, 2018 at 01:25:35PM +0200, Allan Sandfeld Jensen wrote: > 2018-05-08 Allan Sandfeld Jensen 2 spaces between date and name and two spaces between name and email address. > gcc/ > > PR tree-optimization/85692 > * tree-ssa-forwprop.c

Re: Debug Mode ENH 3/4: Add backtrace

On 08/05/18 16:51 -0700, Ian Lance Taylor via libstdc++ wrote: On Tue, May 8, 2018 at 12:54 PM, François Dumont wrote: I'll go with this version for now but I'll look into libbacktrace. It will be perhaps the occasion to play with autoconf & al tools to find out if I

Re: [C++ PATCH] Fix offsetof constexpr handling (PR c++/85662)

On Tue, May 08, 2018 at 11:28:18PM -0400, Jason Merrill wrote: > Maybe add a type parameter that defaults to size_type_node... > > > > > --- gcc/c/c-fold.c.jj 2018-01-17 22:00:12.310228253 +0100 > > +++ gcc/c/c-fold.c 2018-05-08 21:52:43.303940175 +0200 > > @@ -473,7 +473,8 @@

Re: Incremental LTO linking part 2: lto-plugin support

> On Tue, 8 May 2018, Jan Hubicka wrote: > > > > On Tue, May 8, 2018 at 8:14 AM, Jan Hubicka wrote: > > > > Hi, > > > > with lto, incremental linking can be meaninfuly done in three ways: > > > > 1) read LTO file and produce non-LTO .o file > > > > this is current behaviour

Re: [patch] Fix PR rtl-optimization/85638

> 2018-05-07 Eric Botcazou > > PR rtl-optimization/85638 > * bb-reorder.c: Include common/common-target.h. > (create_forwarder_block): New function extracted from... > (fix_up_crossing_landing_pad): ...here. Rename into... >

Re: Incremental LTO linking part 2: lto-plugin support

On Tue, 8 May 2018, Jan Hubicka wrote: > > On Tue, May 8, 2018 at 8:14 AM, Jan Hubicka wrote: > > > Hi, > > > with lto, incremental linking can be meaninfuly done in three ways: > > > 1) read LTO file and produce non-LTO .o file > > > this is current behaviour of gcc -r or

Re: [PATCH 3/4] shrink-wrap: Improve spread_components (PR85645)

> Now, neither of the two branches needs to have LR restored at all, > because both of the branches end up in an infinite loop. > > This patch makes spread_component return a boolean saying if anything > was changed, and if so, it is called again. This obviously is finite > (there is a finite

Re: [PATCH 2/4] regrename: Don't rename the dest of a REG_CFA_REGISTER (PR85645)

> 2018-05-08 Segher Boessenkool > > PR rtl-optimization/85645 > * regrename.c (build_def_use): Also kill the chains that include the > destination of a REG_CFA_REGISTER note. OK, thanks. -- Eric Botcazou

Re: [PATCH 1/4] regcprop: Avoid REG_CFA_REGISTER notes (PR85645)

> 2018-05-08 Segher Boessenkool > > PR rtl-optimization/85645 > * regcprop.c (copyprop_hardreg_forward_1): Don't propagate into an > insn that has a REG_CFA_REGISTER note. OK, thanks. -- Eric Botcazou