Re: [PATCH] testsuite: effective_target_march_option: support checking for -march=*

2020-01-18 Thread Mike Stump
> On Jan 18, 2020, at 4:35 AM, Hans-Peter Nilsson wrote: > > testsuite: > * lib/target-supports.exp (effective_target_march_option): New. > > I see no (other) way to, depending on the absence of an option, > add an option for a specific target. Specifically, I don't see > how to do this

Re: [PATCH] analyzer: ensure that all DejaGnu tests have unique names

2020-01-18 Thread Mike Stump
On Jan 17, 2020, at 1:30 PM, David Malcolm wrote: > > Jeff noticed various name collisions of test names in the analyzer > testsuite, due to me using empty comment strings when using line > offsets in test directives. > > This patch adds non-empty comment strings to such DejaGnu directives > thr

Re: [PATCH] RFC: testsuite: add minimium version to dg-require-dot (PR analyzer/93293)

2020-01-18 Thread Mike Stump
On Jan 17, 2020, at 2:07 PM, David Malcolm wrote: > > I ran into difficulties with the Graphviz format changing from under > me during an upgrade, where the new version of "dot" would reject .dot > files generated by the analyzer. > > Thoughts? [ consider this a peanut gallery comment ] Another

Re: [patch, fortran] Fix PR 44960, accepts invalid reference on function

2020-01-18 Thread Tobias Burnus
Hello Thomas, looks good to me. Nice catch the missing "break". Tobias PS: I think it does not exercise a differ code path, but instead of derived types, using a character string works as well. The following is accepted with the unmodified trunk: character(len=4):: str print *, str(1)(1:4)

[PATCH] c++: Fix coroutines.cc build for nvptx-none target

2020-01-18 Thread Jakub Jelinek
Hi! When building offloading cross-compiler from x86_64-linux to nvptx-none, the build fails with: ../../gcc/cp/coroutines.cc: In function 'tree_node* get_fn_local_identifier(tree, const char*)': ../../gcc/cp/coroutines.cc:2255:12: error: expected ';' before 'char' 2255 | sep = "$" |

Re: [patch, fortran] Fix PR 44960, accepts invalid reference on function

2020-01-18 Thread Thomas Koenig
Hello world, I found an ommission in primary.c which prevented issuing a more specific error instead of "syntax error" for the case when a function was declared in an EXTERNAL statement, and I have now gone for the "Unexpected junk after foo" variant. Regression-tested. OK for trunk? Regards

Re: [PATCH] ipa-profile.c: reset call_sums state within ipa-profile.c (PR ipa/93315)

2020-01-18 Thread Jan Hubicka
> > > On Mon, 2020-01-13 at 11:23 +0800, luoxhu wrote: > > > On 2020/1/10 19:08, Jan Hubicka wrote: > > > > OK. You will need to do the obvious updates for Martin's patch > > > > which turned some member functions into static functions. > > > > > > > > Honza > > > > > > Thanks a lot! Rebased & u

New Chinese (traditional) PO file for 'gcc' (version 9.1.0)

2020-01-18 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Chinese (traditional) team of translators. The file is available at: https://translationproject.org/latest/gcc/zh_TW.po (This file, 'gcc-9.1.

New Chinese (traditional) PO file for 'gcc' (version 9.1.0)

2020-01-18 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Chinese (traditional) team of translators. The file is available at: https://translationproject.org/latest/gcc/zh_TW.po (This file, 'gcc-9.1.

[COMMITTED] libgcc: cris: config/cris/arit.c (DS): Apply attribute __fallthrough__.

2020-01-18 Thread Hans-Peter Nilsson
libgcc: * config/cris/arit.c (DS): Apply attribute __fallthrough__. Without this, there are, for each compilation of arit.c, 30ish occurrences of "this statement may fall through [-Wimplicit-fallthrough=]", for lines that look like case 32: DS; case 31: DS; case 30: DS; case 29: DS; N

Re: Analyzer committed to master (was Re: Analyzer status)

2020-01-18 Thread Rainer Orth
Hi David, >> I'm seeing quite a number of failures on Solaris (both sparc and >> x86), >> but also some on 32-bit Linux/x86: >> >> Running target unix/-m32 >> +FAIL: gcc.dg/analyzer/data-model-1.c (test for warnings, line 610) >> +FAIL: gcc.dg/analyzer/data-model-1.c (test for warnings, line 6

[committed][GCC][PATCH][AArch64] Fix unused variable warning breaking bootstrap.

2020-01-18 Thread Tamar Christina
Hi All, This marks the parameter &fi as unused so it doesn't cause a boostrap failure. Bootstrapped aarch64-none-linux-gnu. committed under the obvious rule. Thanks, Tamar gcc/ChangeLog: 2020-01-18 Tamar Christina * config/aarch64/aarch64-sve-builtins-base.cc (memory_vector_mode):

Re: [PATCH] contrib/download_prerequisites: Use http instead of ftp

2020-01-18 Thread Gerald Pfeifer
On Tue, 12 Nov 2019, Janne Blomqvist wrote: > Convert the download_prerequisites script to use http instead of > ftp. This works better with firewalls, proxies, and so on. It's also > faster, a quick test on my system before patch: Plus common web browsers are starting to deprecate and actually re

[COMMITTED] config.gcc : Add crisv32-*-* and cris-*-linux*

2020-01-18 Thread Hans-Peter Nilsson
I'm sorry to say that there's no incentive to maintain crisv32-*-* and cris-*-linux* configurations beyond nostalgia, (and I'm out of that for the moment). Support in the Linux kernel for either applicable CRIS variant (CRIS v10 and CRIS v32) is gone since 2018. Their related part of the cc0 tran

Re: [PATCH] i386: Fix up -fdollars-in-identifiers with identifiers starting with $ in -masm=att (PR target/91298)

2020-01-18 Thread Uros Bizjak
On Sat, Jan 18, 2020 at 1:30 PM Jakub Jelinek wrote: > > Hi! > > In AT&T syntax leading $ is special, so if we have identifiers that start > with dollar, we usually fail to assemble it (or assemble incorrectly). > As mentioned in the PR, what works is wrapping the identifiers inside of > parens, l

Re: [C++ coroutines] Initial implementation pushed to master.

2020-01-18 Thread Iain Sandoe
Hi Jakub, Jakub Jelinek wrote: On Sat, Jan 18, 2020 at 12:53:48PM +, Iain Sandoe wrote: Thanks. Shouldn't this be mentioned in https://gcc.gnu.org/projects/cxx-status.html ? * This is not enabled by default (even for -std=c++2a), it needs -fcoroutines. And, if this is the planned cas

Re: [C++ coroutines] Initial implementation pushed to master.

2020-01-18 Thread Jakub Jelinek
On Sat, Jan 18, 2020 at 12:53:48PM +, Iain Sandoe wrote: Thanks. Shouldn't this be mentioned in https://gcc.gnu.org/projects/cxx-status.html ? > * This is not enabled by default (even for -std=c++2a), it needs -fcoroutines. And, if this is the planned case even for GCC 10 release, I think th

Re: [PATCH] Fix ICE caused by swallowing a token in c_parser_consume_token

2020-01-18 Thread Jakub Jelinek
On Fri, Jan 17, 2020 at 03:11:40PM +0100, Jakub Jelinek wrote: > On Fri, Jan 17, 2020 at 09:07:01AM -0500, Jason Merrill wrote: > > On 12/30/19 3:51 PM, Kerem Kat wrote: > > > +/* { dg-message "expected" "expected" { target *-*-* } .3 } */ > > > > Dejagnu doesn't like this: > > > > ERROR: c-c++-c

[C++ coroutines] Initial implementation pushed to master.

2020-01-18 Thread Iain Sandoe
Hi, Thanks to: * the reviewers, the code was definitely improved by your reviews. * those folks who tested the branch and/or compiler explorer instance and reported problems with reproducers. * WG21 colleagues, especially Lewis and Gor for valuable input and discussions on the

[committed] Reword a comment in varpool_node::ctor_useable_for_folding_p.

2020-01-18 Thread Gerald Pfeifer
2019-01-18 Gerald Pfeifer * varpool.c (ctor_useable_for_folding_p): Fix grammar. --- gcc/ChangeLog | 4 gcc/varpool.c | 7 --- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/gcc/varpool.c b/gcc/varpool.c index e5d632e0eb5..458cdf1bf37 100644 --- a/gcc/varpool.c

Re: [patch, fortran] Fix PR 44960, accepts invalid reference on function

2020-01-18 Thread Thomas Koenig
Am 18.01.20 um 12:44 schrieb Tobias Burnus: function_reference_1.f90:9:8:     9 | print *, foo(1)%a ! { dg-error "Syntax error" }   |    1 Error: Syntax error in expression at (1) The error message is not helpful at all. Well, yes. It is no better than what we currently have for the

[PATCH] testsuite: effective_target_march_option: support checking for -march=*

2020-01-18 Thread Hans-Peter Nilsson
testsuite: * lib/target-supports.exp (effective_target_march_option): New. I see no (other) way to, depending on the absence of an option, add an option for a specific target. Specifically, I don't see how to do this with dg-skip-if and its friends. For gcc.dg/torture/pr26515.c and cris-

[PATCH] i386: Fix up -fdollars-in-identifiers with identifiers starting with $ in -masm=att (PR target/91298)

2020-01-18 Thread Jakub Jelinek
Hi! In AT&T syntax leading $ is special, so if we have identifiers that start with dollar, we usually fail to assemble it (or assemble incorrectly). As mentioned in the PR, what works is wrapping the identifiers inside of parens, like: movl$($a), %eax leaq($a)(,%rdi,4), %ra

Re: [Patch, Fortran + wwwdocs] PR93253 – Document BOZ changes, make it friendlier in legacy code

2020-01-18 Thread Gerald Pfeifer
On Wed, 15 Jan 2020, Tobias Burnus wrote: > Regarding the patch, I enclosed a revised version. I have applied the following markup fix on top. Not a bug (in HTML 5, it would have been in XHTML), though surely not what you intended... Gerald - Log

Re: analyzer branch renamed to "devel/analyzer"

2020-01-18 Thread Gerald Pfeifer
On Wed, 15 Jan 2020, David Malcolm wrote: > The new git server doesn't seem to like such branch names [1], so I'm > now using "devel/analyzer" Do you plan to also document this in git.html (per the message below you quoted ;-)? > remote: *** Shared development branches should be named devel/*,

[wwwdocs] Have the projects list refer to Git instead of SVN for branches.

2020-01-18 Thread Gerald Pfeifer
Pushed. Gerald --- htdocs/projects/index.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/projects/index.html b/htdocs/projects/index.html index 2c6472cc..26f667bd 100644 --- a/htdocs/projects/index.html +++ b/htdocs/projects/index.html @@ -23,7 +23,7 @@ help develo

Re: [patch, fortran] Fix PR 44960, accepts invalid reference on function

2020-01-18 Thread Tobias Burnus
On 1/17/20 11:20 PM, Thomas Koenig wrote: function_reference_1.f90:9:8:     9 | print *, foo(1)%a ! { dg-error "Syntax error" }   |    1 Error: Syntax error in expression at (1) The error message is not helpful at all. I was recently struggling to understand why/where some code was f

[wwwdocs] Condense and trim three aspects of our snapshot instructions.

2020-01-18 Thread Gerald Pfeifer
This includes folding in a reference to our lists overview as a link into some other text, removing a long obsolete reference to age-old versions of the patch command, and skippking details that are covered by the gcc_update script. Pushed. Gerald --- htdocs/snapshots.html | 11 +++ 1 fi

Re: [PATCH] Fix noreorder symbol partitioning reversion.

2020-01-18 Thread Jan Hubicka
> On Thu, 2020-01-16 at 20:43 +0100, Martin Liška wrote: > > Hi. > > > > The patch is fixes a regression in libgcrypt package where > > we incorrectly forget to stream out a definition of a no-reorder symbol. > > It's caused by LTO balanced map reversion, where we do not revert > > also best_noreo

[wwwdocs] Generalize instructions and remove notes on repository mirroring via rsync.

2020-01-18 Thread Gerald Pfeifer
Remove all references how to perform local checkouts, to SVN, and mirroring the repository. Instead generalize descriptions since with the move to Git syncing the repository with rsync and then checking out locally became mostly pointless. Pushed. Gerald --- htdocs/rsync.html | 45 -

[wwwdocs,C++] Update notes on C++17 and C++2a support.

2020-01-18 Thread Gerald Pfeifer
Streamline these two sections, remove references to svn.html, note the latter has landed by now. Pushed. Jonathan, Marek, Jason, it might be good if you could go through the page and see whether there are similar simplifications/updates. Thanks, Gerald --- htdocs/projects/cxx-status.html | 8

Re: [PATCH][GCC][arm] Add CLI and multilib support for Armv8.1-M Mainline MVE extensions

2020-01-18 Thread Jakub Jelinek
On Fri, Jan 17, 2020 at 07:58:46PM +0100, Jakub Jelinek wrote: > One is I wonder if it has been bootstrapped, because at least in a cross > from x86_64-linux to armv7hl-linux-gnueabi I'm seeing: > ../../gcc/config/arm/arm.c: In function ‘void > cmse_nonsecure_call_inline_register_clear()’: > ../..