[PATCH] PR fortran/98017 - [8/9/10/11 Regression] Suspected regression using PACK

2020-11-28 Thread Harald Anlauf
When substituting an array-valued character parameter variable, the call to gfc_copy_expr returns character length 1. Fix up the resulting length. I could not figure out whether this is a bug or a feature of gfc_copy_expr. But the fix to simplify_parameter_variable would not do any harm in any ca

[PATCH] d: Add darwin support for D language front-end

2020-11-28 Thread Iain Buclaw via Gcc-patches
Hi, This patch adds necessary predefined version symbols and support for moduleinfo sections for darwin to allow testing libphobos support. OK for mainline? Regards Iain. --- gcc/ChangeLog: * config.gcc (*-*-darwin*): Set d_target_objs and target_has_targetdm. * config/elfos.h

Re: [PATCH 00/31] VAX: Bring the port up to date (yes, MODE_CC conversion is included)

2020-11-28 Thread Paul Koning via Gcc-patches
> On Nov 25, 2020, at 12:07 PM, Maciej W. Rozycki wrote: > > On Mon, 23 Nov 2020, Paul Koning wrote: > >>> ... > >> I've hacked together a primitive newlib based "bare metal" execution >> test setup that uses SIMH, but it's not a particularly clean setup. >> And it hasn't been posted, I h

Re: H8 cc0 conversion

2020-11-28 Thread Paul Koning via Gcc-patches
> On Nov 25, 2020, at 5:05 PM, Hans-Peter Nilsson wrote: > > On Tue, 24 Nov 2020, Eric Botcazou wrote: > >>> I'm intested in any notes, however vague, on that matter. I was >>> a bit surprised to see that myself...that is, after fixing >>> *some* related regressions, like the one in combine.

[PATCH] configure: Support building D front-end on *-*-darwin*

2020-11-28 Thread Iain Buclaw via Gcc-patches
Hi, The bootstrap has been succeeding for some time now, there's no need to set it as an unsupported language. OK for mainline? Regards Iain. --- ChangeLog: PR d/87788 * configure.ac: Don't disable D for *-*-darwin*. * configure: Regenerate. --- configure| 3 --- c

[PATCH] d: Add freebsd support for D compiler and runtime

2020-11-28 Thread Iain Buclaw via Gcc-patches
Hi, This patch adds the necessary version conditions and configure rules in place to allow building the D compiler on FreeBSD. Running the testsuite on both i386 and x86_64, all tests except for one passes (gdc.test/runnable/test17338.d, though the problem appears to be the linker producing a cor

[c-family] Improve support of simple bit-fields in -fdump-ada-spec

2020-11-28 Thread Eric Botcazou
This fixes an issue with nested structures and adds an Alignment clause to counter the effect of the Pack aspect. Tested on x86_64/Linux, applied on the mainline. 2020-11-28 Eric Botcazou c-family/ * c-ada-spec.c (dump_nested_type) : Remove obsolete code. (dump_ada_structure): A

Go patch committed: Avoid follow-on errors for bad types

2020-11-28 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend marks bad types are erroneous, to avoid generating further errors. This required some code using array types to check for errors. This is for https://golang.org/issue/19880. This requires updating one of the tests in the testsuite. Bootstrapped and ran Go testsuite o

Go patch committed: Better error for x, x := 1, 2

2020-11-28 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend gives a better error message when the same variable is declared multiple times on the left hand side of a := statement. Was assign.go:59:28: error: multiple assignments to x Now assign.go:59:28: error: ‘x’ repeated on left side of := Bootstrapped and ran Go t

Re: [PATCH] expansion: Improve double-word modulo by certain constant divisors [PR97459]

2020-11-28 Thread Thomas Koenig via Gcc-patches
Hello Jakub, thanks a lot for taking this on! As far as I can tell, the patch works very well, and really speeds up things. As (somewhat confusingly) discussed in the PR, there are a couple of things that could still be done incrementally with this method. Fist, it can be combined with, or eve

[SPARC] Fix PR target/97939

2020-11-28 Thread Eric Botcazou
I overlooked the little dance around 4096 that the add/sub instructions do on the SPARC when implementing the overflow arithmetic operations. It cannot be done for unsigned overflow, but it can be done for signed overflow. Tested on SPARC64/Linux and SPARC/Solaris, applied on the mainline, 10

Re: [patch, fortran] Correct fndecls for some library functions

2020-11-28 Thread Thomas Koenig via Gcc-patches
Am 27.11.20 um 16:46 schrieb Tobias Burnus: Hi Thomas, On 25.11.20 12:58, Tobias Burnus wrote: On 15.11.20 18:52, Thomas Koenig via Fortran wrote: +#define ADD_CHAR(c) do { *fp++ = c; *fp++ = ' '; } while(0) ... +  ADD_CHAR ('.');  /* Function return.  */ Shouldn't this be ".c" instead of ".

Re: [PATCH] Objective-C++ : Allow prefix attrs on linkage specs.

2020-11-28 Thread Iain Sandoe
(resending - this didn’t seem to reach gcc-patches@) Jason Merrill wrote: On Mon, Nov 23, 2020 at 8:52 AM Iain Sandoe wrote: Jason Merrill wrote: (NOTE: likewise, ^~~ starting indent is below ‘int’ for a fixed spacing font) === I’m inclined to think that the second is more useful, but