Re: [PATCH 3/3] i386: Enable AVX512 memory broadcast for INT andnot

2018-10-21 Thread Uros Bizjak
On Mon, Oct 22, 2018 at 12:59 AM H.J. Lu wrote: > > Many AVX512 vector operations can broadcast from a scalar memory source. > This patch enables memory broadcast for INT andnot operations. > > gcc/ > > PR target/72782 > * config/i386/sse.md (*andnot3_bst): New. This pattern shoul

Re: [PATCH 2/3] i386: Enable AVX512 memory broadcast for INT logic

2018-10-21 Thread Uros Bizjak
On Mon, Oct 22, 2018 at 12:59 AM H.J. Lu wrote: > > Many AVX512 vector operations can broadcast from a scalar memory source. > This patch enables memory broadcast for INT logic operations. > > gcc/ > > PR target/72782 > * config/i386/sse.md (*3_bcst): New. > > gcc/testsuite/ > >

Re: [PATCH 1/3] i386: Enable AVX512 memory broadcast for INT add

2018-10-21 Thread Uros Bizjak
On Mon, Oct 22, 2018 at 12:59 AM H.J. Lu wrote: > > Many AVX512 vector operations can broadcast from a scalar memory source. > This patch enables memory broadcast for INT add operations. > > gcc/ > > PR target/72782 > * config/i386/sse.md (avx512bcst): Updated for V4SI, V2DI, V8SI,

Re: [PATCH 14/14] Add D Phobos config, makefiles, and testsuite.

2018-10-21 Thread Iain Buclaw
On Tue, 16 Oct 2018 at 19:01, Richard Sandiford wrote: > > Iain Buclaw writes: > > diff --git a/libphobos/d_rules.am b/libphobos/d_rules.am > > new file mode 100644 > > index 000..b16cf5052d2 > > --- /dev/null > > +++ b/libphobos/d_rules.am > > @@ -0,0 +1,60 @@ > > +# This file contains s

Re: [PATCH 10/14] Add GDC Testsuite files.

2018-10-21 Thread Iain Buclaw
On Fri, 21 Sep 2018 at 22:47, Mike Stump wrote: > > On Sep 17, 2018, at 5:37 PM, Iain Buclaw wrote: > > > > This patch adds a further number of tests, but were added as part of > > fixing gdc-specific bugs. > > Ok. Trivial, and self-review applicable. Attaching Dejagnu file changes to support w

Re: [PATCH 12/14] Add GDC/GCC builtins and runtime support (part of D runtime)

2018-10-21 Thread Iain Buclaw
On Wed, 19 Sep 2018 at 22:45, Iain Buclaw wrote: > > On 18 September 2018 at 02:37, Iain Buclaw wrote: > > This patch adds GCC builtins and runtime support for GDC compiled code. > > > > - module __entrypoint defines the C main function. Its contents are > > parsed and compiled in during compi

[PATCH v2] Avoid unnecessarily numbered clone symbols

2018-10-21 Thread Michael Ploujnikov
Continuing from https://gcc.gnu.org/ml/gcc-patches/2018-10/msg01258.html Fixed up the code after the change to concat suggested by Bernhard Reutner. Outstanding question still remains: To write an exact replacement for numbered_clone_function_name (apart from the numbering) I also need to copy t

Re: [PATCH 09/14] Add D2 Testsuite Dejagnu files.

2018-10-21 Thread Iain Buclaw
On Fri, 21 Sep 2018 at 21:59, Mike Stump wrote: > > Ok. Richard requested that all tests use issue links rather than numbers so as not to confuse with gcc's bugzilla tracker. Posting just the patch overlay to original that applies just that. Regards -- Iain --- 09-v4v5-d-testsuite-dejagnu.pa

Re: [PATCH 08/14] Add D2 Testsuite files.

2018-10-21 Thread Iain Buclaw
On Tue, 25 Sep 2018 at 00:58, Iain Buclaw wrote: > > On 24 September 2018 at 20:28, Mike Stump wrote: > > On Sep 21, 2018, at 2:38 PM, Iain Buclaw wrote: > >> > >> On 21 September 2018 at 22:54, Mike Stump wrote: > >>> On Sep 17, 2018, at 5:36 PM, Iain Buclaw wrote: > > This patch ad

Re: [PATCH 06/14] Add patches for D language support in GCC proper.

2018-10-21 Thread Iain Buclaw
On Wed, 19 Sep 2018 at 22:21, Iain Buclaw wrote: > > On 18 September 2018 at 02:35, Iain Buclaw wrote: > > > > This patch adds D language support to GCC itself. > > > > Likewise, this was approved so long as all prerequisites have been > approved also. > > https://gcc.gnu.org/ml/gcc-patches/2017-

Re: [PATCH 07/14] Add patches for D language support in GCC targets.

2018-10-21 Thread Iain Buclaw
On Tue, 16 Oct 2018 at 17:28, Richard Sandiford wrote: > > Iain Buclaw writes: > > diff --git a/gcc/Makefile.in b/gcc/Makefile.in > > index 4b7cec82382..0b2daa320c3 100644 > > --- a/gcc/Makefile.in > > +++ b/gcc/Makefile.in > > @@ -2496,6 +2525,7 @@ s-tm-texi: build/genhooks$(build_exeext) > > $

Re: [PATCH 04/14] Add D front-end (GDC) config, Makefile, and manpages.

2018-10-21 Thread Iain Buclaw
On Tue, 16 Oct 2018 at 11:31, Richard Sandiford wrote: > > I'll pretend to be a doc reviewer... > > Iain Buclaw writes: > > +@page > > +@vskip 0pt plus 1filll > > +Published by the Free Software Foundation @* > > +51 Franklin Street, Fifth Floor@* > > +Boston, MA 02110-1301, USA@* > > Would be go

Re: [PATCH 02/14] Add D frontend (GDC) implementation.

2018-10-21 Thread Iain Buclaw
On Sat, 20 Oct 2018 at 11:03, Richard Sandiford wrote: > > Iain Buclaw writes: > > On 14 October 2018 at 17:29, Richard Sandiford > > wrote: > >> [Sorry if this turns out to do be a dup] > >> > >> Iain Buclaw writes: > >>> +/* Build nodes that are used by the D front-end. > >>> + These are di

Re: [PATCH 01/14] Add D front-end (DMD) language implementation and license.

2018-10-21 Thread Iain Buclaw
On Wed, 19 Sep 2018 at 20:46, Iain Buclaw wrote: > > On 18 September 2018 at 02:31, Iain Buclaw wrote: > > This patch adds the DMD front-end proper and license (Boost) files, > > comprised of a lexer, parser, and semantic analyzer. > > > > ftp://ftp.gdcproject.org/patches/v4/01-v4-d-frontend-dmd.

Re: [PATCH, rs6000] Use unaligned vector types for some pointer casts

2018-10-21 Thread Bill Schmidt
On 10/20/18 10:53 AM, Segher Boessenkool wrote: > Hi! > > On Fri, Oct 19, 2018 at 04:27:27PM -0500, Bill Schmidt wrote: >> The x86 intrinsic compatibility headers contain a couple of instances of >> undefined behavior where a cast to an aligned type is used when that >> alignment is not guaranteed

Re: [patch, fortran] Implement FINDLOC

2018-10-21 Thread Dominique d'Humières
Hi Thomas, With your patch, compiling the following test program logtest3 implicit none logical :: x = .true. integer, parameter :: I_FINDLOC_BACK(1) = findloc([1,1],1, & back=x) end program logtest3 gives an ICE gfc: internal compiler error: Segmentation fault: 11 signal t

[PATCH 3/3] i386: Enable AVX512 memory broadcast for INT andnot

2018-10-21 Thread H.J. Lu
Many AVX512 vector operations can broadcast from a scalar memory source. This patch enables memory broadcast for INT andnot operations. gcc/ PR target/72782 * config/i386/sse.md (*andnot3_bst): New. gcc/testsuite/ PR target/72782 * gcc.target/i386/avx512f-andn-di

[PATCH 1/3] i386: Enable AVX512 memory broadcast for INT add

2018-10-21 Thread H.J. Lu
Many AVX512 vector operations can broadcast from a scalar memory source. This patch enables memory broadcast for INT add operations. gcc/ PR target/72782 * config/i386/sse.md (avx512bcst): Updated for V4SI, V2DI, V8SI, V4DI, V16SI and V8DI. (*minus3_bcst): New.

[PATCH 2/3] i386: Enable AVX512 memory broadcast for INT logic

2018-10-21 Thread H.J. Lu
Many AVX512 vector operations can broadcast from a scalar memory source. This patch enables memory broadcast for INT logic operations. gcc/ PR target/72782 * config/i386/sse.md (*3_bcst): New. gcc/testsuite/ PR target/72782 * gcc.target/i386/avx512f-and-di-zmm-1.

V3 [PATCH] i386: Use scalar operand in floating point vec_dup patterns

2018-10-21 Thread H.J. Lu
On 10/21/18, H.J. Lu wrote: > On 10/17/18, H.J. Lu wrote: >> Since vector registers are also used for scalar floating point values, >> we can use scalar operand in floating point vec_dup patterns, which >> enables combiner to generate >> >> (set (reg:V8SF 84) >> (vec_duplicate:V8SF (mem/c:SF

[[C++ PATCH]] Implement C++2a P0330R2 - Literal Suffixes for ptrdiff_t and size_t

2018-10-21 Thread Ed Smith-Rowland
All, This patch implements C++2a proposal P0330R2 Literal Suffixes for ptrdiff_t and size_t*.  It's not official yet but looks very likely to pass.  It is incomplete because I'm looking for some opinions. 9We also might wait 'till it actually passes). This paper takes the direction of a lang

Fwd: [patch, fortran] Implement FINDLOC

2018-10-21 Thread Thomas Koenig
Hi, again rejected due to being overly large... The whole message can be found at https://gcc.gnu.org/ml/fortran/2018-10/msg00102.html Weitergeleitete Nachricht Betreff: [patch, fortran] Implement FINDLOC Datum: Sun, 21 Oct 2018 19:36:35 +0200 Von: Thomas Koenig An: fort...@

[Patch, fortran] PR71880 - pointer to allocatable character

2018-10-21 Thread Paul Richard Thomas
Committed as revision 265353. > Not quite fixed. The lhs character length doesn't get set and so it > fails at runtime. I will commit the patch as 'obvious'. > > Paul > > 2018-10-19 Paul Thomas > > PR fortran/71880 > * trans-expr.c (gfc_trans_pointer_assignment): Set the string > le

Re: gOlogy: do not change code in isolate-paths for warnings only

2018-10-21 Thread Richard Biener
On October 21, 2018 10:06:06 AM GMT+02:00, Alexandre Oliva wrote: >The isolate-paths pass is activated by various -f flags, but also by >-Wnull-dereference. Most of its codegen changes are conditioned on at >least one of the -f flags, but those that detect, warn about and >isolate paths that ret

Re: [PATCH 1/4] i386: Enable AVX512 memory broadcast for FMSUB

2018-10-21 Thread Uros Bizjak
On Sun, Oct 21, 2018 at 6:04 PM Uros Bizjak wrote: > > On Sat, Oct 20, 2018 at 8:46 AM H.J. Lu wrote: > > > > Many AVX512 vector operations can broadcast from a scalar memory source. > > This patch enables memory broadcast for FMSUB operations. In order to > > support AVX512 memory broadcast for

Re: [PATCH 1/4] i386: Enable AVX512 memory broadcast for FMSUB

2018-10-21 Thread Uros Bizjak
On Sat, Oct 20, 2018 at 8:46 AM H.J. Lu wrote: > > Many AVX512 vector operations can broadcast from a scalar memory source. > This patch enables memory broadcast for FMSUB operations. In order to > support AVX512 memory broadcast for FMSUB, FMSUB builtin functions are > also added, instead of pas

Re: [patch] new API for value_range

2018-10-21 Thread Aldy Hernandez
Is this fixed by Richard's patch to 87640? if so, perhaps this is a duplicate of said PR. On Sun, Oct 21, 2018 at 3:34 AM H.J. Lu wrote: > > On Wed, Oct 17, 2018 at 7:39 AM Aldy Hernandez wrote: > > > > > > > > On 10/17/18 6:50 AM, Richard Biener wrote: > > > On Thu, Oct 11, 2018 at 8:25 PM Aldy

[PATCH,FORTRAN] Fix memory leak of gsymbol

2018-10-21 Thread Bernhard Reutner-Fischer
Hi! Regtested on x86_64-unknown-linux, installing on aldot/fortran-fe-stringpool. We did not free global symbols. For a simplified abstract_type_3.f03 valgrind reports: 96 bytes in 1 blocks are still reachable in loss record 461 of 602 at 0x48377D5: calloc (vg_replace_malloc.c:711) by 0x21

Re: [PATCH 1/2] i386: Enable AVX512 memory broadcast for FP mul

2018-10-21 Thread H.J. Lu
On 10/21/18, Uros Bizjak wrote: > On Sat, Oct 20, 2018 at 11:47 PM H.J. Lu wrote: >> >> On 10/20/18, Uros Bizjak wrote: >> > On Fri, Oct 19, 2018 at 11:08 PM H.J. Lu wrote: >> >> >> >> Many AVX512 vector operations can broadcast from a scalar memory >> >> source. >> >> This patch enables memory

Re: [Patc, fortran] PR85603 - ICE with character array substring assignment

2018-10-21 Thread Thomas Koenig
Hi Paul, 2018-10-18 Paul Thomas PR fortran/85603 * frontend-passes.c (get_len_call): New function to generate a call to intrinsic LEN. (create_var): Use this to make length expressions for variable rhs string lengths. Clean up some white space issues. 2018-10-

Re: [PATCH] i386: Add missing AVX512VL or/xor intrinsics

2018-10-21 Thread Uros Bizjak
On Sun, Oct 21, 2018 at 2:03 AM H.J. Lu wrote: > > gcc/ > > PR target/87662 > * i386/avx512vlintrin.h (_mm256_or_epi32): New. > (_mm_or_epi32): Likewise. > (_mm256_xor_epi32): Likewise. > (_mm_xor_epi32): Likewise. > (_mm256_or_epi64): Likewise. >

Re: [PATCH 1/2] i386: Enable AVX512 memory broadcast for FP mul

2018-10-21 Thread Uros Bizjak
On Sat, Oct 20, 2018 at 11:47 PM H.J. Lu wrote: > > On 10/20/18, Uros Bizjak wrote: > > On Fri, Oct 19, 2018 at 11:08 PM H.J. Lu wrote: > >> > >> Many AVX512 vector operations can broadcast from a scalar memory source. > >> This patch enables memory broadcast for FP mul operations. > >> > >> gcc

[PATCH][PR 87633] Do not generate unordered integer comparisons

2018-10-21 Thread Yuri Gribov
Hi all, My recent patch which replaced (float_type)int_var1 CMP (float_type)int_var2 with int_var1 CMP int_var2 may generate unordered comparisons of integer values which are both unnecessary and may also upset later passes. This patch fixes the problem and bootstraps without regressions on x

V2 [PATCH] i386: Use scalar operand in floating point vec_dup patterns

2018-10-21 Thread H.J. Lu
On 10/17/18, H.J. Lu wrote: > Since vector registers are also used for scalar floating point values, > we can use scalar operand in floating point vec_dup patterns, which > enables combiner to generate > > (set (reg:V8SF 84) > (vec_duplicate:V8SF (mem/c:SF (symbol_ref:DI ("y") > > For AVX

gOlogy: fix debug binds in auto-inc-dec

2018-10-21 Thread Alexandre Oliva
As auto_inc_dec pass combines incs and mems from different insns, it often causes regs to temporarily hold a value different from the one it would before the transformation. Debug insns within that range would therefore end up binding to the wrong expression after the transformation. This patch a

gOlogy: do not change code in isolate-paths for warnings only

2018-10-21 Thread Alexandre Oliva
The isolate-paths pass is activated by various -f flags, but also by -Wnull-dereference. Most of its codegen changes are conditioned on at least one of the -f flags, but those that detect, warn about and isolate paths that return the address of local variables are enabled even if the pass is activ