This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Add LVAL argument to c_fully_fold* and propagate it through (PR c/66618, PR c/69960)

From: Tom de Vries <Tom_deVries at mentor dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: Marek Polacek <polacek at redhat dot com>, Joseph Myers <joseph at codesourcery dot com>, <gcc-patches at gcc dot gnu dot org>
Date: Tue, 26 Dec 2017 01:48:50 +0100
Subject: Re: [PATCH] Add LVAL argument to c_fully_fold* and propagate it through (PR c/66618, PR c/69960)
Authentication-results: sourceware.org; auth=none
References: <20171108162245.GV14653@tucnak> <alpine.DEB.2.20.1711081712590.11470@digraph.polyomino.org.uk> <20171108173821.GX14653@tucnak> <20171108175133.GJ1743@redhat.com> <20171108175755.GY14653@tucnak> <20171109105434.GZ14653@tucnak>

On 11/09/2017 11:54 AM, Jakub Jelinek wrote:

On Wed, Nov 08, 2017 at 06:57:55PM +0100, Jakub Jelinek wrote:

On Wed, Nov 08, 2017 at 06:51:34PM +0100, Marek Polacek wrote:

Ok, so like this if it passes bootstrap/regtest?

Changes from the last patch:
1) false instead of lval for COMPOUND_EXPR and *COND_EXPR op1/op2


So...


Oops, I've hand-edited it in the patch and then regenerated the patch
after doing there more changes.

Here is updated patch with that in again:


FYI, bootstrapped/regtested on x86_64-linux and i686-linux successfully.

This caused: PR83590 - [nvptx] openacc reduction C regressions (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83590 ).


Thanks,
- Tom

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]