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

2021-09-16 Thread Xionghu Luo via Gcc-patches
On 2021/9/15 21:11, David Edelsohn wrote: 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. Thanks, updated the patchset with Segher's

[PATCH v2 1/2] rs6000: Fix wrong code generation for vec_sel [PR94613]

2021-09-16 Thread Xionghu Luo via Gcc-patches
The vsel instruction is a bit-wise select instruction. Using an IF_THEN_ELSE to express it in RTL is wrong and leads to wrong code being generated in the combine pass. Per element selection is a subset of per bit-wise selection,with the patch the pattern is written using bit operations. But

[PATCH v2 0/2] Fix vec_sel code generation and merge xxsel to vsel

2021-09-16 Thread Xionghu Luo via Gcc-patches
These two patches are updated version from: https://gcc.gnu.org/pipermail/gcc-patches/2021-September/579490.html Changes: 1. Fix alignment error in md files. 2. Replace rtx_equal_p with match_dup. 3. Use register_operand instead of gpc_reg_operand to align with vperm/xxperm. 4. Regression

[PATCH v2 2/2] rs6000: Fold xxsel to vsel since they have same semantics

2021-09-16 Thread Xionghu Luo via Gcc-patches
Fold xxsel to vsel like xxperm/vperm to avoid duplicate code. gcc/ChangeLog: 2021-09-17 Xionghu Luo * config/rs6000/altivec.md: Add vsx register constraints. * config/rs6000/vsx.md (vsx_xxsel): Delete. (vsx_xxsel2): Likewise. (vsx_xxsel3): Likewise.

Re: [PATCH] better handle MIN/MAX_EXPR of unrelated objects [PR102200]

2021-09-16 Thread Jeff Law via Gcc-patches
On 9/16/2021 6:28 PM, Martin Sebor via Gcc-patches wrote: When computing the size of an object pointed to by the result of a MIN/MAX_EXPR, the handle_min_max_size() function tries to deal gracefully with operands that designate distinct objects.  But the handling fails to consider an edge

[PATCH, Fortran] Use _Float128 rather than __float128 for c_float128 kind

2021-09-16 Thread Sandra Loosemore
On 9/5/21 11:20 PM, Sandra Loosemore wrote: Unless the aarch64 maintainers think it is a bug that __float128 is not supported, I think the right solution here is the one I was thinking of previously, to fix the Fortran front end to tie the C_FLOAT128 kind to _Float128 rather than __float128,

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

2021-09-16 Thread Hongtao Liu via Gcc-patches
On Thu, Sep 16, 2021 at 8:31 PM Richard Biener wrote: > > On Thu, 16 Sep 2021, Hongtao Liu wrote: > > > On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches > > wrote: > > > > > > On Thu, 16 Sep 2021, liuhongt wrote: > > > > > > > Ping > > > > rebased on latest trunk. > > > > > > > >

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

2021-09-16 Thread Cui, Lili via Gcc-patches
> -Original Message- > From: Uros Bizjak > Sent: Thursday, September 16, 2021 2:28 PM > To: Cui, Lili > Cc: gcc-patches@gcc.gnu.org; Liu, Hongtao ; H. J. Lu > > Subject: Re: [PATCH 3/4] [PATCH 3/4] x86: Properly handle > USE_VECTOR_FP_CONVERTS/USE_VECTOR_CONVERTS > > On Wed, Sep 15,

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

2021-09-16 Thread Jeff Law via Gcc-patches
On 9/15/2021 1:28 PM, Iain Sandoe wrote: 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) OK.  Sorry I didn't look at it before.  I largely ignore things like JIT these days. Jeff

Re: [PATCH/RFC 1/2] WPD: Enable whole program devirtualization

2021-09-16 Thread Feng Xue OS via Gcc-patches
>On 9/16/21 05:25, Feng Xue OS via Gcc-patches wrote: >> This and following patches are composed to enable full devirtualization >> under whole program assumption (so also called whole-program >> devirtualization, WPD for short), which is an enhancement to current >> speculative devirtualization.

[PATCH] better handle MIN/MAX_EXPR of unrelated objects [PR102200]

2021-09-16 Thread Martin Sebor via Gcc-patches
When computing the size of an object pointed to by the result of a MIN/MAX_EXPR, the handle_min_max_size() function tries to deal gracefully with operands that designate distinct objects. But the handling fails to consider an edge case when one of the operands is a PHI one of whose operands

libgo patch committed: Update to go1.17.1 release

2021-09-16 Thread Ian Lance Taylor via Gcc-patches
This patch updates libgo to the go1.17.1 release. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian 1d62d26192bf7c2f303d993f9a2963a0fd5b475a diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE index f4816816500..e2abd5fc4b7 100644 ---

Re: [PATCH 04/18] rs6000: Handle some recent MMA builtin changes

2021-09-16 Thread Segher Boessenkool
Hi! On Wed, Sep 01, 2021 at 11:13:40AM -0500, Bill Schmidt wrote: > Peter Bergner recently added two new builtins __builtin_vsx_lxvp and > __builtin_vsx_stxvp. These happened to break a pattern in MMA builtins that > I had been using to automate gimple folding of MMA builtins. Previously, >

Re: [PATCH] libstdc++: Fix UB in atomic_ref/wait_notify.cc [PR101761]

2021-09-16 Thread Jonathan Wakely via Gcc-patches
On Thu, 16 Sep 2021, 23:24 Thomas Rodgers, wrote: > From: Thomas Rodgers > > Remove UB in atomic_ref/wait_notify test. > > Signed-off-by: Thomas Rodgers > > libstdc++-v3/ChangeLog: > > PR libstdc++/101761 > * testsuite/29_atomics/atomic_ref/wait_notify.cc (test): Use >

Re: [PATCH 03/18] rs6000: Handle gimple folding of target built-ins

2021-09-16 Thread Segher Boessenkool
On Wed, Sep 01, 2021 at 11:13:39AM -0500, Bill Schmidt wrote: > This is another patch that looks bigger than it really is. Because we > have a new namespace for the builtins, allowing us to have both the old > and new builtin infrastructure supported at once, we need versions of > these functions

Re: [PATCH 02/18] rs6000: Move __builtin_mffsl to the [always] stanza

2021-09-16 Thread Segher Boessenkool
On Wed, Sep 01, 2021 at 11:13:38AM -0500, Bill Schmidt wrote: > I over-restricted use of __builtin_mffsl, since I was unaware that it > automatically uses mffs when mffsl is not available. Paul Clarke pointed > this out in discussion of his SSE 4.1 compatibility patches. Right. Do we need to

[PATCH] nvptx: Add (experimental) support for HFmode with -misa=sm_53

2021-09-16 Thread Roger Sayle
The recent flurry of activity around HFmode on gcc-patches intrigued me to investigate adding HFmode support to the nvptx backend. NVidia GPUs with an SM ISA above 5.3 support IEEE 16-bit floating point instructions. Hence, this patch adds support for -misa=sm_53, and implements some backend

Re: [PATCH v2] analyzer: Define INCLUDE_UNIQUE_PTR

2021-09-16 Thread Gerald Pfeifer
On Tue, 14 Sep 2021, Maxim Blinov wrote: > Un-break the build for AArch64 Darwin, see PR bootstrap/102242. Build > fails with log below: David already acked this with "Does the patch fix the build for you? If so, looks good for trunk. Please reference PR bootstrap/102242 in the

[PATCH] libstdc++: Fix UB in atomic_ref/wait_notify.cc [PR101761]

2021-09-16 Thread Thomas Rodgers
From: Thomas Rodgers Remove UB in atomic_ref/wait_notify test. Signed-off-by: Thomas Rodgers libstdc++-v3/ChangeLog: PR libstdc++/101761 * testsuite/29_atomics/atomic_ref/wait_notify.cc (test): Use va and vb as arguments to wait/notify, remove unused bb local. Tested

[committed] libstdc++: Regenerate the src/debug Makefiles as needed

2021-09-16 Thread Jonathan Wakely via Gcc-patches
When the build configuration changes and Makefiles are recreated, the src/debug/Makefile and src/debug/*/Makefile files are not recreated, because they're not managed in the usual way by automake. This can lead to build failures or surprising inconsistencies between the main and debug versions of

[committed] libstdc++: Increase timeout factor for slow pb_ds tests

2021-09-16 Thread Jonathan Wakely via Gcc-patches
Compiling these tests still times out too often when running the testsuite with more parallel jobs than there are available cores. Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * testsuite/ext/pb_ds/regression/tree_map_rand.cc: Increase timeout factor to 3. *

[committed] libstdc++: Update documentation that only refers to c++98 and c++11

2021-09-16 Thread Jonathan Wakely via Gcc-patches
Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * doc/xml/manual/using.xml: Generalize to apply to more than just -std=c++11. * doc/html/manual/using_macros.html: Regenerate. Committed to trunk. commit bd0df30a7bc7a2e98e643cf84901e5383f83c005 Author: Jonathan

[committed] libstdc++: Add noexcept to std::nullopt_t constructor

2021-09-16 Thread Jonathan Wakely via Gcc-patches
Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * include/std/optional (nullptr_t): Make constructor noexcept. Tested x86_64-linux. Committed to trunk. commit cbe705a2f749c98a5f803afeb207e175b4c9a3c3 Author: Jonathan Wakely Date: Thu Sep 16 14:14:38 2021 libstdc++: Add

[committed] libstdc++: Remove non-deducible parameter for std::advance overload

2021-09-16 Thread Jonathan Wakely via Gcc-patches
This was just a copy and paste error. Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * include/bits/fs_path.h (advance): Remove non-deducible template parameter. Tested x86_64-linux. Committed to trunk. commit 21c760510d31253074577a14021fdc6ad44084b6 Author: Jonathan

[committed] libstdc++: Add missing 'constexpr' to std::tuple [PR102270]

2021-09-16 Thread Jonathan Wakely via Gcc-patches
Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: PR libstdc++/102270 * include/std/tuple (_Head_base, _Tuple_impl): Add _GLIBCXX20_CONSTEXPR to allocator-extended constructors. (tuple<>::swap(tuple&)): Add _GLIBCXX20_CONSTEXPR. *

[committed] libstdc++: Add missing constraint to std::span deduction guide [PR102280]

2021-09-16 Thread Jonathan Wakely via Gcc-patches
Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: PR libstdc++/102280 * include/std/span (span(Range&&)): Add constraint to deduction guide. Tested x86_64-linux. Committed to trunk. commit e67917f5df9d84f5aed3513b3931a82870d25135 Author: Jonathan Wakely Date:

[committed] libstdc++: Fix recipes for C++11-compiled files in src/c++98

2021-09-16 Thread Jonathan Wakely via Gcc-patches
Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * src/c++98/Makefile.am: Use CXXCOMPILE not LTCXXCOMPILE. * src/c++98/Makefile.in: Regenerate. Tested x86_64-linux. Committed to trunk. commit 2c351dafcbc871c088ce09ae69bc08871f7df57b Author: Jonathan Wakely Date: Wed

[committed] libstdc++: Add noexcept to std::to_string overloads that don't allocate

2021-09-16 Thread Jonathan Wakely via Gcc-patches
When the values is guaranteed to fit in the SSO buffer we know the string won't allocate, so the function can be noexcept. For 32-bit integers, we know they need no more than 9 bytes (or 10 with a minus sign) and the SSO buffer is 15 bytes. Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog:

[committed] libstdc++: Add noexcept to unique_ptr accessors

2021-09-16 Thread Jonathan Wakely via Gcc-patches
Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: * include/bits/unique_ptr.h (__uniq_ptr_impl::_M_ptr) (__uniq_ptr_impl::_M_deleter): Add noexcept. Tested powerpc64le-linux. Committed to trunk. commit 869107c9c9752c9a53cdb06179c1e6be6d2e5f44 Author: Jonathan Wakely

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

2021-09-16 Thread Qing Zhao via Gcc-patches
> On Sep 16, 2021, at 12:39 PM, Iain Sandoe wrote: > > > >> On 16 Sep 2021, at 18:11, Qing Zhao via Gcc-patches >> wrote: >> >> >> >>> On Sep 16, 2021, at 10:47 AM, Jakub Jelinek wrote: >>> >>> On Thu, Sep 16, 2021 at 03:39:46PM +, Qing Zhao wrote: > Even -mtune= is needed if

Re: [PATCH] c++: constrained variable template issues [PR98486]

2021-09-16 Thread Jason Merrill via Gcc-patches
On 9/16/21 12:44, Patrick Palka wrote: On Thu, 16 Sep 2021, Jason Merrill wrote: On 9/16/21 09:11, Patrick Palka wrote: This fixes some issues with constrained variable templates: * Constraints aren't checked when explicitly specializing a variable template * Constraints aren't

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

2021-09-16 Thread Jason Merrill via Gcc-patches
On 9/16/21 11:50, Michel Morin wrote: On Thu, Sep 16, 2021 at 5:44 AM Jason Merrill wrote: 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

[PATCH v2] tree-optimization/101186 - extend FRE with "equivalence map" for condition prediction

2021-09-16 Thread Di Zhao OS via Gcc-patches
Sorry about updating on this after so long. It took me much time to work out a new plan and pass the tests. The new idea is to use one variable to represent a set of equal variables at some basic-block. This variable is called a "equivalence head" or "equiv-head" in the code. (There's no-longer a

Re: [PATCH/RFC 1/2] WPD: Enable whole program devirtualization

2021-09-16 Thread Jason Merrill via Gcc-patches
On 9/16/21 05:25, Feng Xue OS via Gcc-patches wrote: This and following patches are composed to enable full devirtualization under whole program assumption (so also called whole-program devirtualization, WPD for short), which is an enhancement to current speculative devirtualization. The base of

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

2021-09-16 Thread Iain Sandoe via Gcc-patches
> On 16 Sep 2021, at 18:11, Qing Zhao via Gcc-patches > wrote: > > > >> On Sep 16, 2021, at 10:47 AM, Jakub Jelinek wrote: >> >> On Thu, Sep 16, 2021 at 03:39:46PM +, Qing Zhao wrote: Even -mtune= is needed if you want to stay safe, otherwise people testing with

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

2021-09-16 Thread Qing Zhao via Gcc-patches
> On Sep 16, 2021, at 10:47 AM, Jakub Jelinek wrote: > > On Thu, Sep 16, 2021 at 03:39:46PM +, Qing Zhao wrote: >>> Even -mtune= is needed if you want to stay safe, otherwise people testing >>> with --target_board=unix/-mtune=cascadelake (or whatever else) might get >>> failures. >> >>

[PATCH] [i386] Change ix86_decompose_address return type to bool.

2021-09-16 Thread Uros Bizjak via Gcc-patches
After a recent change only a boolean value is returned. 2021-09-16 Uroš Bizjak gcc/ * config/i386/i386-protos.h (ix86_decompose_address): Change return type to bool. * config/i386/i386.c (ix86_decompose_address): Ditto. Bootstrapped and regression tested on x86_64-linux-gnu

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

2021-09-16 Thread Koning, Paul via Gcc-patches
> On Sep 16, 2021, at 11:05 AM, Jeff Law wrote: > > > On 9/16/2021 1:41 AM, Richard Biener wrote: >> ... >> That said - yes, I'd consider a.out purely legacy and not fit >> for the future. But it never came up on the radar of standing >> in the way of modernizing GCC in any area. > I'd

Re: [PATCH] c++: constrained variable template issues [PR98486]

2021-09-16 Thread Patrick Palka via Gcc-patches
On Thu, 16 Sep 2021, Jason Merrill wrote: > On 9/16/21 09:11, Patrick Palka wrote: > > This fixes some issues with constrained variable templates: > > > >* Constraints aren't checked when explicitly specializing a variable > > template > >* Constraints aren't attached to a static

RE: [PATCH 1/5]AArch64 sve: combine inverted masks into NOTs

2021-09-16 Thread Tamar Christina via Gcc-patches
Hi esteemed reviewer! > -Original Message- > From: Richard Sandiford > Sent: Tuesday, August 31, 2021 4:46 PM > To: Tamar Christina > Cc: gcc-patches@gcc.gnu.org; nd ; Richard Earnshaw > ; Marcus Shawcroft > ; Kyrylo Tkachov > Subject: Re: [PATCH 1/5]AArch64 sve: combine inverted masks

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

2021-09-16 Thread Michel Morin via Gcc-patches
On Thu, Sep 16, 2021 at 5:44 AM Jason Merrill wrote: > > 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,

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

2021-09-16 Thread Jakub Jelinek via Gcc-patches
On Thu, Sep 16, 2021 at 03:39:46PM +, Qing Zhao wrote: > > Even -mtune= is needed if you want to stay safe, otherwise people testing > > with --target_board=unix/-mtune=cascadelake (or whatever else) might get > > failures. > > Okay. Will try this. > > > >>> and ideally also

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

2021-09-16 Thread Qing Zhao via Gcc-patches
> On Sep 16, 2021, at 9:56 AM, Jakub Jelinek wrote: > > On Thu, Sep 16, 2021 at 02:49:23PM +, Qing Zhao wrote: >>> Testing for many instructions is always very fragile and dependent on exact >>> compiler flags etc. >> >> Yes, It’s indeed very fragile. >>> So, either the test should have

Re: [Patch] PowerPC: Fix rs6000-gen-builtins with build != host [PR102353]

2021-09-16 Thread Segher Boessenkool
Hi! On Thu, Sep 16, 2021 at 11:07:25AM +0200, Tobias Burnus wrote: > As mentioned in https://gcc.gnu.org/PR102353 and in the patch, > rs6000-gen-builtins was build to be run on "host" – and then linked and > on on "build". > > That caused bootstrap fails at link time. > Tested with a

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

2021-09-16 Thread Jeff Law via Gcc-patches
On 9/15/2021 2:56 PM, apinski--- via Gcc-patches wrote: 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: *

Re: [patch] Fix PR rtl-optimization/102306

2021-09-16 Thread Jeff Law via Gcc-patches
-09-16 Eric Botcazou PR rtl-optimization/102306 * combine.c (try_combine): Abort the combination if we are about to duplicate volatile references. 2021-09-16 Eric Botcazou * gcc.target/sparc/20210916-1.c: New test. OK jeff

Re: [PATCH] middle-end/102360 - adjust .DEFERRED_INIT expansion

2021-09-16 Thread Michael Matz via Gcc-patches
Hello, On Thu, 16 Sep 2021, Richard Biener via Gcc-patches wrote: > > Typically for the native_interpret/native_encode we punt if > > BITS_PER_UNIT != 8 || CHAR_BIT != 8 because nobody had the energy to > > deal with the weird platforms (especially if we have currently none, I > > believe

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

2021-09-16 Thread Jason Merrill via Gcc-patches
On 9/16/21 05:11, 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 they all have this issue. 2021-08-06 wangpc gcc/cp/ChangeLog * decl.c (cp_finish_decl): Add type

Re: [PATCH 1/4] cgraph: Do not warn about caller count mismatches of removed functions

2021-09-16 Thread Martin Jambor
Hi, On Fri, Aug 20 2021, Martin Jambor wrote: > To verify other changes in the patch series, I have been searching for > "Invalid sum of caller counts" string in symtab dump but found that > there are false warnings about functions which have their body removed > because they are now unreachable.

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

2021-09-16 Thread Jeff Law via Gcc-patches
On 9/16/2021 1:41 AM, Richard Biener wrote: On Wed, 15 Sep 2021, Koning, Paul wrote: 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

Re: [PATCH] c++: constrained variable template issues [PR98486]

2021-09-16 Thread Jason Merrill via Gcc-patches
On 9/16/21 09:11, Patrick Palka wrote: This fixes some issues with constrained variable templates: * Constraints aren't checked when explicitly specializing a variable template * Constraints aren't attached to a static data member template at parse time * Constraints aren't

Re: [PATCH v3 0/6] rs6000: Support more SSE4 intrinsics

2021-09-16 Thread Paul A. Clarke via Gcc-patches
Ping. On Mon, Aug 23, 2021 at 02:03:04PM -0500, Paul A. Clarke via Gcc-patches wrote: > v3: Add "nmmintrin.h". _mm_cmpgt_epi64 is part of SSE4.2 > and users will expect to be able to include "nmmintrin.h", > even though "nmmintrin.h" just includes "smmintrin.h" > where all of the SSE4.2

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

2021-09-16 Thread Jakub Jelinek via Gcc-patches
On Thu, Sep 16, 2021 at 02:49:23PM +, Qing Zhao wrote: > > Testing for many instructions is always very fragile and dependent on exact > > compiler flags etc. > > Yes, It’s indeed very fragile. > > So, either the test should have a particular > > -march=/-mtune= options > > I might add

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

2021-09-16 Thread Qing Zhao via Gcc-patches
Hi, Jakub, > On Sep 16, 2021, at 4:19 AM, Jakub Jelinek wrote: > > On Wed, Sep 15, 2021 at 05:59:08PM +, Qing Zhao 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

[PATCH] wwwdocs: Move inactive branches to the right section

2021-09-16 Thread Jonathan Wakely via Gcc-patches
The https://gcc.gnu.org/git.html page is a total mess, listing dozens and dozens of branches which haven't seen updates in a decade and which are under the refs/dead/heads/* are of the Git repo. This moves them all to the "Inactive" or "Merged" section, as appropriate. OK for wwwdocs? commit

Re: [Patch][GCC][middle-end] - Lower store and load neon builtins to gimple

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, 16 Sep 2021, Jirui Wu wrote: > Hi all, > > This patch lowers the vld1 and vst1 variants of the > store and load neon builtins functions to gimple. > > The changes in this patch covers: > * Replaces calls to the vld1 and vst1 variants of the builtins > * Uses MEM_REF gimple assignments

Re: [PATCH 1/N] Rename asm_out_file function arguments.

2021-09-16 Thread Iain Sandoe
Hi Martin, > On 16 Sep 2021, at 11:00, Martin Liška wrote: > > As preparation for a new global object that will encapsulate > asm_out_file, we would need to live with a macro that will > define asm_out_file as casm->out_file and thus the name > can't be used in function arguments. So, if I

[Patch][GCC][middle-end] - Lower store and load neon builtins to gimple

2021-09-16 Thread Jirui Wu via Gcc-patches
Hi all, This patch lowers the vld1 and vst1 variants of the store and load neon builtins functions to gimple. The changes in this patch covers: * Replaces calls to the vld1 and vst1 variants of the builtins * Uses MEM_REF gimple assignments to generate better code * Updates test cases to prevent

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

2021-09-16 Thread wangpc via Gcc-patches
This patch adds type checking for static local vector variable in C++ template, both AArch64 SVE and RISCV RVV are of sizeless type and they all have this issue. 2021-08-06 wangpc gcc/cp/ChangeLog * decl.c (cp_finish_decl): Add type checking. gcc/testsuite/ChangeLog *

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

2021-09-16 Thread Martin Jambor
Hi, On Thu, Sep 16 2021, Kewen.Lin wrote: > 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

Re: [Patch] PowerPC: Fix rs6000-gen-builtins with build != host [PR102353]

2021-09-16 Thread Bill Schmidt via Gcc-patches
Thank you, Tobias!  This looks good to me and doesn't break host=target=build bootstrap.  I appreciate the patch very much.  (I can't approve it, so please wait for Segher/David to weigh in.) Bill On 9/16/21 4:07 AM, Tobias Burnus wrote: As mentioned in https://gcc.gnu.org/PR102353 and in

[PATCH] c++: constrained variable template issues [PR98486]

2021-09-16 Thread Patrick Palka via Gcc-patches
This fixes some issues with constrained variable templates: * Constraints aren't checked when explicitly specializing a variable template * Constraints aren't attached to a static data member template at parse time * Constraints aren't propagated when (partially) instantiating a

[Patch] Fortran: Add gfc_simple_for_loop aux function

2021-09-16 Thread Tobias Burnus
This patch adds a useful auxiliary function to generate a loop. I intent to use it for: (A) An updated/cleaned-up version of "[Patch] Fortran: Fix Bind(C) Array-Descriptor Conversion (Move to Front-End Code)" https://gcc.gnu.org/pipermail/gcc-patches/2021-September/578904.html → new

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

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, 16 Sep 2021, Hongtao Liu wrote: > On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches > wrote: > > > > On Thu, 16 Sep 2021, liuhongt wrote: > > > > > Ping > > > rebased on latest trunk. > > > > > > gcc/ChangeLog: > > > > > > * common.opt (ftree-vectorize): Add

PING – Re: [Patch] Fortran: Handle allocated() with coindexed scalars [PR93834] (was: [PATCH] PR fortran/93834 - [9/10/11/12 Regression] ICE in trans_caf_is_present, at fortran/trans-intrinsic.c:8469)

2021-09-16 Thread Tobias Burnus
Patch PING – see comment in the follow-up email of the patch email - and in the email(s) before in that thread. Tobias On 07.09.21 16:33, Tobias Burnus wrote: Now I actually tested the patch – and fixed some issues. OK? – It does add support for 'allocated(a[i])' by treating it as

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

2021-09-16 Thread Richard Biener via Gcc-patches
vectorizer.c (vec_info_shared::check_datarefs): Do not compare the lazily computed alternate set of indices. * gcc.dg/torture/20210916.c: New testcase. * gcc.dg/vect/pr65206.c: Likewise. --- gcc/testsuite/gcc.dg/torture/20210916.c | 20 +++ gcc/testsuite/gcc.dg/vect/pr65206

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

2021-09-16 Thread Iain Sandoe
> On 15 Sep 2021, at 20:50, Jason Merrill wrote: > 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

Re: [PATCH] middle-end/102360 - adjust .DEFERRED_INIT expansion

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, 16 Sep 2021, Jakub Jelinek wrote: > On Thu, Sep 16, 2021 at 12:41:20PM +0200, Richard Biener wrote: > > On Thu, 16 Sep 2021, Jakub Jelinek wrote: > > > > > On Thu, Sep 16, 2021 at 11:48:49AM +0200, Richard Biener via Gcc-patches > > > wrote: > > > > 2021-09-16 Richard Biener > > > >

Re: [PATCH] middle-end/102360 - adjust .DEFERRED_INIT expansion

2021-09-16 Thread Jakub Jelinek via Gcc-patches
On Thu, Sep 16, 2021 at 12:41:20PM +0200, Richard Biener wrote: > On Thu, 16 Sep 2021, Jakub Jelinek wrote: > > > On Thu, Sep 16, 2021 at 11:48:49AM +0200, Richard Biener via Gcc-patches > > wrote: > > > 2021-09-16 Richard Biener > > > > > > PR middle-end/102360 > > > * internal-fn.c

Re: [PATCH] middle-end/102360 - adjust .DEFERRED_INIT expansion

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, 16 Sep 2021, Jakub Jelinek wrote: > On Thu, Sep 16, 2021 at 11:48:49AM +0200, Richard Biener via Gcc-patches > wrote: > > 2021-09-16 Richard Biener > > > > PR middle-end/102360 > > * internal-fn.c (expand_DEFERRED_INIT): Make pattern-init > > of non-memory more robust. >

Re: [PATCH] middle-end/102360 - adjust .DEFERRED_INIT expansion

2021-09-16 Thread Jakub Jelinek via Gcc-patches
On Thu, Sep 16, 2021 at 11:48:49AM +0200, Richard Biener via Gcc-patches wrote: > 2021-09-16 Richard Biener > > PR middle-end/102360 > * internal-fn.c (expand_DEFERRED_INIT): Make pattern-init > of non-memory more robust. > > * g++.dg/pr102360.C: New testcase. > +

[PATCH 2/N] Do not hide asm_out_file in ASM_OUTPUT_ASCII.

2021-09-16 Thread Martin Liška
Again a preparation patch that was tested on all cross compilers. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, MartinFrom 0e5095274bb4e16ad28a5a52f30bd3887df25fde Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Wed, 15 Sep 2021

[PATCH 1/N] Rename asm_out_file function arguments.

2021-09-16 Thread Martin Liška
As preparation for a new global object that will encapsulate asm_out_file, we would need to live with a macro that will define asm_out_file as casm->out_file and thus the name can't be used in function arguments. I've built all cross compilers with the change and can bootstrap on

[PATCH] middle-end/102360 - adjust .DEFERRED_INIT expansion

2021-09-16 Thread Richard Biener via Gcc-patches
This avoids using native_interpret_type when we cannot do it with the original type of the variable, instead use an integer type for the initialization and side-step the size limitation of native_interpret_int. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress (note the reported ICE

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

2021-09-16 Thread Kito Cheng
Use TARGET_COMPUTE_MULTILIB to search the multi-lib reuse for riscv*-*-elf*, according following rules: 1. Check ABI is same. 2. Check both has atomic extension or both don't have atomic extension. - Because mix soft and hard atomic operation doesn't make sense and won't work as

[PATCH v3 1/2] Add TARGET_COMPUTE_MULTILIB hook to override multi-lib result.

2021-09-16 Thread Kito Cheng
Create a new hook to let target could override the multi-lib result, the motivation is RISC-V might have very complicated multi-lib re-use rule*, which is hard to maintain and use current multi-lib scripts, we even hit the "argument list too long" error when we tried to add more multi-lib reuse

[PATCH 0/2 v3] New target hook TARGET_COMPUTE_MULTILIB and implementation for RISC-V

2021-09-16 Thread Kito Cheng
This patch set allow target to use customized multi-lib mechanism rather than the built-in multi-lib mechanism. The motivation of this patch is RISC-V might have very complicated multi-lib re-use rule*, which is hard to maintain and use current multi-lib scripts, we even hit the "argument list

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

2021-09-16 Thread Christophe Lyon via Gcc-patches
On Thu, Sep 16, 2021 at 11:21 AM Richard Earnshaw via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: > > > On 16/09/2021 10:12, Christophe LYON via Gcc-patches wrote: > > > > On 15/09/2021 18:43, Richard Earnshaw via Gcc-patches wrote: > >> > >> > >> On 15/09/2021 17:13, Christophe Lyon via

[PATCH/RFC 2/2] WPD: Enable whole program devirtualization at LTRANS

2021-09-16 Thread Feng Xue OS via Gcc-patches
This patch is to extend applicability of full devirtualization to LTRANS stage. Normally, whole program assumption would not hold when WPA splits whole compilation into more than one LTRANS partitions. To avoid information lost for WPD at LTRANS, we will record all vtable nodes and related member

[PATCH/RFC 1/2] WPD: Enable whole program devirtualization

2021-09-16 Thread Feng Xue OS via Gcc-patches
This and following patches are composed to enable full devirtualization under whole program assumption (so also called whole-program devirtualization, WPD for short), which is an enhancement to current speculative devirtualization. The base of the optimization is how to identify class type that is

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

2021-09-16 Thread Richard Earnshaw via Gcc-patches
On 16/09/2021 10:12, Christophe LYON via Gcc-patches wrote: On 15/09/2021 18:43, Richard Earnshaw via Gcc-patches wrote: 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

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

2021-09-16 Thread Jakub Jelinek via Gcc-patches
On Wed, Sep 15, 2021 at 05:59:08PM +, Qing Zhao 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

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

2021-09-16 Thread Martin Liška
On 9/15/21 15:08, Martin Liška wrote: 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

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

2021-09-16 Thread wangpc via Gcc-patches
This patch adds type checking for static local vector variable in C++ template, both AArch64 SVE and RISCV RVV are of sizeless type and they all have this issue. 2021-08-06 wangpc gcc/cp/ChangeLog * decl.c (cp_finish_decl): Add type checking. gcc/testsuite/ChangeLog *

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

2021-09-16 Thread Christophe LYON via Gcc-patches
On 15/09/2021 18:43, Richard Earnshaw via Gcc-patches wrote: 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

[Patch] PowerPC: Fix rs6000-gen-builtins with build != host [PR102353]

2021-09-16 Thread Tobias Burnus
As mentioned in https://gcc.gnu.org/PR102353 and in the patch, rs6000-gen-builtins was build to be run on "host" – and then linked and on on "build". That caused bootstrap fails at link time. The patch now does the same as Makefile.in for 'gen*', i.e. build under build/ (using the Makefile.in

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

2021-09-16 Thread pc.wang via Gcc-patches
I move the verify_type_context code to cp_finish_decl and it works. I will send the new patch later. -- Sender:Jason Merrill Sent At:2021 Sep. 16 (Thu.) 05:04 Recipient:wangpc ; gcc-patches Subject:Re: [PATCH] C++: add type

[patch] Fix PR rtl-optimization/102306

2021-09-16 Thread Eric Botcazou
rtl-optimization/102306 * combine.c (try_combine): Abort the combination if we are about to duplicate volatile references. 2021-09-16 Eric Botcazou * gcc.target/sparc/20210916-1.c: New test. -- Eric Botcazoudiff --git a/gcc/combine.c b/gcc/combine.c index 290a3667

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

2021-09-16 Thread wangpc via Gcc-patches
This patch adds type checking for static local vector variable in C++ template, both AArch64 SVE and RISCV RVV are of sizeless type and they all have this issue. 2021-08-06 wangpc gcc/cp/ChangeLog * pt.c (tsubst_decl): Add type checking. gcc/testsuite/ChangeLog *

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

2021-09-16 Thread Hongtao Liu via Gcc-patches
On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches wrote: > > On Thu, 16 Sep 2021, liuhongt wrote: > > > Ping > > rebased on latest trunk. > > > > gcc/ChangeLog: > > > > * common.opt (ftree-vectorize): Add Var(flag_tree_vectorize). > > * doc/invoke.texi (Options That

Re: [PATCH] Check mask type when doing cond_op related gimple simplification.

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, Sep 16, 2021 at 8:27 AM liuhongt wrote: > > Ping. > > Bootstrapped and regtest on x86_64-linux-gnu{-m32,}, > aarch64-unknown-linux-gnu{-m32,} > Ok for trunk? > > gcc/ChangeLog: > > PR middle-end/102080 > * match.pd: Check mask type when doing cond_op related gimple >

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

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, 16 Sep 2021, liuhongt wrote: > 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):

[PATCH] [AVX512FP16] Support embedded broadcast for AVX512FP16 instructions.

2021-09-16 Thread liuhongt via Gcc-patches
Bootstrapped and regtest on x86_64-pc-linux-gnu{-m32,}. Runtime tests passed under sde{-m32,}. gcc/ChangeLog: PR target/87767 * config/i386/i386.c (ix86_print_operand): Handle V8HF/V16HF/V32HFmode. * config/i386/i386.h (VALID_BCST_MODE_P): Add HFmode.

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

2021-09-16 Thread Richard Biener via Gcc-patches
On Wed, 15 Sep 2021, Koning, Paul wrote: > > > > 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

Re: GNU Tools @ LPC 2021: Program is published

2021-09-16 Thread Thomas Schwinge
Hi! On 2021-09-16T05:58:22+0200, Gerald Pfeifer wrote: > 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

Re: [PATCH] Fix target/101934: aarch64 memset code creates unaligned stores for -mstrict-align

2021-09-16 Thread Andrew Pinski via Gcc-patches
On Wed, Sep 1, 2021 at 1:52 AM Richard Sandiford via Gcc-patches wrote: > > apinski--- via Gcc-patches writes: > > From: Andrew Pinski > > > > The problem here is the aarch64_expand_setmem code did not check > > STRICT_ALIGNMENT if it is creating an overlapping store. > > This patch adds that

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

2021-09-16 Thread Uros Bizjak via Gcc-patches
On Wed, Sep 15, 2021 at 10:10 AM wrote: > > 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

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

2021-09-16 Thread Uros Bizjak via Gcc-patches
On Wed, Sep 15, 2021 at 10:09 AM wrote: > > From: "H.J. Lu" > > Initial -mtune=tremont update > > 1. Use Haswell scheduling model. > 2. Assume that stack engine allows to execute push instructions in > parall. > 3. Prepare for scheduling pass as -mtune=generic. > 4. Use the same issue rate as

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

2021-09-16 Thread Uros Bizjak via Gcc-patches
On Wed, Sep 15, 2021 at 10:10 AM wrote: > > 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

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

2021-09-16 Thread Uros Bizjak via Gcc-patches
On Wed, Sep 15, 2021 at 10:10 AM wrote: > > 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/ > >

  1   2   >