[PATCH v3 2/2] MIPS: add builtime option for -mcompact-branches

2021-02-19 Thread YunQiang Su
For R6+ target, it allows to configure gcc to use compact branches only. gcc/ChangeLog: * config.gcc: add -with-compact-branches=policy build option. * doc/install.texi: Likewise. --- gcc/ChangeLog| 5 + gcc/config.gcc | 12 +++- gcc/doc/install.texi | 1

[PATCH v3 1/2] MIPS: Not trigger error for pre-R6 and -mcompact-branches=always

2021-02-19 Thread YunQiang Su
For MIPSr6, we may wish to use compact-branches only. Currently, we have to use `always' option, while it is mark as conflict with pre-R6. cc1: error: unsupported combination: ‘mips32r2’ -mcompact-branches=always Just ignore -mcompact-branches=always for pre-R6. This patch also defines __mip

Re: [PATCH v7] Practical improvement to libgcc complex divide

2021-02-19 Thread Joseph Myers
On Fri, 19 Feb 2021, Patrick McGehearty via Gcc-patches wrote: > > Here you're properly computing the mapping from mode to float.h macro > > prefix (though I think "modename" is a confusing name for the variable > > used for float.h macro prefixes; to me, "modename" suggests the names such > > as

Re: [PATCH v7] Practical improvement to libgcc complex divide

2021-02-19 Thread Patrick McGehearty via Gcc-patches
Comments inline On 2/19/2021 3:27 PM, Joseph Myers wrote: On Tue, 2 Feb 2021, Patrick McGehearty via Gcc-patches wrote: if (mode == TYPE_MODE (double_type_node)) - ; /* Empty suffix correct. */ + { + ; /* Empty suffix correct. */ + memcpy

c++: Incorrect module-number ordering [PR 98741]

2021-02-19 Thread Nathan Sidwell
One of the very strong invariants in modules is that module numbers are allocated such that (other than the current TU), all imports have lesser module numbers, and also that the binding vector is only appended to with increasing module numbers. This broke down when module-directives became a

Re: [PATCH v7] Practical improvement to libgcc complex divide

2021-02-19 Thread Joseph Myers
On Tue, 2 Feb 2021, Patrick McGehearty via Gcc-patches wrote: > if (mode == TYPE_MODE (double_type_node)) > - ; /* Empty suffix correct. */ > + { > + ; /* Empty suffix correct. */ > + memcpy (modename, "DBL", 4); > + } > else if (mode =

[PATCH] PR fortran/99169 - [9/10/11 Regression] Segfault when passing allocatable scalar into intent(out) dummy argument

2021-02-19 Thread Harald Anlauf via Gcc-patches
Dear all, we should not clobber the pointer in case of an allocatable scalar being an intent(out) dummy argument to a procedure. Regtested on x86_64-pc-linux-gnu. OK for master? Since this is a regression, also for backports to 10/9? Thanks, Harald PR fortran/99169 - Do not clobber allocatab

Re: [PATCH] PR 99133, Mark xxspltiw, xxspltidp, and xxsplti32x as being prefixed

2021-02-19 Thread Segher Boessenkool
On Thu, Feb 18, 2021 at 10:44:14PM -0500, Michael Meissner wrote: > On Wed, Feb 17, 2021 at 06:09:39PM -0600, Segher Boessenkool wrote: > > Why test a p10 patch on a p8? > > Well it is just a code gen patch, so I can test it on any system, even an > x86_64 with a cross compiler. Given that right

Re: [committed] jit: fix ICE on BUILT_IN_TRAP [PR99126]

2021-02-19 Thread David Malcolm via Gcc-patches
On Fri, 2021-02-19 at 11:16 +0100, Andrea Corallo wrote: > David Malcolm via Gcc-patches writes: > > > I tried several approaches to fixing this; this seemed the > > least invasive. > > Hi Dave, > > thanks for fixing this. > > I do have a trivial question: are we guaranteed that the middle-end

Re: [PATCH] rs6000: Use rldimi for vec init instead of shift + ior

2021-02-19 Thread Segher Boessenkool
Hi! On Fri, Feb 19, 2021 at 11:06:16AM +0800, Kewen.Lin wrote: > on 2021/2/19 上午2:33, Segher Boessenkool wrote: > > Is there a PR you should mention here? > > I thought this is trivial so didn't file one upstream PR. I did a > searching in upstream bugzilla, PR93453 looks similar but different.

libgo patch committed: Update to Go1.16 release

2021-02-19 Thread Ian Lance Taylor via Gcc-patches
This patch updates libgo to the final Go 1.16 release. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian c89b42d3b9d76dedb35e8478913ddf5367f3b5e6 diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE index eed9ce01905..217bdd55f1d 100644 --- a/gcc/go/g

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

2021-02-19 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 French team of translators. The file is available at: https://translationproject.org/latest/gcc/fr.po (This file, 'gcc-11.1-b20210207.fr.po',

[WIP] Re: [PATCH] openmp: Fix intermittent hanging of task-detach-6 libgomp tests [PR98738]

2021-02-19 Thread Kwok Cheung Yeung
Hello Sorry for taking so long in replying. On 29/01/2021 3:03 pm, Jakub Jelinek wrote: It can also crash if team is NULL, which will happen any time this is called outside of a parallel. Just try (should go into testsuite too): #include int main () { omp_event_handle_t ev; #pragma omp

Re: [PATCH] run -Wnonnull later (PR 87489)

2021-02-19 Thread Martin Sebor via Gcc-patches
On 2/19/21 2:48 AM, Franz Sirl wrote: Am 2021-02-01 um 01:31 schrieb Martin Sebor via Gcc-patches: The initial -Wnonnull implementation in the middle end took place too late in the pipeline (just before expansion), and as a result was prone to false positives (bug 78817).  In an attempt to avoid

Re: [PATCH] clear VLA bounds from all arguments (PR 97172)

2021-02-19 Thread Martin Sebor via Gcc-patches
On 2/18/21 8:51 PM, Jeff Law wrote: On 2/18/21 7:23 PM, Martin Sebor wrote: The fix for PR 97172 that removes non-constant VLA bounds from attribute access is incomplete: it inadvertently removes the bounds corresponding to just the first VLA argument, and not from subsequent arguments. The a

c++: Inform of CMI reads [PR 99166]

2021-02-19 Thread Nathan Sidwell
When successfully reading a module CMI, the user gets no indication of where that CMI was located. I originally didn't consider this a problem -- the read was successful after all. But it can make it difficult to interact with build systems, particularly when caching can be involved. Grovelli

[PATCH] Document the GCC11 change to DWARF5 default.

2021-02-19 Thread Mark Wielaard
* gcc-11/changes.html (General Improvements): Add a section on the DWARF version 5 default. --- htdocs/gcc-11/changes.html | 30 ++ 1 file changed, 30 insertions(+) diff --git a/htdocs/gcc-11/changes.html b/htdocs/gcc-11/changes.html index de75b8d6..04c

Re: [patch, fortran] PR96686 Namelist group objects shall be defined before appearing in namelist

2021-02-19 Thread Tobias Burnus
Hi Jerry, On 17.02.21 04:02, Jerry DeLisle wrote: Attached patch adds to checks. In the case of IMPLICIT typing it checks to see if the objects listed in the NAMELIST have defined types andf if not, sets them to the default implicit types. In the case of IMPLICIT NONE, the types are required b

[PATCH][obvious] Fix typo in param description.

2021-02-19 Thread Martin Liška
Pushed to master as obvious. Martin gcc/ChangeLog: PR translation/99167 * params.opt: Fix typo. --- gcc/params.opt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/params.opt b/gcc/params.opt index a4e1ac0e88e..0dd9ac406eb 100644 --- a/gcc/params.opt +++

Re: *PING**2 Re: [Patch] Fortran: Fix coarray handling for gfc_dep_resolver [PR99010] (was: Re: [Patch] Fortran: Fix Array dependency with local coarrays [PR98913]

2021-02-19 Thread Jerry DeLisle
On 2/19/21 1:33 AM, Tobias Burnus wrote: On 09.02.21 12:52, Tobias Burnus wrote: Hi all, hi Thomas, On 02.02.21 18:15, Tobias Burnus wrote: I think I will do a combination: If 'identical' is true, I think I cannot remove it. If it is false, it can be identical or nonoverlapping – which makes

Re: [Patch] Fortran: Fix DTIO with type ICE [PR99146]

2021-02-19 Thread Jerry DeLisle
On 2/19/21 8:00 AM, Tobias Burnus wrote: In this example, the formal argument is a derived type and not a class – hence, there is an ICE. OK for the trunk? This is OK, could you also check 89219 and 81499 and see if these are the same or similar. Much appreciated. Jerry

[Patch] Fortran: Fix DTIO with type ICE [PR99146]

2021-02-19 Thread Tobias Burnus
In this example, the formal argument is a derived type and not a class – hence, there is an ICE. OK for the trunk? Tobias - Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank Thürauf F

Re: [PATCH][PR98791]: IRA: Make sure allocno copy mode's are ordered

2021-02-19 Thread Vladimir Makarov via Gcc-patches
On 2021-02-19 5:53 a.m., Andre Vieira (lists) wrote: Hi, This patch makes sure that allocno copies are not created for unordered modes. The testcases in the PR highlighted a case where an allocno copy was being created for: (insn 121 120 123 11 (parallel [     (set (reg:VNx2QI 217)

RE: [PATCH] slp: fix sharing of SLP only patterns. (PR99149)

2021-02-19 Thread Tamar Christina via Gcc-patches
Ps. The code in comment is needed, but was commented out due to the early free issue described in the patch. Thanks, Tamar > -Original Message- > From: Tamar Christina > Sent: Friday, February 19, 2021 2:42 PM > To: gcc-patches@gcc.gnu.org > Cc: nd ; rguent...@suse.de > Subject: [PATCH]

[PATCH] slp: fix sharing of SLP only patterns. (PR99149)

2021-02-19 Thread Tamar Christina via Gcc-patches
Hi Richi, The attached testcase ICEs due to a couple of issues. In the testcase you have two SLP instances that share the majority of their definition with each other. One tree defines a COMPLEX_MUL sequence and the other tree a COMPLEX_FMA. The ice happens because: 1. the refcounts are wrong,

[PATCH] middle-end/99122 - more VLA inlining fixes

2021-02-19 Thread Richard Biener
This avoids declaring a function with VLA arguments or return values as inlineable. IPA CP still ICEs, so the testcase has that disabled. Bootstrapped and tested on x86_64-unknown-linux-gnu, pushed. 2021-02-19 Richard Biener PR middle-end/99122 * tree-inline.c (inline_forbidd

[Patch] OpenACC: C/C++ - fix async parsing [PR99137]

2021-02-19 Thread Tobias Burnus
For: #pragma acc parallel async(1,2) avoid with C an ICE for the original tree: #pragma acc parallel async(<<< Unknown tree: c_maybe_const_expr 1, 2 >>>) It did not ICE with C++, but I think the tree does not make sense, either: #pragma acc parallel async(<<< Unknown tree: void_cst >>>, 2)

Re: [PATCH] gcov: use mmap pools for KVP.

2021-02-19 Thread Jakub Jelinek via Gcc-patches
On Tue, Feb 09, 2021 at 09:37:12AM +0100, Martin Liška wrote: > PING^2 > > @Honza: ? Just concerning Windows (though I don't have access to that OS and can't verify), e.g. https://github.com/m-labs/uclibc-lm32/blob/master/utils/mmap-windows.c contains public domain code to emulate mmap on top of

RE: ARM patch ping

2021-02-19 Thread Kyrylo Tkachov via Gcc-patches
Hi Jakub, > -Original Message- > From: Jakub Jelinek > Sent: 19 February 2021 10:45 > To: Richard Sandiford ; Richard Earnshaw > ; Ramana Radhakrishnan > ; Kyrylo Tkachov > > Cc: gcc-patches@gcc.gnu.org > Subject: ARM patch ping > > Hi! > > I'd like to ping the > https://gcc.gnu.org/pi

Re: [committed] libstdc++: Fix __thread_yield for non-gthreads targets

2021-02-19 Thread Jonathan Wakely via Gcc-patches
On 15/02/21 16:13 +, Jonathan Wakely wrote: The __gthread_yield() function is only defined for gthreads targets, so check _GLIBCXX_HAS_GTHREADS before using it. Also reorder __thread_relax and __thread_yield so that the former can use the latter instead of repeating the same preprocessor che

Re: [PATCH] improve warning suppression for inlined functions (PR 98465, 98512)

2021-02-19 Thread Florian Weimer via Gcc-patches
* Jeff Law via Gcc-patches: > I'd lean towards deferring to gcc12 stage1 given the libstdc++ hack is > in place.  That does mean that glibc will need to work around the > instance they've stumbled over in ppc's rawmemchr. We'll need to work around this in the glibc build, too. I'll check if the

[PATCH][PR98791]: IRA: Make sure allocno copy mode's are ordered

2021-02-19 Thread Andre Vieira (lists) via Gcc-patches
Hi, This patch makes sure that allocno copies are not created for unordered modes. The testcases in the PR highlighted a case where an allocno copy was being created for: (insn 121 120 123 11 (parallel [     (set (reg:VNx2QI 217)     (vec_duplicate:VNx2QI (subreg/s/v:QI (re

Re: [RFC][patch for gcc12][version 1] add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-02-19 Thread Martin Jambor
On Thu, Feb 18 2021, Qing Zhao via Gcc-patches wrote: > (CC’ing Kees Cook on this topic) > > Hi, > > This is the first version of the complete patch for the new security feature > for GCC: > > Initialize automatic variables with new first class option > -ftrivial-auto-var-init=[uninitialized|pa

Re: [PATCH] tree-cfg: Fix up gimple_merge_blocks FORCED_LABEL handling [PR99034]

2021-02-19 Thread Richard Biener
On Fri, 19 Feb 2021, Jakub Jelinek wrote: > Hi! > > The verifiers require that DECL_NONLOCAL or EH_LANDING_PAD_NR > labels are always the first label if there is more than one label. > > When merging blocks, we don't honor that though. > On the following testcase, we try to merge blocks: > [cou

ARM patch ping

2021-02-19 Thread Jakub Jelinek via Gcc-patches
Hi! I'd like to ping the https://gcc.gnu.org/pipermail/gcc-patches/2021-February/565225.html patch - PR98998 P1 fix. Thanks Jakub

Re: [committed] jit: fix ICE on BUILT_IN_TRAP [PR99126]

2021-02-19 Thread Andrea Corallo via Gcc-patches
Andrea Corallo via Gcc-patches writes: > David Malcolm via Gcc-patches writes: > >> I tried several approaches to fixing this; this seemed the >> least invasive. > > Hi Dave, > > thanks for fixing this. > > I do have a trivial question: are we guaranteed that the middle-end will > not try to add

Re: [committed] jit: fix ICE on BUILT_IN_TRAP [PR99126]

2021-02-19 Thread Andrea Corallo via Gcc-patches
David Malcolm via Gcc-patches writes: > I tried several approaches to fixing this; this seemed the > least invasive. Hi Dave, thanks for fixing this. I do have a trivial question: are we guaranteed that the middle-end will not try to add any build-in other than a trap? Regards Andrea

[PATCH] tree-cfg: Fix up gimple_merge_blocks FORCED_LABEL handling [PR99034]

2021-02-19 Thread Jakub Jelinek via Gcc-patches
Hi! The verifiers require that DECL_NONLOCAL or EH_LANDING_PAD_NR labels are always the first label if there is more than one label. When merging blocks, we don't honor that though. On the following testcase, we try to merge blocks: [count: 0]: : S::~S (&s); and [count: 0]: : resx 1 where is

Re: [PATCH] run -Wnonnull later (PR 87489)

2021-02-19 Thread Franz Sirl
Am 2021-02-01 um 01:31 schrieb Martin Sebor via Gcc-patches: The initial -Wnonnull implementation in the middle end took place too late in the pipeline (just before expansion), and as a result was prone to false positives (bug 78817).  In an attempt to avoid the worst of those, the warning was mo

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

2021-02-19 Thread Kito Cheng via Gcc-patches
Hi Marcus: > I could also disable the -mlittle-endian and -mbig-endian options if > the binutils is too old, but it's questionable if getting a "I don't > understand -mbig-endian" from gcc is more useful than getting it from > gas. And it would be more work for the user to fix it since then they

*PING**2 Re: [Patch] Fortran: Fix coarray handling for gfc_dep_resolver [PR99010] (was: Re: [Patch] Fortran: Fix Array dependency with local coarrays [PR98913]

2021-02-19 Thread Tobias Burnus
On 09.02.21 12:52, Tobias Burnus wrote: Hi all, hi Thomas, On 02.02.21 18:15, Tobias Burnus wrote: I think I will do a combination: If 'identical' is true, I think I cannot remove it. If it is false, it can be identical or nonoverlapping – which makes sense. Well, it turned out that coarray

[PATCH] aarch64: Introduce prefer_advsimd_autovec to GCC 10

2021-02-19 Thread Kyrylo Tkachov via Gcc-patches
Hi all, This patch introduces the prefer_advsimd_autovec internal tune flag that's already available on the GCC 8 and 9 branches. It allows a CPU tuning to specify that it prefers Advanced SIMD for autovectorisation rather than SVE. In GCC 10 onwards this can be easily adjusted through the aarc

Re: [PATCH 0/2] IBM Z: Fix long double <-> DFP conversions

2021-02-19 Thread Andreas Krebbel via Gcc-patches
On 2/18/21 1:57 PM, Ilya Leoshkevich wrote: > This series fixes PR99134. Patch 1 is factored out from the pending > [1], patch 2 is the actual fix. Bootstrapped and regtested on > s390x-redhat-linux. Ok for master? > > [1] https://gcc.gnu.org/pipermail/gcc-patches/2021-January/564380.html > >

Re: [PATCH] PR fortran/99147 - Sanitizer detects heap-use-after-free in gfc_add_flavor

2021-02-19 Thread Paul Richard Thomas via Gcc-patches
Hi Harald, It looks 'obvious' to me too and is certainly OK for master. Thanks Paul On Thu, 18 Feb 2021 at 21:30, Harald Anlauf via Fortran wrote: > Dear all, > > the PR reports an issue detected with an ASAN instrumented compiler, > which can also be verified with valgrind. It appears that

[PR94092] Re: [RFC] test builtin ratio for loop distribution

2021-02-19 Thread Alexandre Oliva
Here's an improved version of the patch. Regstrapped on x86_64-linux-gnu, with and without a patchlet that moved multi-pieces ahead of setmem, and also tested with riscv32-elf. Is it ok to install? Or should it wait for stage1? [PR94092] introduce try store by multiple pieces From: Alexandre