Re: [PATCH] c++: Fix ICE in tsubst_default_argument [PR92010]

2020-03-22 Thread Patrick Palka via Gcc-patches
On Sun, 22 Mar 2020, Patrick Palka wrote: > This patch relaxes an assertion in tsubst_default_argument that exposes a > latent > bug in how we substitute an array type into a cv-qualified wildcard function > parameter type. Concretely, the latent bug is that given the function > template > >

[RS6000] PR94145, make PLT loads volatile

2020-03-22 Thread Alan Modra via Gcc-patches
On Wed, Mar 18, 2020 at 04:53:59PM -0500, Segher Boessenkool wrote: > Could you please send a new patch (could be the same patch even) that > is easier to review for me? The PLT is volatile. On PowerPC it is a bss style section which the dynamic loader initialises to point at resolver stubs

Re: [PATCH] Support the new ("v0") mangling scheme in rust-demangle.

2020-03-22 Thread Eduard-Mihai Burtescu
Ping: https://gcc.gnu.org/pipermail/gcc-patches/2020-March/542012.html Thanks, - Eddy B. On Fri, Mar 13, 2020, at 10:28 PM, Eduard-Mihai Burtescu wrote: > This is the libiberty (mainly for binutils/gdb) counterpart of > https://github.com/alexcrichton/rustc-demangle/pull/23. > > Relevant links

[PATCH] c++: Fix ICE in tsubst_default_argument [PR92010]

2020-03-22 Thread Patrick Palka via Gcc-patches
This patch relaxes an assertion in tsubst_default_argument that exposes a latent bug in how we substitute an array type into a cv-qualified wildcard function parameter type. Concretely, the latent bug is that given the function template template void foo(const T t); one would expect the type

Re: [PATCH v2 7/9] aarch64: Adjust result of aarch64_gen_compare_reg

2020-03-22 Thread Richard Henderson
On 3/22/20 2:55 PM, Segher Boessenkool wrote: > Maybe this stuff would be simpler (and more obviously correct) if it > was more explicit CC_REGNUM is a fixed register, and the code would use > it directly everywhere? Indeed the biggest issue I have in this patch is what CC_MODE to expose from the

Re: [PATCH v2 7/9] aarch64: Adjust result of aarch64_gen_compare_reg

2020-03-22 Thread Segher Boessenkool
Hi! On Fri, Mar 20, 2020 at 07:42:29PM -0700, Richard Henderson via Gcc-patches wrote: > @@ -2382,7 +2382,7 @@ aarch64_gen_compare_reg_maybe_ze (RTX_CODE code, rtx x, > rtx y, > cc_mode = CC_SWPmode; > cc_reg = gen_rtx_REG (cc_mode, CC_REGNUM); > emit_set_insn (cc_reg,

[PATCH] c++: Avoid a suspicious -Wnoexcept warning [PR93805]

2020-03-22 Thread Patrick Palka via Gcc-patches
In this PR we're emitting -Wnoexcept warnings about potentially-throwing NSDMIs when computing the noexcept specification of a class's defaulted default constructor. Alhough these warnings are in some sense valid, this patch takes the route of suppressing them, because: 1. the warning message

Re: [PATCH v2 3/9] aarch64: Add cmp_*_carryinC patterns

2020-03-22 Thread Richard Henderson via Gcc-patches
On 3/22/20 12:30 PM, Segher Boessenkool wrote: > Hi! > > On Fri, Mar 20, 2020 at 07:42:25PM -0700, Richard Henderson via Gcc-patches > wrote: >> Duplicate all usub_*_carryinC, but use xzr for the output when we >> only require the flags output. The signed versions use sign_extend >> instead of

Re: [PATCH v2 3/9] aarch64: Add cmp_*_carryinC patterns

2020-03-22 Thread Segher Boessenkool
Hi! On Fri, Mar 20, 2020 at 07:42:25PM -0700, Richard Henderson via Gcc-patches wrote: > Duplicate all usub_*_carryinC, but use xzr for the output when we > only require the flags output. The signed versions use sign_extend > instead of zero_extend for combine's benefit. You actually use

Re: [PATCH] Save ref->speculative_id before clone_reference.

2020-03-22 Thread Jan Hubicka
> Hi. > > Similarly to other ipa_ref field, we must preserve the values > before create_reference is called. It's due to fact that ipa_ref > is a pointer to a vector that can be relocated to a different location. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > >

New Swedish PO file for 'gcc' (version 10.1-b20200209)

2020-03-22 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 Swedish team of translators. The file is available at: https://translationproject.org/latest/gcc/sv.po (This file,

Re: [PR 94044] ICE with sizeof & argument pack

2020-03-22 Thread Jim Wilson
On Fri, Mar 20, 2020 at 8:41 AM Nathan Sidwell wrote: > If it could be tested on arm &| riscv, that'd be additional verification. I did riscv testing, both cross and native, and didn't see any new problems with the patch. Jim

Re: [PING^2][PATCH] Fix documentation of -mpoke-function-name ARM option

2020-03-22 Thread Jérémy Lefaure
Hi Wilco, On Mon, Mar 09, 2020 at 05:53:41PM +, Wilco Dijkstra wrote: > Hi, > > There is no single PC offset that is correct given CPUs may use different > offsets. Isn't it always an offset of 8 in ARM mode and 4 bytes in Thumb mode ? At least in ARMv7 and in AArch32 state in ARMv8 ? >

Re: [PATCH] Test for sigsetjmp support in analyzer tests requiring that feature.

2020-03-22 Thread David Malcolm via Gcc-patches
On Sun, 2020-03-22 at 11:31 -0600, Sandra Loosemore wrote: > The new-ish analyzer test cases sigsetjmp-5.c and sigsetjmp-6.c were > failing on nios2-elf and probably other newlib targets due to lack > of > support for sigsetjmp. Sorry about the breakage. > I didn't see a suitable existing >

[PATCH] Save ref->speculative_id before clone_reference.

2020-03-22 Thread Martin Liška
Hi. Similarly to other ipa_ref field, we must preserve the values before create_reference is called. It's due to fact that ipa_ref is a pointer to a vector that can be relocated to a different location. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be

Re: [Patch, 10 Regression] fortran: ICE in gfc_match_assignment PR93600

2020-03-22 Thread Thomas Koenig via Gcc-patches
Hi Mark, Please find attached a fix for PR93600. This builds on the patch originally submitted to the PR by Steve Kargl, the dreaded "Unclassifiable statement error" is replaced by the correct error message. It would have been posted earlier had not one of the test cases failed as a result

Re: [Patch, 9/10 Regression] fortran: ICE in build_entry_thunks PR93814

2020-03-22 Thread Thomas Koenig via Gcc-patches
Hi Mark, Please find attached Steve Kargl's fix for PR93814. The attached patch does not match the ChangeLog; it seems to be for PR93484. Regards Thomas

[PATCH] Test for sigsetjmp support in analyzer tests requiring that feature.

2020-03-22 Thread Sandra Loosemore
The new-ish analyzer test cases sigsetjmp-5.c and sigsetjmp-6.c were failing on nios2-elf and probably other newlib targets due to lack of support for sigsetjmp. I didn't see a suitable existing effective-target test for this, so I added one. OK to commit? -Sandra commit

New Spanish PO file for 'gcc' (version 10.1-b20200209)

2020-03-22 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 Spanish team of translators. The file is available at: https://translationproject.org/latest/gcc/es.po (This file,

[committed] d: Generate phony targets for content imported files (PR93038)

2020-03-22 Thread Iain Buclaw via Gcc-patches
Hi, This patch is an addition to the last change, which started including content imported files in the make dependency list. Now phony targets are also written out if -MP is given. Bootstrapped and regression tested on x86_64-linux-gnu. Committed to trunk. Regards Iain. --- gcc/d/ChangeLog:

[committed] testsuite: Fix lambda-vis.C for targets with user label prefix '_'.

2020-03-22 Thread Iain Sandoe
Hi The lambda-vis.C test fails everywhere for targets that use a ‘_’ as USER_LABEL_PREFIX. This prepends an optional match for the additional USER_LABEL_PREFIX to the scan assembler checks. Tested on x86_64-darwin, and linux, applied to master as obvious, thanks Iain 2020-03-22 Iain