[PATCH, PR d/90601] Committed fix for ICE: gimplification failed (gimplify.c at 13436)

2019-08-10 Thread Iain Buclaw
Hi, The expression that caused the ICE ++(a += 1.0); The D front-end rewrites and applies implicit type conversions so the expression gets simplified as (int)((double) a += 1.0) += 1 The codegen pass would subsequently generate the following invalid code (int)(double)

Re: Using gcc/ChangeLog instead of gcc/testsuite/ChangeLog?

2019-08-10 Thread Hans-Peter Nilsson
> From: Jakub Jelinek > Date: Sat, 10 Aug 2019 12:12:46 +0200 > I ran the gcc/ subdirectory ChangeLogs through following script that doesn't > seem to have false positives ATM except one... > # Date not separated from name by two spaces, but just one. [...] > # Email not wrapped in <>s. > grep

Re: [committed, wwwdocs] OpenACC status for GCC 9

2019-08-10 Thread Gerald Pfeifer
On Wed, 15 May 2019, Thomas Schwinge wrote: > Sure, and thanks for having a look. > >> On this I'd omit the second link if that's okay with you? (I'm >> happy to make the change, wanted to check in with you first.) > > If you'd like to, then please do. Done thusly. Committed. Gerald Index: ch

Re: [wwwdocs] nds32 documentation - remove broken reference

2019-08-10 Thread Gerald Pfeifer
On Tue, 30 Jul 2019, Shiva Chen wrote: > The update link will be > http://www.andestech.com/en/products-solutions/product-documentation/ Thank you. Restored via the patch below that I just committed. Gerald Index: readings.html ===

Re: [PATCH,fortran] PR 91413 Generate warning when making array static

2019-08-10 Thread Steve Kargl
On Sat, Aug 10, 2019 at 11:34:15PM +0300, Janne Blomqvist wrote: > When moving a local variable from the stack to static storage, the > procedure is no longer safe to be called recursively or concurrently > from multiple threads. Thus generate a warning when this is done. > Also double the default

Re: [wwwdocs] gcc-9/changes.html - Mention new OpenRISC backend

2019-08-10 Thread Gerald Pfeifer
On Wed, 10 Apr 2019, Stafford Horne wrote: > Committed. Thanks! > It turns out I do have CVS access, which is granted along with SVN access. Yes, you do, and you are very welcome making changes like this (plus anything that remotely qualifies as obvious). :-) If you have a recommendation (or pa

[PATCH,fortran] PR 91413 Generate warning when making array static

2019-08-10 Thread Janne Blomqvist
When moving a local variable from the stack to static storage, the procedure is no longer safe to be called recursively or concurrently from multiple threads. Thus generate a warning when this is done. Also double the default limit for switching from stack to static. Regtested on x86_64-pc-linux-

C++ PATCH for more checking of DECL_MUTABLE_P

2019-08-10 Thread Marek Polacek
Some preparatory work before adding constinit. mutable is only appropriate for FIELD_DECLs (when they're not static), but we've never made sure that we're not setting DECL_MUTABLE_P on a different _DECL. I mean to use DECL_LANG_FLAG_0 in a VAR_DECL for DECL_DECLARED_CONSTINIT_P. Bootstrapped/reg

Re: [PATCH,Fortran] -- Tidy up the BOZ rewrite

2019-08-10 Thread Steve Kargl
On Sat, Aug 10, 2019 at 10:54:30AM -0700, Steve Kargl wrote: > On Sat, Aug 10, 2019 at 07:29:10PM +0200, Thomas Koenig wrote: > > Hi Steve, > > > > > OK to commit? > > > > OK with > > > > %s/rational/relational/g > > > > and with Mark's nit. > > > > Thanks a lot for going down that road. I li

Re: [PATCH,Fortran] -- Tidy up the BOZ rewrite

2019-08-10 Thread Steve Kargl
On Sat, Aug 10, 2019 at 07:29:10PM +0200, Thomas Koenig wrote: > Hi Steve, > > > OK to commit? > > OK with > > %s/rational/relational/g > > and with Mark's nit. > > Thanks a lot for going down that road. I like the approach of > -fallow-invalid-boz to downgrade the error to a warning, which >

Re: [PATCH,Fortran] -- Tidy up the BOZ rewrite

2019-08-10 Thread Thomas Koenig
Hi Steve, OK to commit? OK with %s/rational/relational/g and with Mark's nit. Thanks a lot for going down that road. I like the approach of -fallow-invalid-boz to downgrade the error to a warning, which then cannot be turned off. Regards Thomas

Re: Using gcc/ChangeLog instead of gcc/testsuite/ChangeLog?

2019-08-10 Thread Segher Boessenkool
Hi! On Sat, Aug 10, 2019 at 12:12:46PM +0200, Jakub Jelinek wrote: > On Sat, Aug 10, 2019 at 08:53:45AM +0200, Hans-Peter Nilsson wrote: > > Has there been a change of policy so it's a valid option to use > > gcc/ChangeLog for testsuite changes? I was about to move a > > semi-randomly spotted mis

[PATCH, PR d/91238] Committed fix for ICE in add_expr, at tree.c:7794

2019-08-10 Thread Iain Buclaw
Hi, This patch fixes an ICE in the D compiler, where certain kinds of lowered code mean the address of a CALL_EXPR is required. When that happens, use a TARGET_EXPR to hold the result, and take the address of that instead. Bootstrapped and regression tested on x86_64-linux-gnu. Committed to tru

[committed] declare target tweaks + device_type clause parsing

2019-08-10 Thread Jakub Jelinek
Hi! The following patch adds parsing of the device_type clause (though it is in the end ignored right now, I have some omp-lang questions on what exactly is the semantics supposed to be, where exactly we should apply the excluded definitions), plus declare target is now explicitly allowed to be ne

[committed] Fix mmx usadv8qi expander (PR target/91408)

2019-08-10 Thread Jakub Jelinek
Hi! As mentioned in the PR, both instructions this expander uses use register_mmxmem_operand predicate in the operands operands[2] and [3] are passed to, and register_mmxmem_operand is for TARGET_MMX_WITH_SSE just register_operand. Thus, the following patch adjusts the expander's predicates to mat

Re: Using gcc/ChangeLog instead of gcc/testsuite/ChangeLog?

2019-08-10 Thread Jakub Jelinek
On Sat, Aug 10, 2019 at 08:53:45AM +0200, Hans-Peter Nilsson wrote: > Has there been a change of policy so it's a valid option to use > gcc/ChangeLog for testsuite changes? I was about to move a > semi-randomly spotted misplaced entry, and when checking if > there were others, I noticed that there

Re: [PATCH v6][C][ADA] use function descriptors instead of trampolines in C

2019-08-10 Thread Uecker, Martin
Am Freitag, den 09.08.2019, 16:45 -0600 schrieb Jeff Law: > On 6/24/19 3:35 PM, Uecker, Martin wrote: > > > > > > Hi, > > > > here is a new version of this patch. It makes "-fno-trampolines" > > work for C which then makes it possible to use nested functions > > without executable stack. The onl