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

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] 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,

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 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 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 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

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

[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

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

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

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'

[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

[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 : 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 : 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] 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] 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 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 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