[PATCH 2/2] Bypass BLKmode before try_const_anchors

2021-03-14 Thread HAO CHEN GUI via Gcc-patches
Hi,     This patch fixes an ICE found by enabling const_anchor for rs6000. The BLKmode constant rtx is sent to try_const_anchors which causes assertion failure in try_const_anchors.     The attachment are the patch diff and change log file.     Bootstrapped and tested on powerpc64le with no

[PATCH 1/2, rs6000] Add const_anchor for rs6000 [PR33699]

2021-03-14 Thread HAO CHEN GUI via Gcc-patches
Hi,     This patch adds const_anchor for rs6000. The const_anchor is used in cse pass.     The attachment are the patch diff and change log file.     Bootstrapped and tested on powerpc64le with no regressions. Is this okay for trunk? Any  recommendations? Thanks a lot. * config/rs6

[PATCH] libstdc++, testsuite, Darwin : Adjust for names used in system headers.

2021-03-14 Thread Iain Sandoe
Hi, What is slightly odd here is that this problem shows up for installed testing, but not when the test-suite is run in-tree. However, the issue is clear (and doesn’t need to depend on figuring out why it doesn’t show in-tree). tested on powerpc-darwin9, x86_64-darwin OK for master? thanks Iai

[PATCH] libstdc++, Darwin, ppc : Add new long double symbols.

2021-03-14 Thread Iain Sandoe
Hi We need to add the symbols for to_chars and from_chars for the long double cases. The testsuite test cases fail to build without this, so there’s no need for an additional test. Tested on powerpc-darwin9. OK for master? thanks Iain libstdc++-v3/ChangeLog: * config/os/bsd/darwin/ppc-

[PATCH] coroutines : Convert await_ready () expressions to bool [PR99047].

2021-03-14 Thread Iain Sandoe
Hi The awaiter.await_ready() should be converted per [expr.await]/3 (3.6) await-ready is the expression e.await_ready(), contextually converted to bool. The conversion had been omitted, fixed as below, tested on x86_64-darwin, x86_64-linux-gnu, and on cppcoro and folly/coroutines. OK for

[PATCH] coroutines : Handle rethrow from unhandled_exception [PR98704].

2021-03-14 Thread Iain Sandoe
Hi Although there is still some discussion in CWG2451 on this, the implementors are agreed on the intent (thus it is wording that is expected to change - the implementations should be brought into sync). tested on x86_64-darwin, x86_64-linux-gnu and with cppcoro and folly/coroutines. OK for mast

[PATCH] coroutines : Handle for await expressions in for stmts [PR98480].

2021-03-14 Thread Iain Sandoe
Hi Apparently, I had a brainstorm when posting patches to cover the cases with await expressions in do {} while; and while {} ; and omitted the for loop case. Fixed thus. tested on x86_64-darwin, x86_64-linux-gnu and with cppcoro and folly/coroutines. OK for master / 10.x? thanks Iain gcc/cp/

[PATCH] coroutines : Avoid generating empty statements [96749].

2021-03-14 Thread Iain Sandoe
Hi In the compiler-only idiom: " a = (target expr creates temp, op uses temp) " the target expression variable needs to be promoted to a frame one (if the expression has a suspend point). However, the only uses of the var are in the second part of the compound expression - and we were creating an

New Swedish PO file for 'gcc' (version 11.1-b20210207)

2021-03-14 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, 'gcc-11.1-b20210207.sv.po'

Re: [PATCH v2 0/5] RISC-V big endian support

2021-03-14 Thread Marcus Comstedt
Hello again Kito. I've now delved a bit deeper into the failure of the testcase gcc.c-torture/compile/pr35318.c on big endian RV32. The point at which big endian diverges from little endian is where process_alt_operands() is processing the "%0" constraint. It calls operands_match_p(), which su

Re: [PATCH v3] x86: Update 'P' operand modifier for -fno-plt

2021-03-14 Thread H.J. Lu via Gcc-patches
On Sun, Mar 14, 2021 at 12:43 PM Uros Bizjak wrote: > > On Sun, Mar 14, 2021 at 8:14 PM H.J. Lu wrote: > > > > Done. Here is the updated patch. Tested on Linux/x86-64. OK for > > > > master? > > > > > > I don't understand the purpose of the current_output_insn check and I > > > don't know if

[patch, fortran] Fix PR 99345, ICE with DO loop checking

2021-03-14 Thread Thomas Koenig via Gcc-patches
Hello world, the attached, rather obvious patch fixes an ICE on valid which came about because I did not handle EXEC_IOLENGTH as start of an I/O statement when checking for the DO loop variable. This is an 11 regression. Thanks to Harald for reducing this down to the bare minimum. Regression-te

Aw: Re: [PATCH] PR fortran/99112 - [11 Regression] ICE with runtime diagnostics for SIZE intrinsic function

2021-03-14 Thread Harald Anlauf via Gcc-patches
Hi Tobias, > I do not see any reason why 'tmp' is replaced by 'temp' in this > code. Also for doing patch archeology, it helps if there are no > changes unless it makes sense. Adding an -e- does not count ;-) > > Hence, OK with that change. I've corrected that. This also reduces the size of the

Re: [PATCH v3] x86: Update 'P' operand modifier for -fno-plt

2021-03-14 Thread Uros Bizjak via Gcc-patches
On Sun, Mar 14, 2021 at 8:14 PM H.J. Lu wrote: > > > Done. Here is the updated patch. Tested on Linux/x86-64. OK for master? > > > > I don't understand the purpose of the current_output_insn check and I > > don't know if the usage of current_output_insn is correct. The > > comments are not help

Re: [PATCH v3] x86: Update 'P' operand modifier for -fno-plt

2021-03-14 Thread H.J. Lu via Gcc-patches
On Sun, Mar 14, 2021 at 11:49 AM Uros Bizjak wrote: > > On Sat, Mar 13, 2021 at 8:39 PM H.J. Lu wrote: > > > > On Fri, Mar 12, 2021 at 8:37 AM Uros Bizjak wrote: > > > > > > On Fri, Mar 12, 2021 at 2:20 PM H.J. Lu wrote: > > > > > > > > On Thu, Mar 11, 2021 at 11:21 PM Uros Bizjak wrote: > > >

Re: [PATCH v3] x86: Update 'P' operand modifier for -fno-plt

2021-03-14 Thread Uros Bizjak via Gcc-patches
On Sat, Mar 13, 2021 at 8:39 PM H.J. Lu wrote: > > On Fri, Mar 12, 2021 at 8:37 AM Uros Bizjak wrote: > > > > On Fri, Mar 12, 2021 at 2:20 PM H.J. Lu wrote: > > > > > > On Thu, Mar 11, 2021 at 11:21 PM Uros Bizjak wrote: > > > > > > > > On Thu, Mar 11, 2021 at 11:22 PM H.J. Lu wrote: > > > > >

Re: [PATCH] i386: Hopefully last set of -mavx512vl -mno-avx512bw fixes [PR99321]

2021-03-14 Thread Uros Bizjak via Gcc-patches
On Fri, Mar 12, 2021 at 11:56 PM Jakub Jelinek wrote: > > On Fri, Mar 12, 2021 at 07:52:16PM +0100, Uros Bizjak via Gcc-patches wrote: > > > I can test it on avx512{bw,vl,dq} hw tonight if you want. > > > > I'm testing the patch on avx2 hw, which is not representative of this > > change. So if you

Re: [PATCH] Fix ICE: in function_and_variable_visibility, at ipa-visibility.c:795 (PR99466)

2021-03-14 Thread Iain Buclaw via Gcc-patches
Excerpts from Iain Sandoe's message of March 13, 2021 6:09 pm: > Hi Iain, > > Iain Buclaw via Gcc-patches wrote: > >> This patch fixes an ICE caused by emutls routines generating a weak, >> non-public symbol for storing the initializer of a weak TLS variable. >> >> In get_emutls_init_templ_addr,

Early ping: Re: [Patch] Fortran: Fix func decl mismatch [PR93660]

2021-03-14 Thread Tobias Burnus
Early ping – and minor post script: + hidden_typelist = TREE_CHAIN (hidden_typelist); This change is to avoid running into the ICE: gcc_assert (hidden_typelist == NULL_TREE || TREE_VALUE (hidden_typelist) == void_type_node); The purpose of this assert is to check that t

Re: [PATCH] PR fortran/99112 - [11 Regression] ICE with runtime diagnostics for SIZE intrinsic function

2021-03-14 Thread Tobias Burnus
Hi Harald, hi Paul, On 13.03.21 09:58, Paul Richard Thomas via Fortran wrote: I am not sure of the etiquette for this - it looks OK to me :-) :-) On Fri, 12 Mar 2021 at 21:20, Harald Anlauf via Fortran the addition of runtime checks for the SIZE intrinsic created a regression that showed