On 06/29/2018 10:12 AM, Cesar Philippidis wrote:
> Ping.
While porting the vector length patches to trunk, I realized that I
mistakenly removed support for the environment variable GOMP_OPENACC_DIM
in this patch (thanks for adding those test case Tom!). I'll post an
updated version of this patch
On Wed, Jun 27, 2018 at 07:35:15PM -0400, Jason Merrill wrote:
> On Wed, Jun 27, 2018 at 12:53 PM, Marek Polacek wrote:
> > This PR complains about us accepting invalid code like
> >
> > template struct A {};
> > A<-1> a;
> >
> > Where we should detect the narrowing: [temp.arg.nontype] says
>
On 03/22/2018 05:56 AM, Renlin Li wrote:
> Hi all,
>
> As described in PR84877. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84877
> The local copy of parameter on stack is not aligned.
>
> For BLKmode paramters, a local copy on the stack will be saved.
> There are three cases:
> 1) arguments
The attached patch adds support Fortran support for OpenACC deviceptr
and the use of common block variables in data clauses (both implicit and
explicit). This patch also relaxes the Fortran parser to not error
certain types of integral expressions and assumed-sized arrays.
With respect to those
On 05/21/2018 12:58 PM, marxin wrote:
> gcc/ChangeLog:
>
> 2018-05-25 Denys Vlasenko
> Martin Liska
>
> PR middle-end/66240
> PR target/45996
> PR c/84100
> * common.opt: Rename align options with 'str_' prefix.
> * common/config/i386/i386-common.c
[ Returning to another old patch... ]
On 11/07/2017 10:33 AM, Aldy Hernandez wrote:
> [One more time, but without rejected HTML mail, because apparently this
> is my first post to gcc-patches *ever* ;-)].
>
> Howdy!
>
> While poking around in the backwards threader I noticed that we bail if
>
On Thu, Jun 28, 2018 at 12:16 PM, Richard Biener wrote:
>
> The fortran FE creates array descriptor types via build_distinct_type_copy
> which ends up re-using the TYPE_FIELDs chain of FIELD_DECLs between
> types in different type-variant chains. While that seems harmless
> in practice it
So this patch converts the v850 port to use LRA. From a code generation
standpoint it looks like the old bare v850 gets slightly worse code.
However, we get slightly better code in general on the newer parts like
v850e3v5.
The only really interesting part of the conversion is the removal of
On Fri, Jun 29, 2018 at 12:52:59AM +0200, Jakub Jelinek wrote:
> Hi!
>
> The newly added rs6000-modes.h is now included from rs6000.h, so it is
> needed when building plugins that include tm.h, but it wasn't listed in the
> Makefile fragments and therefore included among PLUGIN_HEADERS.
>
>
Attached are the updated reductions tests cases. Again, these have been
bootstrapped and regression tested cleanly for x86_64 with nvptx
offloading. Is it OK for trunk?
Thanks,
Cesar
2018-06-29 Cesar Philippidis
Nathan Sidwell
gcc/testsuite/
*
Attaches are the FE changes for the OpenACC reduction enhancements. It
depends on the ME patch.
Is this patch OK for trunk? It bootstrapped / regression tested cleanly
for x86_64 with nvptx offloading.
Thanks,
Cesar
2018-06-29 Cesar Philippidis
Nathan Sidwell
gcc/c/
* c-parser.c
The attached patch includes the nvptx and GCC ME reductions enhancements.
Is this patch OK for trunk? It bootstrapped / regression tested cleanly
for x86_64 with nvptx offloading.
Thanks,
Cesar
2018-06-29 Cesar Philippidis
Nathan Sidwell
gcc/
* config/nvptx/nvptx.c
The following patch set includes various OpenACC reduction enhancements
present in og8. These include the following individual og8 commits:
* (4469fc4) [Fortran] Permit reductions in gfc_omp_clause_copy_ctor
* (704f1a2) [nxptx, OpenACC] vector reductions
* (8a35c89) [OpenACC] Fix a
On Fri, Jun 29, 2018 at 11:07:48AM -0700, Cesar Philippidis wrote:
> On 06/29/2018 10:49 AM, Jakub Jelinek wrote:
> > On Fri, Jun 29, 2018 at 10:33:56AM -0700, Cesar Philippidis wrote:
> >> @@ -1044,21 +1046,6 @@ gfc_omp_finish_clause (tree c, gimple_seq *pre_p)
> >> return;
> >>
> >>
On 06/29/2018 10:49 AM, Jakub Jelinek wrote:
> On Fri, Jun 29, 2018 at 10:33:56AM -0700, Cesar Philippidis wrote:
>> @@ -1044,21 +1046,6 @@ gfc_omp_finish_clause (tree c, gimple_seq *pre_p)
>> return;
>>
>>tree decl = OMP_CLAUSE_DECL (c);
>> -
>> - /* Assumed-size arrays can't be
Howdy!
Attached are some cleanups to the VRP code dealing with PLUS/MINUS_EXPR
on ranges. This will make it easier to share code with any other range
implementation in the future, but is completely independent from any
other work.
Currently there is a lot of code duplication in the
On Fri, Jun 29, 2018 at 10:33:56AM -0700, Cesar Philippidis wrote:
> @@ -1044,21 +1046,6 @@ gfc_omp_finish_clause (tree c, gimple_seq *pre_p)
> return;
>
>tree decl = OMP_CLAUSE_DECL (c);
> -
> - /* Assumed-size arrays can't be mapped implicitly, they have to be
> - mapped
Bill:
> > * config/rs6000/rs6000-c.c: Map ALTIVEC_BUILTIN_VEC_UNPACKH for
> > float argument to ALTIVEC_BUILTIN_UNPACKH_V4SF.
> > Map ALTIVEC_BUILTIN_VEC_UNPACKL for float argument to
> > ALTIVEC_BUILTIN_UNPACKH_V4SF.
>
> UNPACKL
>
> That's all I see; will leave to Segher to
The attached patch includes various bug fixes and performance
improvements involving the use of Fortran arrays in OpenACC data
clauses. More specifically,
* Transfers Fortran arrays using GOMP_MAP_FIRSTPRIVATE_POINTERs.
* Privatizes array descriptors in the Fortran FE.
* Corrects a couple
On 6/29/18, Ian Lance Taylor wrote:
> On Fri, Jun 29, 2018 at 8:43 AM, Jakub Jelinek wrote:
>> On Fri, Jun 29, 2018 at 09:09:38AM -0600, Jeff Law wrote:
>>> > Btw, running find to search for libtool.m4/ltmain.sh I find extra
>>> > copies in
>>> >
>>> > ./libgo/config/ltmain.sh
>>> >
On 6/29/18, Jeff Law wrote:
> On 06/27/2018 08:40 PM, Martin Sebor wrote:
>> On 06/27/2018 03:53 PM, Jeff Law wrote:
>>> On 06/27/2018 09:27 AM, Jakub Jelinek wrote:
On Wed, Jun 27, 2018 at 09:17:07AM -0600, Jeff Law wrote:
>> About 115 tests fail due to incompatible declarations of
Ping.
Ceasr
On 06/20/2018 02:59 PM, Cesar Philippidis wrote:
> At present, the nvptx libgomp plugin does not take into account the
> amount of shared resources on GPUs (mostly shared-memory are register
> usage) when selecting the default num_gangs and num_workers. In certain
> situations, an
On Jun 29, 2018, at 9:38 AM, Carl Love wrote:
>
> GCC Maintainers:
>
> The vec_unpackh, vec_unpackl builtins with vector float arguments
> unpack the high or low half of a floating point vector and convert the
> elements to a vector of doubles. The current implementation of the
> builtin for
On 06/07/2018 09:57 AM, Martin Sebor wrote:
> The attached patch enhances the strlen pass to more consistently
> deal with MEM_REF assignments (PR 86042) and to track string
> lengths across calls to memcpy that overwrite parts of a string
> with sequences of non-nul characters (PR 86043).
>
>
On 06/29/2018 06:56 AM, Rainer Orth wrote:
Hi Martin,
While looking into opportunities to detect strnlen/strlen coding
mistakes (pr86199) I noticed a bug in the strnlen implementation
I committed earlier today that lets a strnlen() result be saved
and used in subsequent calls to strlen() with
On Fri, Jun 29, 2018 at 8:43 AM, Jakub Jelinek wrote:
> On Fri, Jun 29, 2018 at 09:09:38AM -0600, Jeff Law wrote:
>> > Btw, running find to search for libtool.m4/ltmain.sh I find extra copies in
>> >
>> > ./libgo/config/ltmain.sh
>> > ./libgo/config/libtool.m4
>> >
>> > which are nearly identical
On 06/29/2018 09:11 AM, Jeff Law wrote:
On 06/27/2018 08:40 PM, Martin Sebor wrote:
On 06/27/2018 03:53 PM, Jeff Law wrote:
On 06/27/2018 09:27 AM, Jakub Jelinek wrote:
On Wed, Jun 27, 2018 at 09:17:07AM -0600, Jeff Law wrote:
About 115 tests fail due to incompatible declarations of
the
On Fri, Jun 29, 2018 at 09:09:38AM -0600, Jeff Law wrote:
> > Btw, running find to search for libtool.m4/ltmain.sh I find extra copies in
> >
> > ./libgo/config/ltmain.sh
> > ./libgo/config/libtool.m4
> >
> > which are nearly identical besides appearantly patched in GO support?
> >
> > Can we
On Fri, Jun 29, 2018 at 09:11:39AM -0600, Jeff Law wrote:
> > I checked all GCC's config logs and although there are 543
> > instances of the -Wbuiltin-declaration-mismatch warning in
> > an x86_64-linux build, none of them is an error and
> > the number is the same as before the patch.
> That's
On Wed, Jun 27, 2018 at 05:47:25PM -0400, Jason Merrill wrote:
> On Thu, Jun 21, 2018 at 2:22 PM, Marek Polacek wrote:
> > The following testcase is rejected because, for this line:
> >
> > bool b = X() ?: false;
> >
> > arg2 is missing and arg1 is a TARGET_EXPR. A TARGET_EXPR is a class
> >
On 06/27/2018 08:40 PM, Martin Sebor wrote:
> On 06/27/2018 03:53 PM, Jeff Law wrote:
>> On 06/27/2018 09:27 AM, Jakub Jelinek wrote:
>>> On Wed, Jun 27, 2018 at 09:17:07AM -0600, Jeff Law wrote:
> About 115 tests fail due to incompatible declarations of
> the built-in functions below (the
On 06/29/2018 02:13 AM, Richard Biener wrote:
> On Mon, Jun 25, 2018 at 1:39 PM Bernhard M. Wiedemann
> wrote:
>>
>> so that gcc builds in a reproducible way
>> in spite of indeterministic filesystem readdir order
>>
>> See https://reproducible-builds.org/ for why this is good.
>>
>> While
On 06/29/2018 07:41 AM, Martin Liška wrote:
> Hi.
>
> It's revert of a hunk that causes a new ICE.
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Ready to be installed?
> Martin
>
> gcc/ChangeLog:
>
> 2018-06-29 Martin Liska
>
> PR ipa/86323
>
On 06/29/2018 07:41 AM, Martin Liška wrote:
> Hi.
>
> It's revert of a hunk that causes a new ICE in IPA summaries.
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Ready to be installed?
> Martin
>
> gcc/ChangeLog:
>
> 2018-06-29 Martin Liska
>
> PR
GCC Maintainers:
The vec_unpackh, vec_unpackl builtins with vector float arguments
unpack the high or low half of a floating point vector and convert the
elements to a vector of doubles. The current implementation of the
builtin for the vector float argument is incorrectly using the vupklsh
On 06/22/2018 10:35 PM, Jeff Law wrote:
> On 05/16/2018 05:53 AM, Martin Liška wrote:
>> On 12/21/2017 10:13 AM, Martin Liška wrote:
>>> On 12/20/2017 06:45 PM, Jakub Jelinek wrote:
Another thing is that the "/" in there is wrong, so
const char dir_separator_str[] = { DIR_SEPARATOR,
Hi.
It's revert of a hunk that causes a new ICE in IPA summaries.
Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
Ready to be installed?
Martin
gcc/ChangeLog:
2018-06-29 Martin Liska
PR ipa/86279
* ipa-pure-const.c (malloc_candidate_p): Revert
Hi.
It's revert of a hunk that causes a new ICE.
Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
Ready to be installed?
Martin
gcc/ChangeLog:
2018-06-29 Martin Liska
PR ipa/86323
* ipa-inline.c (early_inliner): Revert wrongly added ::get call.
The patch fixes PR86321 without regression.
Thanks,
Dominique
Hi all,
In this testcase the user forces an odd register as the starting reg for a
DFmode value.
The output_move_double function tries to store that using an STRD instruction.
But for TARGET_ARM the starting register of an STRD must be an even one.
This is always the case with
Hi all,
This patch adds support for the Arm Cortex-A76 processor in changes.html for
GCC 9.
It enables the AArch64 section of the page and adds the news blob there.
It also adds an entry to the already-existing arm entry.
Ok to commit to CVS (for the aarch64 parts)?
Thanks,
Kyrill
Index:
Hi Martin,
> While looking into opportunities to detect strnlen/strlen coding
> mistakes (pr86199) I noticed a bug in the strnlen implementation
> I committed earlier today that lets a strnlen() result be saved
> and used in subsequent calls to strlen() with the same argument.
> The attached
Richard Sandiford writes:
> This patch is the main part of PR85694. The aim is to recognise at least:
>
> signed char *a, *b, *c;
> ...
> for (int i = 0; i < 2048; i++)
> c[i] = (a[i] + b[i]) >> 1;
>
> as an over-widening pattern, since the addition and shift can be done
> on shorts
On Fri, Jun 29, 2018 at 11:20 AM Richard Sandiford
wrote:
>
> Richard Sandiford writes:
> > This patch adds helper functions for detecting widened operations and
> > generalises the existing code to handle more cases.
> >
> > One of the main changes is to recognise multi-stage type conversions,
On Wed, Jun 20, 2018 at 12:28 PM Richard Sandiford
wrote:
>
> Although the first pattern match wins in the sense that no later
> function can match the *old* gimple statement, it still seems worth
> letting them match the *new* gimple statements, just like we would if
> the original IR had
On Fri, 2018-06-29 at 10:15 +0200, Richard Biener wrote:
> On Fri, 22 Jun 2018, Jan Hubicka wrote:
>
> > Hi,
> > this patch adds dumpfile support for dumps that come in multiple
> > parts. This
> > is needed for WPA stream-out dump since we stream partitions in
> > parallel and
> > the dumps
* testsuite/20_util/add_rvalue_reference/requirements/alias_decl.cc:
Add whitespace to dejagnu directive.
* testsuite/23_containers/array/element_access/at_neg.cc: Likewise.
Tested x86_64-linux, committed to trunk.
commit 0c1a20c4c48f12dfa2797eb193ff9c5edbb4fd8f
Author:
The rationale for the fixinclude ioctl macro wrapper is, as far as I can
tell (https://gcc.gnu.org/ml/gcc-patches/2012-09/msg01619.html)
Fix 2: Add hack for ioctl() on VxWorks.
ioctl() is supposed to be variadic, but VxWorks only has a three
argument version with the third argument of type
On 29/06/18 09:39 +0200, Christophe Lyon wrote:
On Fri, 29 Jun 2018 at 09:21, Jonathan Wakely wrote:
On 29/06/18 08:55 +0200, Christophe Lyon wrote:
>On Mon, 25 Jun 2018 at 18:23, Jonathan Wakely wrote:
>>
>> The additions to were added in 2015 but the new
>> algorithms in were not. This
Hi.
I'm going to install following tested patch (it's already in GCC-6 branch).
Martin
>From 46584361c5f48925395e8155e6a9b809507a25be Mon Sep 17 00:00:00 2001
From: marxin
Date: Fri, 15 Jun 2018 08:51:28 +
Subject: [PATCH] Partial backport r256656
2018-06-15 Martin Liska
Backport from
This patch adds AArch64 patterns for the new AVG_FLOOR/CEIL operations.
AVG_FLOOR is [SU]HADD and AVG_CEIL is [SU]RHADD.
Tested on aarch64-linux-gnu (with and without SVE). OK to install?
Richard
2018-06-29 Richard Sandiford
gcc/
PR tree-optimization/85694
*
This patch adds detection of average instructions:
a = (((wide) b + (wide) c) >> 1);
--> a = (wide) .AVG_FLOOR (b, c);
a = (((wide) b + (wide) c + 1) >> 1);
--> a = (wide) .AVG_CEIL (b, c);
in cases where users of "a" need only the low half of the result,
making the cast to
Richard Sandiford writes:
> This patch adds helper functions for detecting widened operations and
> generalises the existing code to handle more cases.
>
> One of the main changes is to recognise multi-stage type conversions,
> which are possible even in the original IR and can also occur as a
>
2018-06-29 9:28 GMT+02:00 Jakub Jelinek :
> On Thu, Jun 28, 2018 at 07:36:56PM -0700, Steve Kargl wrote:
>> === gfortran Summary ===
>>
>> # of expected passes47558
>> # of unexpected failures6
>> # of expected failures 104
>> # of unsupported tests
On Wed, Jun 27, 2018 at 7:09 AM Kugan Vivekanandarajah
wrote:
>
> Hi Richard,
>
> Thanks for the review,
>
> On 25 June 2018 at 20:20, Richard Biener wrote:
> > On Fri, Jun 22, 2018 at 11:16 AM Kugan Vivekanandarajah
> > wrote:
> >>
> >> gcc/ChangeLog:
> >
> > @@ -1516,6 +1521,114 @@
On Fri, 22 Jun 2018, Jan Hubicka wrote:
> Hi,
> this patch adds dumpfile support for dumps that come in multiple parts. This
> is needed for WPA stream-out dump since we stream partitions in parallel and
> the dumps would come up in random order. Parts are added by new parameter
> that
> is
On Mon, Jun 25, 2018 at 1:39 PM Bernhard M. Wiedemann
wrote:
>
> so that gcc builds in a reproducible way
> in spite of indeterministic filesystem readdir order
>
> See https://reproducible-builds.org/ for why this is good.
>
> While working on the reproducible builds effort, I found that
> when
On Tue, Jun 26, 2018 at 5:43 PM David Malcolm wrote:
>
> This patch adds a concept of nested "scopes" to dumpfile.c's dump_*_loc
> calls, and wires it up to the DUMP_VECT_SCOPE macro in tree-vectorizer.h,
> so that the nested structure is shown in -fopt-info by indentation.
>
> For example, this
Hi.
I would like to link bash-completion pull request that adjusts gcc option
provides:
https://github.com/scop/bash-completion/pull/222
Martin
On Fri, 29 Jun 2018 at 09:21, Jonathan Wakely wrote:
>
> On 29/06/18 08:55 +0200, Christophe Lyon wrote:
> >On Mon, 25 Jun 2018 at 18:23, Jonathan Wakely wrote:
> >>
> >> The additions to were added in 2015 but the new
> >> algorithms in were not. This adds them.
> >>
> >> *
On Thu, 28 Jun 2018, Christophe Lyon wrote:
> On Fri, 22 Jun 2018 at 12:52, Richard Biener wrote:
> >
> >
> > This is the main part to make considering multiple vector sizes based on
> > costs less compile-time costly. It shares dataref analysis and
> > dependence analysis for loop
On Thu, Jun 28, 2018 at 07:36:56PM -0700, Steve Kargl wrote:
> === gfortran Summary ===
>
> # of expected passes47558
> # of unexpected failures6
> # of expected failures 104
> # of unsupported tests 85
>
> FAIL:
On 29/06/18 08:55 +0200, Christophe Lyon wrote:
On Mon, 25 Jun 2018 at 18:23, Jonathan Wakely wrote:
The additions to were added in 2015 but the new
algorithms in were not. This adds them.
* include/experimental/algorithm (sample, shuffle): Add new overloads
using
On Thu, Jun 28, 2018 at 9:06 PM Martin Liška wrote:
>
> Hi.
>
> I'm sending patch for situation where we create a bit-test for
> entire switch. In that case split BB must be redirected so that
> the original switch is a dead code.
>
> Patch can bootstrap on ppc64le-redhat-linux and survives
On Thu, Jun 28, 2018 at 4:29 PM David Malcolm wrote:
>
> On Thu, 2018-06-28 at 13:29 +0200, Richard Biener wrote:
> > On Tue, Jun 26, 2018 at 3:54 PM David Malcolm
> > wrote:
> > >
> > > On Mon, 2018-06-25 at 15:34 +0200, Richard Biener wrote:
> > > > On Wed, Jun 20, 2018 at 6:34 PM David
On Mon, 25 Jun 2018 at 18:23, Jonathan Wakely wrote:
>
> The additions to were added in 2015 but the new
> algorithms in were not. This adds them.
>
> * include/experimental/algorithm (sample, shuffle): Add new overloads
> using per-thread random number engine.
> *
66 matches
Mail list logo
