[committed] Fix doubled indefinite articles, mostly in comments.

2020-03-14 Thread Jakub Jelinek via Gcc-patches
Hi! Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux and committed to trunk as obvious. 2020-03-14 Jakub Jelinek * gimple-fold.c (gimple_fold_builtin_strncpy): Change "a an" to "an" in a comment. * hsa-common.h (is_a_helper): Likewise. * tree

[PATCH] reassoc: Avoid code generation to depend on hash_map traversal [PR94166]

2020-03-14 Thread Jakub Jelinek via Gcc-patches
Hi! On the following testcase, if there is ASLR, the compiler generates different code each time (out of 1000 invocations 994 unique assembler contents). The problem is that undistribute_bitref_for_vector uses a hash_map from a tree (SSA_NAME) to a vector and such a hash_map is by default doing p

[PATCH] tree-inline: Fix a -fcompare-debug issue in the inliner [PR94167]

2020-03-14 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase fails with -fcompare-debug. The problem is that bar is marked as address_taken only with -g and not without. I've tracked it down to insert_init_stmt calling gimple_regimplify_operands even on DEBUG_STMTs. That function will just insert normal stmts before the DEBUG_ST

[PATCH v3] debug/93751 Generate DIEs for external variables with -g1

2020-03-14 Thread Alexey Neyman
On 2/28/20 1:50 PM, Jason Merrill wrote: On 2/28/20 4:12 PM, Alexey Neyman wrote: On 2/28/20 12:28 PM, Jason Merrill wrote: On 2/28/20 2:11 PM, Alexander Monakov wrote: Hm. So apparently at -g1 we don't emit type information for functions, and gdb sees each function as 'void foo()' regardless

[PATCH] c++: Fix parsing of invalid enum specifiers [PR90995]

2020-03-14 Thread Jakub Jelinek via Gcc-patches
Hi! The testcase shows some accepts-invalid (the ones without alignas) and ice-on-invalid-code (the ones with alignas) cases. If the enum doesn't have an underlying type and is not a definition, the caller retries to parse it as elaborated type specifier. E.g. for enum struct S s it will then pedw

Re: [PATCH v3] debug/93751 Generate DIEs for external variables with -g1

2020-03-14 Thread Alexander Monakov via Gcc-patches
On Sat, 14 Mar 2020, Alexey Neyman wrote: > Attached is a patch that does it: at -g1, the type attributes are not > generated. Two small issues I pointed out the last time are still present: https://gcc.gnu.org/legacy-ml/gcc-patches/2020-02/msg01646.html (I did not review the new patch on a more

[PATCH] rs6000/test: Fix selector in fold-vec-mule-misc.c

2020-03-14 Thread Segher Boessenkool
Run tests should use vmx_hw, not just powerpc_altivec_ok. Committed. Segher gcc/testsuite/ PR target/94176 * gcc.target/powerpc/fold-vec-mule-misc.c: Use vmx_hw selector. --- gcc/testsuite/ChangeLog | 5 + gcc/testsuite/gcc.target/powerpc/fold

[PATCH libphobos] Committed merge with upstream druntime 7915b6a3

2020-03-14 Thread Iain Buclaw via Gcc-patches
Hi, This patch merges the libphobos implementation with upstream druntime 7915b6a3. Includes port fixes for Musl on ARM, AArch64, and SystemZ targets. Bootstrapped and tested on x86_64-linux-gnu, and committed to trunk. Regards Iain. --- diff --git a/libphobos/libdruntime/MERGE b/libphobos/

[pushed] c++: Fix CTAD with multiple-arg ctor template [93248].

2020-03-14 Thread Jason Merrill via Gcc-patches
When cp_unevaluated_operand is set, tsubst_decl thinks that if it sees a PARM_DECL that isn't already in local_specializations, we're in a decltype in a trailing return type or some such, and so we only want a substitution for a single PARM_DECL. In this case, we want the whole chain, so make sure

[pushed] c++: Find parameter pack in typedef in lambda [92909].

2020-03-14 Thread Jason Merrill via Gcc-patches
find_parameter_packs_r doesn't look through typedefs, which is normally correct, but that means we need to handle their declarations specially. Tested x86_64-pc-linux-gnu, applying to trunk. gcc/cp/ChangeLog 2020-03-14 Jason Merrill PR c++/92909 * pt.c (find_parameter_packs_r)

[pushed] c++: Fix ICE-after-error on partial spec [92068]

2020-03-14 Thread Jason Merrill via Gcc-patches
Here the template arguments for the partial specialization are valid arguments for the template, but not for a partial specialization, because 'd' can never be deduced to anything other than an empty pack. Tested x86_64-pc-linux-gnu, applying to 8/9/10. gcc/cp/ChangeLog 2020-03-14 Jason Merrill

[committed] wwwdocs: Slightly shorten/simplify our notes on mailing list spam.

2020-03-14 Thread Gerald Pfeifer
A mail by Jonathan made me look at that page again after a looong while, and I noticed two simplifications. Pushed (though the machinery to update the web site did not appear to be working then, but I manually re-run it now). Gerald patch commit 09a7a9579b59619ea3f601b821ec8b9dd3fe708e

[PATCH v4] debug/93751 Generate DIEs for external variables with -g1

2020-03-14 Thread Alexey Neyman
On 3/14/20 4:53 AM, Alexander Monakov wrote: On Sat, 14 Mar 2020, Alexey Neyman wrote: Attached is a patch that does it: at -g1, the type attributes are not generated. Two small issues I pointed out the last time are still present: https://gcc.gnu.org/legacy-ml/gcc-patches/2020-02/msg01646.html

[committed] tree-nested: Fix handling of *reduction clauses with C array sections [PR93566]

2020-03-14 Thread Jakub Jelinek via Gcc-patches
Hi! tree-nested.c didn't handle C array sections in {,task_,in_}reduction clauses. The following patch implements that, bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk so far. 2020-03-14 Jakub Jelinek PR middle-end/93566 * tree-nested.c (convert_nonl

[PATCH PR94125]Update post order number for merged SCC

2020-03-14 Thread bin.cheng via Gcc-patches
Hi, This simple patch fixes PR94125 by updating post order number for merged SCC. The root cause is after computing SCC with runtime alias edges skipped, the post order info is changed and it's possible a partition is scheduled after another where should be scheduled before. Note that updating to