Re: [PATCH] Maintain (mis-)alignment info in the first element of a group

2021-09-15 Thread Richard Sandiford via Gcc-patches
Richard Biener writes: > On Tue, 14 Sep 2021, Richard Sandiford wrote: > >> Richard Biener via Gcc-patches writes: >> > This changes us to maintain and compute (mis-)alignment info for >> > the first element of a group only rather than for each DR when >> > doing interleaving and for the earliest

Re: [PATCH, rs6000] Optimization for vec_xl_sext

2021-09-15 Thread HAO CHEN GUI via Gcc-patches
Bill,     Yes, I built the gcc with p10 binutils. Then power10_ok tests can pass. Thanks again for your kindly explanation.     I finally realized that the line wrap settings on my thunderbird didn't take any effect. I have to set a very large line size,  just for a workaround. ChangeLog 202

Re: Ping ^ 3: [PATCH] rs6000: Fix wrong code generation for vec_sel [PR94613]

2021-09-15 Thread Xionghu Luo via Gcc-patches
Ping^3, thanks. https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570333.html On 2021/9/6 08:52, Xionghu Luo via Gcc-patches wrote: Ping^2, thanks. https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570333.html On 2021/6/30 09:42, Xionghu Luo via Gcc-patches wrote: Gentle ping, thanks.

[PATCH 0/4] Update mtune=tremont

2021-09-15 Thread lili.cui--- via Gcc-patches
From: "Cui,Lili" Hi, I have four patches for tremont tuning, With all patches applied, performance impacts on SPEC CPU 2017 are: 500.perlbench_r 1.81% 502.gcc_r 0.57% 505.mcf_r 1.16% 520.omnetpp_r 0.00% 523.xalancbmk_r 0.

[PATCH 1/4] [PATCH 1/4] x86: Update -mtune=tremont

2021-09-15 Thread lili.cui--- via Gcc-patches
From: "H.J. Lu" Initial -mtune=tremont update 1. Use Haswell scheduling model. 2. Assume that stack engine allows to execute push&pop instructions in parall. 3. Prepare for scheduling pass as -mtune=generic. 4. Use the same issue rate as -mtune=generic. 5. Enable partial_reg_dependency. 6. Disab

[PATCH 4/4] [PATCH 4/4] x86: Add TARGET_SSE_PARTIAL_REG_[FP_]CONVERTS_DEPENDENCY

2021-09-15 Thread lili.cui--- via Gcc-patches
From: "H.J. Lu" 1. Replace TARGET_SSE_PARTIAL_REG_DEPENDENCY with TARGET_SSE_PARTIAL_REG_FP_CONVERTS_DEPENDENCY in SSE FP to FP splitters. 2. Replace TARGET_SSE_PARTIAL_REG_DEPENDENCY with TARGET_SSE_PARTIAL_REG_CONVERTS_DEPENDENCY in SSE INT to FP splitters. 3. Also check TARGET_SSE_PARTIAL_REG

[PATCH 2/4] [PATCH 2/4] x86: Update memcpy/memset inline strategies for -mtune=tremont

2021-09-15 Thread lili.cui--- via Gcc-patches
From: "H.J. Lu" Simply memcpy and memset inline strategies to avoid branches for -mtune=tremont: 1. Create Tremont cost model from generic cost model. 2. With MOVE_RATIO and CLEAR_RATIO == 17, GCC will use integer/vector load and store for up to 16 * 16 (256) bytes when the data size is fi

[PATCH 3/4] [PATCH 3/4] x86: Properly handle USE_VECTOR_FP_CONVERTS/USE_VECTOR_CONVERTS

2021-09-15 Thread lili.cui--- via Gcc-patches
From: "H.J. Lu" Check TARGET_USE_VECTOR_FP_CONVERTS or TARGET_USE_VECTOR_CONVERTS when handling avx_partial_xmm_update attribute. Don't convert AVX partial XMM register update if vector packed SSE conversion should be used. gcc/ PR target/101900 * config/i386/i386-features.c (r

aix: adjust installation directories for GCC64

2021-09-15 Thread CHIGOT, CLEMENT via Gcc-patches
As gcc on 64bit for AIX is built with "MULTILIB_MATCHES= .=maix32", "-print-multi-directory" and similar flags aren't returning the correct directory when used with -maix32: "." is returned instead of "ppc32". Libgcc installation script needs to be adjust to bypass this problem and correctly instal

Re: [committed] Fortran: Add missing ST_OMP_END_SCOPE handling [PR102313]

2021-09-15 Thread Thomas Schwinge
Hi! On 2021-09-14T14:25:20+0200, Tobias Burnus wrote: > I have created a testcase with all missing ST_OMP_END_* and ST_OACC_END_*; > I am not quite sure why a different code path is triggered for some, but > at least here is now a parse check for all. At least the OpenACC one is explained easily

Re: [PATCH 1/7] ifcvt: Check if cmovs are needed.

2021-09-15 Thread Robin Dapp via Gcc-patches
Hi Richard, Don't we still need this code (without the REG_DEAD handling) for the case in which… + /* As we are transforming +if (x > y) + { +a = b; +c = d; + } +into + a = (x > y) ... + c = (x > y) ... + +

PING^1 [PATCH] rs6000: Remove useless toc-fusion option

2021-09-15 Thread Kewen.Lin via Gcc-patches
Hi, Gentle ping this patch: https://gcc.gnu.org/pipermail/gcc-patches/2021-September/578553.html BR, Kewen on 2021/9/1 下午2:56, Kewen.Lin via Gcc-patches wrote: > Hi! > > Option toc-fusion was intended for Power9 toc fusion previously, > but Power9 doesn't support fusion at all eventually, thi

PING^1 [PATCH] rs6000: Fix some issues in rs6000_can_inline_p [PR102059]

2021-09-15 Thread Kewen.Lin via Gcc-patches
Hi! Gentle ping this patch: https://gcc.gnu.org/pipermail/gcc-patches/2021-September/578552.html BR, Kewen on 2021/9/1 下午2:55, Kewen.Lin via Gcc-patches wrote: > Hi! > > This patch is to fix the inconsistent behaviors for non-LTO mode > and LTO mode. As Martin pointed out, currently the funct

[PATCH] Loop unswitching: support gswitch statements.

2021-09-15 Thread Martin Liška
Hello. The patch extends the loop unswitching pass so that gswitch statements are supported. The pass now uses ranger which marks switch edges that are known to be unreachable in a versioned loop. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Than

Re: [PATCH V2] Set bound/cmp/control for until wrap loop.

2021-09-15 Thread Jiufu Guo via Gcc-patches
Jiufu Guo writes: I may want to have a gentle ping on this. https://gcc.gnu.org/pipermail/gcc-patches/2021-September/578680.html BR, Jiufu > Changes on V1: > * Add more test case > * Add comment for exit-condition transform > * Removing duplicate setting on niter->control > > This patch reset n

[PATCH] rs6000: Parameterize some const values for density test

2021-09-15 Thread Kewen.Lin via Gcc-patches
Hi, This patch follows the discussion here[1], where Segher suggested parameterizing those exact magic constants for density heuristics, to make it easier to tweak if need. Since these heuristics are quite internal, I make these parameters as undocumented and be mainly used by developers. The ch

Re: [PATCH] libstdc++-v3: Check for TLS support on mingw

2021-09-15 Thread Jonathan Wakely via Gcc-patches
On Wed, 1 Sept 2021 at 10:52, Jonathan Wakely wrote: > > On Wed, 1 Sept 2021 at 02:44, Jonathan Yong <10wa...@gmail.com> wrote: > > > > On 8/31/21 9:02 AM, Jonathan Wakely wrote: > > > It looks like my questions about this patch never got an answer, and > > > it never got applied. > > > > > > Coul

Re: [PATCH][RFC] pru: Named address space for R30/R31 I/O access

2021-09-15 Thread Richard Biener via Gcc-patches
On Tue, Sep 14, 2021 at 11:13 PM Dimitar Dimitrov wrote: > > Hi, > > I'm sending this patch to get feedback for a new PRU CPU port feature. > My intention is to push it to master by end of September, so that it gets > included in GCC 12. > > The PRU architecture provides single-cycle access to GPI

[PATCH] tree-optimization/102318 - reduction epilogue re-use

2021-09-15 Thread Richard Biener via Gcc-patches
This refines the fix for PR102226 to do the mode conversion from V2DI to VNx2DI separately from the sign-conversion, retaining the signedness of the saved accumulator as before the original fix. Bootstrapped and tested on x86_64-unknown-linux-gnu, pushed. 2021-09-15 Richard Biener PR t

[PATCH] sparc: Print out bit names for LEON and LEON3 with -mdebug

2021-09-15 Thread Daniel Cederman
From: Andreas Larsson gcc/ChangeLog: * config/sparc/sparc.c (dump_target_flag_bits): Print bit names for LEON and LEON3. --- gcc/config/sparc/sparc.c | 4 1 file changed, 4 insertions(+) diff --git a/gcc/config/sparc/sparc.c b/gcc/config/sparc/sparc.c index 06f41d7bb53..d5

[PATCH 1/4] sparc: Treat more instructions as load or store in errata workarounds

2021-09-15 Thread Daniel Cederman
Check the attribute of instruction to determine if it performs a store or load operation. This more generic approach sees the last instruction in the GOTdata_op model as a potential load and treats the memory barrier as a potential store instruction. gcc/ChangeLog: * config/sparc/sparc.c

[PATCH 2/4] sparc: Skip all empty assembly statements

2021-09-15 Thread Daniel Cederman
This version detects multiple empty assembly statements in a row and also detects non-memory barrier empty assembly statements (__asm__("")). It can be used instead of next_active_insn(). gcc/ChangeLog: * config/sparc/sparc.c (next_active_non_empty_insn): New function that returns

[PATCH] sparc: Add scheduling information for LEON5

2021-09-15 Thread Daniel Cederman
The LEON5 can often dual issue instructions from the same 64-bit aligned double word if there are no data dependencies. Add scheduling information to avoid scheduling unpairable instructions back-to-back. gcc/ChangeLog: * config/sparc/sparc-opts.h (enum sparc_processor_type): Add LEON5

[PATCH 3/4] sparc: Prevent atomic instructions in beginning of functions for UT700

2021-09-15 Thread Daniel Cederman
A call to the function might have a load instruction in the delay slot and a load followed by an atomic function could cause a deadlock. gcc/ChangeLog: * config/sparc/sparc.c (sparc_do_work_around_errata): Do not begin functions with atomic instruction in the UT700 errata workarou

[PATCH 4/4] sparc: Add NOP in stack_protect_setsi if sparc_fix_b2bst enabled

2021-09-15 Thread Daniel Cederman
This is needed to prevent the Store -> (Non-store or load) -> Store sequence. gcc/ChangeLog: * config/sparc/sparc.md: Add NOP to prevent sensitive sequence for B2BST errata workaround. --- gcc/config/sparc/sparc.md | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-)

[PATCH] Optimize for V{8,16,32}HFmode vec_set/extract/init.

2021-09-15 Thread liuhongt via Gcc-patches
Hi: The optimization is decribled in PR. Bootstrapped and regtest on x86_64-linux-gnu{-m32,}. All avx512fp16 runtest cases passed on SPR. gcc/ChangeLog: PR target/102327 * config/i386/i386-expand.c (ix86_expand_vector_init_interleave): Use puncklwd to pack 2

Re: [PATCH] Maintain (mis-)alignment info in the first element of a group

2021-09-15 Thread Richard Biener via Gcc-patches
On Wed, 15 Sep 2021, Richard Sandiford wrote: > Richard Biener writes: > > On Tue, 14 Sep 2021, Richard Sandiford wrote: > > > >> Richard Biener via Gcc-patches writes: > >> > This changes us to maintain and compute (mis-)alignment info for > >> > the first element of a group only rather than fo

Re: [PATCH] Maintain (mis-)alignment info in the first element of a group

2021-09-15 Thread Richard Sandiford via Gcc-patches
Richard Biener via Gcc-patches writes: > On Wed, 15 Sep 2021, Richard Sandiford wrote: >> Richard Biener writes: >> > On Tue, 14 Sep 2021, Richard Sandiford wrote: >> > >> >> Richard Biener via Gcc-patches writes: >> >> > This changes us to maintain and compute (mis-)alignment info for >> >> > t

Re: [PATCH] sparc: Add scheduling information for LEON5

2021-09-15 Thread Eric Botcazou
> The LEON5 can often dual issue instructions from the same 64-bit aligned > double word if there are no data dependencies. Add scheduling information > to avoid scheduling unpairable instructions back-to-back. > > gcc/ChangeLog: > > * config/sparc/sparc-opts.h (enum sparc_processor_type)

Re: [PATCH] sparc: Print out bit names for LEON and LEON3 with -mdebug

2021-09-15 Thread Eric Botcazou
> gcc/ChangeLog: > > * config/sparc/sparc.c (dump_target_flag_bits): Print bit names for > LEON and LEON3. OK everywhere. -- Eric Botcazou

Re: [PATCH 1/4] sparc: Treat more instructions as load or store in errata workarounds

2021-09-15 Thread Eric Botcazou
> gcc/ChangeLog: > > * config/sparc/sparc.c (store_insn_p): Add predicate for store > attributes. > (load_insn_p): Add predicate for load attributes. > (sparc_do_work_around_errata): Use new predicates. OK everywhere on principle, but can we avoid the multiple call

Re: [PATCH RFC] c++: implement C++17 hardware interference size

2021-09-15 Thread Richard Earnshaw via Gcc-patches
On 14/09/2021 08:56, Christophe LYON via Gcc-patches wrote: On 10/09/2021 15:16, Jason Merrill via Gcc-patches wrote: OK, time to finish this up.  The main change relative to the last patch I sent to the list is dropping the -finterference-tune flag and making that behavior the default.  A

Re: [PATCH 2/4] sparc: Skip all empty assembly statements

2021-09-15 Thread Eric Botcazou
> gcc/ChangeLog: > > * config/sparc/sparc.c (next_active_non_empty_insn): New function > that returns next active non empty assembly instruction. > (sparc_do_work_around_errata): Use new function. OK everywhere, modulo a couple of nits: > +rtx_insn * > +next_active_non_em

Re: [PATCH 3/4] sparc: Prevent atomic instructions in beginning of functions for UT700

2021-09-15 Thread Eric Botcazou
> gcc/ChangeLog: > > * config/sparc/sparc.c (sparc_do_work_around_errata): Do not begin > functions with atomic instruction in the UT700 errata workaround. OK everywhere. -- Eric Botcazou

Re: [PATCH 4/4] sparc: Add NOP in stack_protect_setsi if sparc_fix_b2bst enabled

2021-09-15 Thread Eric Botcazou
> gcc/ChangeLog: > > * config/sparc/sparc.md: Add NOP to prevent sensitive sequence for > B2BST errata workaround. OK everywhere, but the ChangeLog entry should be: * config/sparc/sparc.md (stack_protect_set32): Add NOP... Note that it's stack_protect_set32 on mainline a

Re: [PATCH] testsuite: Make sure double-precision is supported in g++.dg/eh/arm-vfp-unwind.C

2021-09-15 Thread Richard Earnshaw via Gcc-patches
On 26/08/2021 16:53, Christophe Lyon via Gcc-patches wrote: g++.dg/eh/arm-vfp-unwind.C uses an asm statement relying on double-precision FPU support, but does not make sure it is actually supported by the target. Check (__ARM_FP & 8) to ensure this. 2021-08-26 Christophe Lyon gcc/

Re: [PATCH] sparc: Add scheduling information for LEON5

2021-09-15 Thread Daniel Cederman
Thank you for reviewing the patches! I will address your comments and push the patches after testing. Thanks again, Daniel On 2021-09-15 12:18, Eric Botcazou wrote: The LEON5 can often dual issue instructions from the same 64-bit aligned double word if there are no data dependencies. Add sched

[PATCH] aix: Add FAT library support for libffi

2021-09-15 Thread CHIGOT, CLEMENT via Gcc-patches
Even if GCC64 is able to bootstrap without libffi being a FAT library on AIX, the tests for "-maix32" are not working without it. libffi/ChangeLog: 2021-09-10  Clément Chigot   * Makefile.am (tmake_file): Build and install AIX-style FAT  libraries. * Makefile.in: Regenera

Re: [PATCH RFC] c++: implement C++17 hardware interference size

2021-09-15 Thread Christophe Lyon via Gcc-patches
On Wed, Sep 15, 2021 at 12:25 PM Richard Earnshaw via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: > > > On 14/09/2021 08:56, Christophe LYON via Gcc-patches wrote: > > > > On 10/09/2021 15:16, Jason Merrill via Gcc-patches wrote: > >> OK, time to finish this up. The main change relative to the

Re: [PATCH] testsuite: Make sure double-precision is supported in g++.dg/eh/arm-vfp-unwind.C

2021-09-15 Thread Christophe LYON via Gcc-patches
On 15/09/2021 13:02, Richard Earnshaw wrote: On 26/08/2021 16:53, Christophe Lyon via Gcc-patches wrote: g++.dg/eh/arm-vfp-unwind.C uses an asm statement relying on double-precision FPU support, but does not make sure it is actually supported by the target. Check (__ARM_FP & 8) to ensure thi

Re: [PATCH RFC] c++: implement C++17 hardware interference size

2021-09-15 Thread Martin Liška
On 9/14/21 09:56, Christophe LYON via Gcc-patches wrote: So adjustment is needed for both arm and aarch64 targets Hello. I noticed the same problem and I've got a patch candidate for it. What do you think about it? MartinFrom 2ecedfe5cc421dd36f5770b04553343ecebd3430 Mon Sep 17 00:00:00 2001 F

Re: [PATCH] testsuite: Make sure double-precision is supported in g++.dg/eh/arm-vfp-unwind.C

2021-09-15 Thread Richard Earnshaw via Gcc-patches
On 15/09/2021 13:26, Christophe LYON via Gcc-patches wrote: On 15/09/2021 13:02, Richard Earnshaw wrote: On 26/08/2021 16:53, Christophe Lyon via Gcc-patches wrote: g++.dg/eh/arm-vfp-unwind.C uses an asm statement relying on double-precision FPU support, but does not make sure it is actua

Re: [PATCH v2] ipa-inline: Add target info into fn summary [PR102059]

2021-09-15 Thread Martin Jambor
Hi, since this is inlining-related, I would somewhat prefer Honza to have a look too, but I have the following comments: On Wed, Sep 08 2021, Kewen.Lin wrote: > [...] > diff --git a/gcc/ipa-fnsummary.h b/gcc/ipa-fnsummary.h > index 78399b0b9bb..300b8da4507 100644 > --- a/gcc/ipa-fnsummary.h > +

Re: [PATCH] aix: Add FAT library support for libffi

2021-09-15 Thread David Edelsohn via Gcc-patches
Clement, GCC is not the primary repository for libffi. This patch must be submitted to the libffi project first, not GCC. If it is accepted in libffi, then you can ask for a backport to GCC. https://github.com/libffi/libffi Thanks, David On Wed, Sep 15, 2021 at 7:20 AM CHIGOT, CLEMENT wrote:

Re: [PATCH v2 1/2] MIPS: use mips_isa enum instead hardcoded numbers

2021-09-15 Thread Martin Liška
Hello. I noticed the change likely caused the following failure when building x86_64-linux-gnu cross compiler: g++ -fno-PIE -c -DIN_GCC_FRONTEND -DIN_GCC_FRONTEND -g -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwri

Re: Ping ^ 3: [PATCH] rs6000: Fix wrong code generation for vec_sel [PR94613]

2021-09-15 Thread David Edelsohn via Gcc-patches
Hi, Xionhu Should "altivec_vsel2" .. 3 .. 4 be "*altivec_vsel2", etc. because they are combiner patterns and never referenced by name? Only the first, named pattern is referenced by the builtin code. Other than that question / suggestion, this patch is okay. Please coordinate with Bill and his

Re: [PATCH] aix: Add FAT library support for libffi

2021-09-15 Thread CHIGOT, CLEMENT via Gcc-patches
Hi David, The problem is that it has no meaning in libffi itself... This patch is specific to gcc because the multilib part is specific to gcc. I'll ask the community but the patch cannot be merged inside libffi. Thanks, Clément From: David Edelsohn Sent: Wednesd

Re: PING^1 [PATCH] rs6000: Remove useless toc-fusion option

2021-09-15 Thread David Edelsohn via Gcc-patches
On Wed, Sep 15, 2021 at 4:41 AM Kewen.Lin wrote: > > Hi, > > Gentle ping this patch: > > https://gcc.gnu.org/pipermail/gcc-patches/2021-September/578553.html > > > BR, > Kewen > > on 2021/9/1 下午2:56, Kewen.Lin via Gcc-patches wrote: > > Hi! > > > > Option toc-fusion was intended for Power9 toc fus

Re: [PATCH] aix: Add FAT library support for libffi

2021-09-15 Thread David Edelsohn via Gcc-patches
Clement, GCC libffi cherry-picks / backports patches from upstream, but it does not maintain local patches, so we need to find another solution. Thanks, David On Wed, Sep 15, 2021 at 9:14 AM CHIGOT, CLEMENT wrote: > > Hi David, > > The problem is that it has no meaning in libffi itself... > Thi

Re: [PATCH][v2] Always default to DWARF2_DEBUG if not specified, warn about deprecated STABS

2021-09-15 Thread John David Anglin
On 2021-09-15 2:26 a.m., Richard Biener wrote: >> I believe the 32-bit SOM target should be deprecated.  I'm the only one >> maintaining it and I had some health issues earlier this year. >> The current versions should suffice for several years. > Do you think it's worth keeping the 32bit pa hpux

[PATCH] target/102348 - fix powerpc-lynxos build

2021-09-15 Thread Richard Biener via Gcc-patches
This fixes a similar issue for powerpc-lynxos as fixed for i686-lynxos already. Build-tested for powerpc-lynxos. 2021-09-15 Richard Biener PR target/102348 * config/rs6000/lynx.h: Remove undef of PREFERRED_DEBUGGING_TYPE to inherit from elfos.h --- gcc/config/rs6000/l

Re: [PATCH][v2] Always default to DWARF2_DEBUG if not specified, warn about deprecated STABS

2021-09-15 Thread Richard Biener via Gcc-patches
On Wed, 15 Sep 2021, John David Anglin wrote: > On 2021-09-15 2:26 a.m., Richard Biener wrote: > >> I believe the 32-bit SOM target should be deprecated.  I'm the only one > >> maintaining it and I had some health issues earlier this year. > >> The current versions should suffice for several year

[PATCH][RFC] tree-optimization/65206 - dependence analysis on mixed pointer/array

2021-09-15 Thread Richard Biener via Gcc-patches
This adds the capability to analyze the dependence of mixed pointer/array accesses. The example is from where using a masked load/store creates the pointer-based access when an otherwise unconditional access is array based. Other examples would include accesses to an array mixed with accesses fro

Re: [PATCH 4/4] [PATCH 4/4] x86: Add TARGET_SSE_PARTIAL_REG_[FP_]CONVERTS_DEPENDENCY

2021-09-15 Thread H.J. Lu via Gcc-patches
There is no need to add [PATCH N/4] in the first line of the git commit message. "git format-patch" or "git send-email" will add them automatically. On Wed, Sep 15, 2021 at 1:10 AM wrote: > > From: "H.J. Lu" > > 1. Replace TARGET_SSE_PARTIAL_REG_DEPENDENCY with > TARGET_SSE_PARTIAL_REG_FP_CONVE

Re: [PATCH] rs6000: Fix wrong code generation for vec_sel [PR94613]

2021-09-15 Thread Segher Boessenkool
Hi! Please do not send patches as attachments to replies. Each patch (or patch series) starts its own thread. New versions of patches (or patch series) are new threads. > From: Xionghu Luo > Date: Tue, 27 Apr 2021 01:07:25 -0500 > Subject: [PATCH 1/2] rs6000: Fix wrong code generation for vec_

Re: [PATCH] aix: Add FAT library support for libffi

2021-09-15 Thread H.J. Lu via Gcc-patches
On Wed, Sep 15, 2021 at 6:18 AM David Edelsohn via Gcc-patches wrote: > > Clement, > > GCC libffi cherry-picks / backports patches from upstream, but it does > not maintain local patches, so we need to find another solution. > Please take a look at my libffi patches: https://gcc.gnu.org/pipermai

[PATCH][pushed] gcc-changelog: check git commit email address

2021-09-15 Thread Martin Liška
contrib/ChangeLog: * gcc-changelog/git_commit.py: Check commit email. * gcc-changelog/test_email.py: Add new test. * gcc-changelog/test_patches.txt: Likewise. --- contrib/gcc-changelog/git_commit.py| 10 ++ contrib/gcc-changelog/test_email.py| 5 + co

Re: GNU Tools @ LPC 2021: Program is published

2021-09-15 Thread Thomas Schwinge
Hi! On 2021-09-10T09:10:23+0100, Jeremy Bennett wrote: > The program for the GNU Tools Track at Linux Plumbers Conference is > published: > > https://linuxplumbersconf.org/event/11/sessions/109/ Yay! This may qualify "as obvious", but I better get reviewed what I change on our front page to t

[PATCH][OBVIOUS] rs6000: fix symtab_node::get == NULL issue

2021-09-15 Thread Martin Liška
Hello. The patch is approved by David and fixes the issue described in the PR. Martin PR target/102349 gcc/ChangeLog: * config/rs6000/rs6000.c (rs6000_xcoff_encode_section_info): Check that we have a symbol summary for a symbol. --- gcc/config/rs6000/rs6000.c | 1 + 1

[pushed] c++: don't warn about internal interference sizes

2021-09-15 Thread Jason Merrill via Gcc-patches
Most any compilation on ARM/AArch64 was warning because the default L1 cache line size of 32B was smaller than the default std::hardware_constructive_interference_size of 64B. This is mostly due to inaccurate --param l1-cache-line-size, but it's not helpful to complain to a user that didn't set th

Re: [PATCH RFC] c++: implement C++17 hardware interference size

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/15/21 8:31 AM, Martin Liška wrote: On 9/14/21 09:56, Christophe LYON via Gcc-patches wrote: So adjustment is needed for both arm and aarch64 targets Hello. I noticed the same problem and I've got a patch candidate for it. What do you think about it? I've now silenced the warning for i

Re: [pushed] c++: don't warn about internal interference sizes

2021-09-15 Thread Jeff Law via Gcc-patches
On 9/15/2021 9:31 AM, Jason Merrill via Gcc-patches wrote: Most any compilation on ARM/AArch64 was warning because the default L1 cache line size of 32B was smaller than the default std::hardware_constructive_interference_size of 64B. This is mostly due to inaccurate --param l1-cache-line-siz

Re: [pushed] c++: don't warn about internal interference sizes

2021-09-15 Thread Jason Merrill via Gcc-patches
On Wed, Sep 15, 2021 at 11:37 AM Jeff Law wrote: > > > On 9/15/2021 9:31 AM, Jason Merrill via Gcc-patches wrote: > > Most any compilation on ARM/AArch64 was warning because the default L1 > cache > > line size of 32B was smaller than the default > > std::hardware_constructive_interference_size o

[PATCH][pushed] i386: port vxworks to TARGET_CPU_P macro

2021-09-15 Thread Martin Liška
Hi. In g:f23881fcf081a6edd538d6d54fa0068d716973d7 I replaced TARGET_* macros with TARGET_CPU_P. Pushed as obvious. Martin PR target/102351 gcc/ChangeLog: * config/i386/vxworks.h: Use new macro TARGET_CPU_P. --- gcc/config/i386/vxworks.h | 24 1 file c

Re: [PATCH][v2] Always default to DWARF2_DEBUG if not specified, warn about deprecated STABS

2021-09-15 Thread John David Anglin
On 2021-09-15 10:06 a.m., Richard Biener wrote: >> Is there a simple way to enable -gstabs in build? > Currently not. If we're retaining more than pdp11 with a non-DWARF > config I'm considering to allow STABS by default for those without > diagnostics for GCC 12. > > With GCC 13 we'll definitely

Re: [PATCH] testsuite: Make sure double-precision is supported in g++.dg/eh/arm-vfp-unwind.C

2021-09-15 Thread Christophe Lyon via Gcc-patches
On Wed, Sep 15, 2021 at 2:49 PM Richard Earnshaw via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: > > > On 15/09/2021 13:26, Christophe LYON via Gcc-patches wrote: > > > > On 15/09/2021 13:02, Richard Earnshaw wrote: > >> > >> > >> On 26/08/2021 16:53, Christophe Lyon via Gcc-patches wrote: > >>>

Re: [pushed] c++: don't warn about internal interference sizes

2021-09-15 Thread Christophe Lyon via Gcc-patches
On Wed, Sep 15, 2021 at 5:39 PM Jason Merrill via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: > On Wed, Sep 15, 2021 at 11:37 AM Jeff Law wrote: > > > > > > > On 9/15/2021 9:31 AM, Jason Merrill via Gcc-patches wrote: > > > Most any compilation on ARM/AArch64 was warning because the default L1

[FYI] zero-call-used-regs attr for ada

2021-09-15 Thread Alexandre Oliva
Make the zero_call_used_regs attribute usable as a Machine_Attribute pragma. Regstrapped on x86_64-linux-gnu. Patch pre-approved by Olivier Hainque. for gcc/ada/ChangeLog * gcc-interface/utils.c: Include opts.h. (handle_zero_call_used_regs_attribute): New. (gnat_inte

Re: [PATCH] c++: default ctor that's also a list ctor [PR102050]

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/14/21 15:16, Patrick Palka wrote: In grok_special_member_properties we need to set TYPE_HAS_COPY_CTOR, TYPE_HAS_DEFAULT_CONSTRUCTOR and TYPE_HAS_LIST_CTOR independently from each other because a single constructor can be both a default and list constructor (as in the first testcase), or both

Re: [PATCH] testsuite: Make sure double-precision is supported in g++.dg/eh/arm-vfp-unwind.C

2021-09-15 Thread Richard Earnshaw via Gcc-patches
On 15/09/2021 17:13, Christophe Lyon via Gcc-patches wrote: On Wed, Sep 15, 2021 at 2:49 PM Richard Earnshaw via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: On 15/09/2021 13:26, Christophe LYON via Gcc-patches wrote: On 15/09/2021 13:02, Richard Earnshaw wrote: On 26/08/2021 16:53,

Re: [FYI] zero-call-used-regs attr for ada

2021-09-15 Thread Alexandre Oliva
On Sep 15, 2021, Alexandre Oliva wrote: > Regstrapped on x86_64-linux-gnu. Patch pre-approved by Olivier Hainque. Uhh, actually, Olivier had only seen and approved these changes: > for gcc/ada/ChangeLog > * gcc-interface/utils.c: Include opts.h. > (handle_zero_call_used_regs_attr

Re: [PATCH][v2] Always default to DWARF2_DEBUG if not specified, warn about deprecated STABS

2021-09-15 Thread Koning, Paul via Gcc-patches
> On Sep 15, 2021, at 11:55 AM, John David Anglin wrote: > > On 2021-09-15 10:06 a.m., Richard Biener wrote: >>> Is there a simple way to enable -gstabs in build? >> Currently not. If we're retaining more than pdp11 with a non-DWARF >> config I'm considering to allow STABS by default for thos

Re: [PATCH] coroutines: Small cleanups to await_statement_walker [NFC].

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/14/21 11:36, Iain Sandoe wrote: Hi Some small code cleanups that allow us to have just one place that we handle a statement with await expression(s) embedded. Also we can reduce the work done to figure out whether a statement contains any such expressions. tested on x86_64,powerpc64le-lin

Re: [PATCH] c++: shortcut bad convs during overload resolution, part 2 [PR101904]

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/12/21 15:45, Patrick Palka wrote: The r12-3346 patch makes us avoid computing excess argument conversions during overload resolution, but only when it turns out there's a strictly viable candidate in the overload set. If there is no such candidate then we still need to compute more conversi

Re: [PATCH] testsuite: Fix c-c++-common/auto-init-* tests

2021-09-15 Thread Qing Zhao via Gcc-patches
> On Sep 11, 2021, at 3:03 AM, Jakub Jelinek wrote: > > Note, the gcc.dg/i386/auto-init* tests fail also, just don't have time to > deal with that right now, just try > make check-gcc RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} > i386.exp=auto-init*' It’s strange that the above testing on

Re: [PATCH] coroutines: Small cleanups to await_statement_walker [NFC].

2021-09-15 Thread Iain Sandoe
Hi Jason, > On 15 Sep 2021, at 18:32, Jason Merrill wrote: > > On 9/14/21 11:36, Iain Sandoe wrote: >> Hi >> Some small code cleanups that allow us to have just one place that >> we handle a statement with await expression(s) embedded. Also we >> can reduce the work done to figure out whether a

Re: [PATCH][OBVIOUS] rs6000: fix symtab_node::get == NULL issue

2021-09-15 Thread David Edelsohn via Gcc-patches
This needs an additional adjustment. The encoding decoration needs to be applied if the decl isn't an alias. That means both a null summary *OR* the decl is not explicitly an alias. I'm proposing the following: diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index d0830a950

[PATCH] c++: Fix handling of decls with flexible array members initialized with side-effects [PR88578]

2021-09-15 Thread Jakub Jelinek via Gcc-patches
On Tue, Sep 14, 2021 at 10:50:32AM -0400, Jason Merrill wrote: > > Note, if the flexible array member is initialized only with non-constant > > initializers, we have a worse bug that this patch doesn't solve, the > > splitting of initializers into constant and dynamic initialization removes > > the

Re: [PATCH][RFC] pru: Named address space for R30/R31 I/O access

2021-09-15 Thread Dimitar Dimitrov
On Wed, Sep 15, 2021 at 11:12:18AM +0200, Richard Biener wrote: > On Tue, Sep 14, 2021 at 11:13 PM Dimitar Dimitrov wrote: > > > > Hi, > > > > I'm sending this patch to get feedback for a new PRU CPU port feature. > > My intention is to push it to master by end of September, so that it gets > > in

Re: [PATCH][OBVIOUS] rs6000: fix symtab_node::get == NULL issue

2021-09-15 Thread Martin Liška
On 9/15/21 20:40, David Edelsohn wrote: This needs an additional adjustment. The encoding decoration needs to be applied if the decl isn't an alias. That means both a null summary *OR* the decl is not explicitly an alias. Oh, sorry, I made a stupid thinko. Please install the patch. Martin

[PING^2] [PATCH] configure, jit: Allow for 'make check-gcc-jit'.

2021-09-15 Thread Iain Sandoe
Hi folks, > On 27 Aug 2021, at 14:00, Iain Sandoe wrote: > > +Jeff > > (it’s probably borderline obvious - but in the top level Makefile .. so) > >> On 17 Aug 2021, at 21:53, David Malcolm wrote: >> >> On Tue, 2021-08-17 at 19:59 +0100, Iain Sandoe wrote: >>> Hi, >>> >>> For those of us who

Re: [External] Re: [PATCH] libstdc++: Optimize 'to_string' with numeric_limits instead of __to_chars_len

2021-09-15 Thread Jonathan Wakely via Gcc-patches
N.B. Please CC *both* the libstdc++ list and the gcc-patches list, as per https://gcc.gnu.org/lists.html On Wed, 15 Sept 2021 at 14:02, 刘可 wrote: > > Thank you for your review, and I apologize for my mistake. I have updated and > tested it! Hmm, it doesn't work though. How did you test it? For

Re: [PATCH v3] c++: Fix cp_tree_equal for template value args using dependent sizeof/alignof/noexcept expressions

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/14/21 00:31, Barrett Adair wrote: I reworked the fix today based on feedback from Jason and Jakub (thank you), and the subject line is now outdated. I added another test for a closely related bug that's also fixed here (dependent-expr11.C -- this one would even fail without the second decl

Re: [PATCH] coroutines: Small cleanups to await_statement_walker [NFC].

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/15/21 14:32, Iain Sandoe wrote: Hi Jason, On 15 Sep 2021, at 18:32, Jason Merrill wrote: On 9/14/21 11:36, Iain Sandoe wrote: Hi Some small code cleanups that allow us to have just one place that we handle a statement with await expression(s) embedded. Also we can reduce the work done

Re: [PATCH] c++: Fix handling of decls with flexible array members initialized with side-effects [PR88578]

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/15/21 14:59, Jakub Jelinek wrote: On Tue, Sep 14, 2021 at 10:50:32AM -0400, Jason Merrill wrote: Note, if the flexible array member is initialized only with non-constant initializers, we have a worse bug that this patch doesn't solve, the splitting of initializers into constant and dynamic

Re: [PATCH][v2] Always default to DWARF2_DEBUG if not specified, warn about deprecated STABS

2021-09-15 Thread Koning, Paul via Gcc-patches
> On Sep 13, 2021, at 3:31 AM, Richard Biener wrote: > > This makes defaults.h choose DWARF2_DEBUG if PREFERRED_DEBUGGING_TYPE > is not specified by the target and NO_DEBUG if DWARF is not supported. As I'm looking at questions about old debug formats, it brings up the question of old object

[pushed] c++: add parsing_function_declarator predicate

2021-09-15 Thread Jason Merrill via Gcc-patches
While looking at PR96184 I noticed that we were recognizing the situation of parsing a function declarator based on current_binding_level, and that we ought to make that a predicate function. This patch is just refactoring, but I just suggested using it in a review of another patch. Tested x86_64

Re: [PATCH] c++: fix wrong fixit hints for misspelled typedef [PR77565]

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/14/21 04:29, Michel Morin via Gcc-patches wrote: On Tue, Sep 14, 2021 at 7:14 AM David Malcolm wrote: On Tue, 2021-09-14 at 03:35 +0900, Michel Morin via Gcc-patches wrote: Hi, PR77565 reports that, with the code `typdef int Int;`, GCC emits "did you mean 'typeof'?" instead of "did you

[PATCH] Fix PR 67102: Add libstdc++ dependancy to libffi

2021-09-15 Thread apinski--- via Gcc-patches
From: Andrew Pinski The error message is obvious -funconfigured-libstdc++-v3 is used on the g++ command line. So we just add the dependancy. OK? Bootstrapped and tested on x86_64-linux-gnu with no regressions. ChangeLog: * Makefile.def: Have configure-target-libffi depend on a

Re: [PATCH] C++: add type checking for static local vector variable in template

2021-09-15 Thread Jason Merrill via Gcc-patches
On 9/6/21 08:10, wangpc via Gcc-patches wrote: This patch adds type checking for static local vector variable in C++ template, both AArch64 SVE and RISCV RVV are of sizeless type and thay all have this issue. 2021-08-06 wangpc gcc/cp/ChangeLog * pt.c (tsubst_decl): Add type checkin

[PATCH] doc: improve -fsanitize=undefined description

2021-09-15 Thread Diane Meirowitz via Gcc-patches
doc: improve -fsanitize=undefined description gcc/ChangeLog: * doc/invoke.texi: add link to UndefinedBehaviorSanitizer documentation, mention UBSAN_OPTIONS, similar to what is done for AddressSanitizer. diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index

Re: [PATCH] c++: fix cases of core1001/1322 by not dropping cv-qualifier of function parameter of type of typename or decltype[PR101402,PR102033,PR102034,PR102039,PR102044]

2021-09-15 Thread Jason Merrill via Gcc-patches
On 8/31/21 09:55, nick huang via Gcc-patches wrote: These bugs are considered duplicate cases of PR51851 which has been suspended since 2012, an issue known as "core1001/1322". Considering this background, it deserves a long comment to explain. Many people believed the root cause of this family

RE: [PATCH 4/4] [PATCH 4/4] x86: Add TARGET_SSE_PARTIAL_REG_[FP_]CONVERTS_DEPENDENCY

2021-09-15 Thread Cui, Lili via Gcc-patches
> -Original Message- > From: H.J. Lu > Sent: Wednesday, September 15, 2021 10:14 PM > To: Cui, Lili > Cc: Uros Bizjak ; GCC Patches patc...@gcc.gnu.org>; Liu, Hongtao > Subject: Re: [PATCH 4/4] [PATCH 4/4] x86: Add > TARGET_SSE_PARTIAL_REG_[FP_]CONVERTS_DEPENDENCY > > There is no nee

[PATCH] rs6000: Modify the way for extra penalized cost

2021-09-15 Thread Kewen.Lin via Gcc-patches
Hi, This patch follows the discussion here[1], where Segher pointed out the existing way to guard the extra penalized cost for strided/elementwise loads with a magic bound doesn't scale. The way with nunits * stmt_cost can get one much exaggerated penalized cost, such as: for V16QI on P8, it's 16

Re: [PATCH 4/4] [PATCH 4/4] x86: Add TARGET_SSE_PARTIAL_REG_[FP_]CONVERTS_DEPENDENCY

2021-09-15 Thread H.J. Lu via Gcc-patches
On Wed, Sep 15, 2021 at 4:54 PM Cui, Lili wrote: > > > > > -Original Message- > > From: H.J. Lu > > Sent: Wednesday, September 15, 2021 10:14 PM > > To: Cui, Lili > > Cc: Uros Bizjak ; GCC Patches > patc...@gcc.gnu.org>; Liu, Hongtao > > Subject: Re: [PATCH 4/4] [PATCH 4/4] x86: Add >

Re: [PATCH v2] ipa-inline: Add target info into fn summary [PR102059]

2021-09-15 Thread Kewen.Lin via Gcc-patches
Hi Martin, Thanks for the review comments! on 2021/9/15 下午8:51, Martin Jambor wrote: > Hi, > > since this is inlining-related, I would somewhat prefer Honza to have a > look too, but I have the following comments: > > On Wed, Sep 08 2021, Kewen.Lin wrote: >> > > [...] > >> diff --git a/gcc/ip

Re: GNU Tools @ LPC 2021: Program is published

2021-09-15 Thread Gerald Pfeifer
On Wed, 15 Sep 2021, Thomas Schwinge wrote: >> The program for the GNU Tools Track at Linux Plumbers Conference is >> published: >> >> https://linuxplumbersconf.org/event/11/sessions/109/ > This may qualify "as obvious", but I better get reviewed what I change on > our front page to the Internet

Re: [PATCH 2/2] RISC-V: Implement TARGET_COMPUTE_MULTILIB

2021-09-15 Thread Kito Cheng via Gcc-patches
> I find the other_cond support a bit confusing. Is this for -mcmodel > perhaps? Why not just say that if so? I suppose we might have other multilib options other than -march, -mabi and -mcmodel, so I keep the flexibility here. > riscv_multi_lib_info_t::parse > Calls riscv_subset_list::parse tw

[PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-15 Thread liuhongt via Gcc-patches
Ping rebased on latest trunk. gcc/ChangeLog: * common.opt (ftree-vectorize): Add Var(flag_tree_vectorize). * doc/invoke.texi (Options That Control Optimization): Update documents. * opts.c (default_options_table): Enable auto-vectorization at O2 with very-c

Re: [PATCH 24/62] AVX512FP16: Add vmovw/vmovsh.

2021-09-15 Thread Hongtao Liu via Gcc-patches
I'm going to check in 6 patches [PATCH 24/62] AVX512FP16: Add vmovw/vmovsh. [PATCH 25/62] AVX512FP16: Add testcase for vmovsh/vmovw. [PATCH 26/62] AVX512FP16: Add vcvtph2dq/vcvtph2qq/vcvtph2w/vcvtph2uw/vcvtph2uqq/vcvtph2udq [PATCH 27/62] AVX512FP16: Add testcase for vcvtph2w/vcvtph2uw/vcvtph2dq/vc

  1   2   >