Re: [PATCH] git-backport: support renamed .cc files in commit message.

2022-01-19 Thread Martin Liška
On 1/18/22 20:10, Harald Anlauf via Fortran wrote: Am 17.01.22 um 22:26 schrieb Martin Liška: On 1/12/22 16:54, Martin Liška wrote: There's a patch that enhances git-backport so that it updates commit messages for files which name ends now with .cc and is still .c on a branch. The patch has

[PATCH] libstdc++: Fix for non-constexpr math_errhandling

2022-01-19 Thread Matthias Kretz
Follow-up to my last patch. This one is a more thorough fix. Tested on x86_64- linux. OK for trunk? 8< Use SFINAE magic to support: "It is unspecified whether math_errhandling is a macro or an identifier with external linkage." [C Standard] Signed-off-by: Matthias Kretz libstdc++-v3/

Re: nvptx: update fix for -Wformat-diag (was: [PATCH] nvptx: fix -Wformat-diag warnings)

2022-01-19 Thread Martin Liška
On 1/19/22 08:31, Thomas Schwinge wrote: |Thanks. I additionally fixed up 'num_workers' in test cases, and then pushed to master branch commit 2aea19bdb12308aac1519ffc5ecc9d1ec24f9cc5 "nvptx: update fix for -Wformat-diag", see attached.| I thank you for testing and adjustment of the patch. C

Re: [PATCH 1/2] [Ada] Compile s-mmap and 128bit on x86_64-linux-gnux32

2022-01-19 Thread Arnaud Charlet via Gcc-patches
OK, thanks. > PR ada/103538 > * Makefile.rtl (LIBGNAT_TARGET_PAIRS): Add > $(TRASYM_DWARF_UNIX_PAIRS), > s-tsmona.adb $(GNATRTL_128BIT_PAIRS). > (EXTRA_GNATRTL_NONTASKING_OBJS): Add $(TRASYM_DWARF_UNIX_OBJS) > and $(GNATRTL_128BIT_OBJS). > --- > gcc/ada/M

Re: [PATCH 2/2] [Ada] Set target_cpu to x32 for x86_64-linux-gnux32

2022-01-19 Thread Arnaud Charlet via Gcc-patches
OK, thanks. > Since the x86_64-linux-gnux32 compiler is actually an x32 compiler, set > target_cpu to x32 for x86_64-linux-gnux32. > > PR ada/103538 > * gcc-interface/Makefile.in (target_cpu): Set to x32 for > x86_64-linux-gnux32. > --- > gcc/ada/gcc-interface/Makefile.in | 7 +

[PATCH] RISC-V: Update testcases info with new implement info

2022-01-19 Thread shihua
From: LiaoShihua After commit 591b6e00d1bfe12932ca31530d5859f95db8a35a " riscv: fix -Wformat-diag errors ", some strings in implement was changed. This patch update the check info in testcases to sync with it. gcc/testsuite/ChangeLog: * gcc.target/riscv/arch-9.c: Update the ch

Re: [PATCH] RISC-V: Update testcases info with new implement info

2022-01-19 Thread Martin Liška
On 1/19/22 10:15, shi...@iscas.ac.cn wrote: |From: LiaoShihua After commit 591b6e00d1bfe12932ca31530d5859f95db8a35a " riscv: fix -Wformat-diag errors ", some strings in implement was changed. This patch update the check info in testcases to sync with it.| Thank you for the fix! Martin

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Richard Biener via Gcc-patches
On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches wrote: > > In this PR the waccess pass was fed: > > D.10779 ={v} {CLOBBER}; > VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), > 64B, _2); > _7 = D.10779.__val[0]; > > However, the tracking of m_clobbers only looked

Re: [PATCH] riscv: fix -Wformat-diag errors.

2022-01-19 Thread Martin Liška
On 1/18/22 23:00, Joseph Myers wrote: On Tue, 18 Jan 2022, Martin Liška wrote: @@ -3309,8 +3309,8 @@ riscv_handle_type_attribute (tree *node ATTRIBUTE_UNUSED, tree name, tree args, && strcmp (string, "machine")) { warning (OPT_Wattributes, -

Re: [PATCH] tree-optimization/103721 - Only add equivalencies that are still valid.

2022-01-19 Thread Richard Biener via Gcc-patches
On Wed, Jan 19, 2022 at 2:37 AM Andrew MacLeod via Gcc-patches wrote: > > This patch happens to fix the PR, but I believe it only papers over a > deeper issue that is uncovered in PR104067. > > That said, examination of the issue uncovered an oversight in the way > equivalence sets are merged by t

Re: [PATCH] vect.exp: Rename .cc files in the folder to .C.

2022-01-19 Thread Richard Biener via Gcc-patches
On Mon, Jan 17, 2022 at 12:10 PM Martin Liška wrote: > > Hi. > > I would like to unify filenames in testsuite and one of the biggest > offenders are vectorizer tests under g++.dg. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? I think .cc is

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Richard Sandiford via Gcc-patches
Richard Biener writes: > On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches > wrote: >> >> In this PR the waccess pass was fed: >> >> D.10779 ={v} {CLOBBER}; >> VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), >> 64B, _2); >> _7 = D.10779.__val[0]; >> >> However, th

Re: [RFC] Port git gcc-descr to Python

2022-01-19 Thread Martin Liška
On 10/18/21 11:01, Martin Liška wrote: On 10/12/21 10:59, Martin Liška wrote: Hello. There's a complete patch that implements both git gcc-descr and gcc-undesrc and sets corresponding git aliases to use them. Ready to be installed? Thanks, Martin All right, so Jakub told me at IRC that we do

Re: [vect] PR103997: Fix epilogue mode skipping

2022-01-19 Thread Richard Biener via Gcc-patches
On Tue, 18 Jan 2022, Andre Vieira (lists) wrote: > > On 14/01/2022 09:57, Richard Biener wrote: > > > > The 'used_vector_modes' is also a heuristic by itself since it registers > > every vector type we query, not only those that are used in the end ... > > > > So it's really all heuristics that c

Re: Enable -Werror=format-diag for bootstrap

2022-01-19 Thread Martin Liška
On 1/18/22 17:28, Martin Liška wrote: On 1/16/22 01:31, Martin Sebor wrote: Hello. (I think your message in unrelated to this email thread). Martin, I've looked into removing the -Wno-error for this warning for just a subset of targets.  It seems doable with some hardcoding in configure.ac bu

[committed] rs6000: Fix bootstrap

2022-01-19 Thread Jakub Jelinek via Gcc-patches
On Wed, Jan 19, 2022 at 07:54:19AM +0100, Sebastian Huber wrote: > On 18/01/2022 22:42, Segher Boessenkool wrote: > > > +default: > > > + break; > > Please don't do that. You can do > > > >default: > > break; > > break; > > /* And just to make sure: */ > > break;

[PATCH][pushed] testsuite: Rename a few C++ to .C or .cc.

2022-01-19 Thread Martin Liška
The tests are C++ code, so use a proper file extension. gcc/testsuite/ChangeLog: * g++.dg/ext/boolcomplex-1.c: Moved to... * g++.dg/ext/boolcomplex-1.C: ...here. * g++.dg/opt/pr47639.c: Moved to... * g++.dg/opt/pr47639.C: ...here. * g++.dg/pr83979.c: Moved

Re: [PATCH] libstdc++: Fix for non-constexpr math_errhandling

2022-01-19 Thread Jonathan Wakely via Gcc-patches
On Wed, 19 Jan 2022 at 08:10, Matthias Kretz wrote: > Follow-up to my last patch. This one is a more thorough fix. Tested on > x86_64- > linux. OK for trunk? > > 8< > > Use SFINAE magic to support: "It is unspecified whether math_errhandling > is a macro or an identifier with external li

Re: [PATCH] libstdc++: Fix for non-constexpr math_errhandling

2022-01-19 Thread Matthias Kretz
On Wednesday, 19 January 2022 13:07:26 CET Jonathan Wakely wrote: > On Wed, 19 Jan 2022 at 08:10, Matthias Kretz wrote: > > Follow-up to my last patch. This one is a more thorough fix. Tested on > > x86_64- > > linux. OK for trunk? > > > > 8< > > > > Use SFINAE magic to support: "It is

Re: [RFC] Port git gcc-descr to Python

2022-01-19 Thread Martin Jambor
Hi, On Wed, Jan 19 2022, Martin Liška wrote: > On 10/18/21 11:01, Martin Liška wrote: >> On 10/12/21 10:59, Martin Liška wrote: >>> Hello. >>> >>> There's a complete patch that implements both git gcc-descr and gcc-undesrc >>> and sets corresponding git aliases to use them. >>> >>> Ready to be ins

[PATCH] tree-optimization/104114 - avoid diagnosing V1mode lowering

2022-01-19 Thread Richard Biener via Gcc-patches
Currently we diagnose vector lowering of V1mode operations that are not natively supported into V_C_E, scalar op plus CTOR with -Wvector-operation-performance but that's hardly useful behavior even though the way we lower things can be improved. The following disables the diagnostics for the cases

Re: [RFC] Port git gcc-descr to Python

2022-01-19 Thread Martin Liška
On 1/19/22 13:49, Martin Jambor wrote: Hi, On Wed, Jan 19 2022, Martin Liška wrote: On 10/18/21 11:01, Martin Liška wrote: On 10/12/21 10:59, Martin Liška wrote: Hello. There's a complete patch that implements both git gcc-descr and gcc-undesrc and sets corresponding git aliases to use them.

Re: [PATCH 1/2] [Ada] Compile s-mmap and 128bit on x86_64-linux-gnux32

2022-01-19 Thread H.J. Lu via Gcc-patches
On Wed, Jan 19, 2022 at 1:06 AM Arnaud Charlet wrote: > > OK, thanks. OK for backports? Thanks. > > PR ada/103538 > > * Makefile.rtl (LIBGNAT_TARGET_PAIRS): Add > > $(TRASYM_DWARF_UNIX_PAIRS), > > s-tsmona.adb > $(GNATRTL_128BIT_PAIRS). > > (EXTRA_GNATRTL_NON

Re: [PATCH 2/2] [Ada] Set target_cpu to x32 for x86_64-linux-gnux32

2022-01-19 Thread H.J. Lu via Gcc-patches
On Wed, Jan 19, 2022 at 1:06 AM Arnaud Charlet wrote: > > OK, thanks. OK for backports? Thanks. > > Since the x86_64-linux-gnux32 compiler is actually an x32 compiler, set > > target_cpu to x32 for x86_64-linux-gnux32. > > > > PR ada/103538 > > * gcc-interface/Makefile.in (target_cp

Re: [PATCH 1/2] [Ada] Compile s-mmap and 128bit on x86_64-linux-gnux32

2022-01-19 Thread Arnaud Charlet via Gcc-patches
> OK for backports? Yes.

Re: [PATCH 2/2] [Ada] Set target_cpu to x32 for x86_64-linux-gnux32

2022-01-19 Thread Arnaud Charlet via Gcc-patches
> > OK, thanks. > > OK for backports? Yes.

[PATCH] tree-optimization/104112 - add check for vect epilogue reduc reuse

2022-01-19 Thread Richard Biener via Gcc-patches
This adds a missing check for the availability of intermediate vector types required to re-use the accumulator of a vectorized reduction in the vectorized epilogue. For SVE and VNx2DF vs V2DF with -msve-vector-bits=512 for example V4DF is not available. In addition to that we have to verify the r

[PATCH] Update per-file selftest and finalization hooks for .c to .cc renaming

2022-01-19 Thread David Malcolm via Gcc-patches
This is mostly a mechanical change, apart from: - fix the name of opt_proposer_c to match its filename (opt-suggestions.cc) - delete a bogus "modref_c_tests" decl from ipa-modref-tree.h that's been present since the initial commit of that file (d119f34c952f8718fdbabc63e2f369a16e92fa07) Success

Re: [PATCH] Update per-file selftest and finalization hooks for .c to .cc renaming

2022-01-19 Thread Richard Biener via Gcc-patches
On Wed, Jan 19, 2022 at 2:45 PM David Malcolm via Gcc-patches wrote: > > This is mostly a mechanical change, apart from: > - fix the name of opt_proposer_c to match its filename (opt-suggestions.cc) > - delete a bogus "modref_c_tests" decl from ipa-modref-tree.h that's been > present since the i

Re: [PATCH] tree-optimization/104112 - add check for vect epilogue reduc reuse

2022-01-19 Thread Richard Sandiford via Gcc-patches
Richard Biener writes: > This adds a missing check for the availability of intermediate vector > types required to re-use the accumulator of a vectorized reduction > in the vectorized epilogue. For SVE and VNx2DF vs V2DF with > -msve-vector-bits=512 for example V4DF is not available. > > In addit

[PATCH] match.pd, optabs: Avoid vectorization of {FLOOR,CEIL,ROUND}_{DIV,MOD}_EXPR [PR102860]

2022-01-19 Thread Jakub Jelinek via Gcc-patches
Hi! power10 has modv4si3 expander and so vectorizes the following testcase where Fortran modulo is FLOOR_MOD_EXPR. optabs_for_tree_code indicates that the optab for all the *_MOD_EXPR variants is umod_optab or smod_optab, but that isn't true, that optab actually expands just TRUNC_MOD_EXPR. For t

Re: [PATCH] match.pd, optabs: Avoid vectorization of {FLOOR,CEIL,ROUND}_{DIV,MOD}_EXPR [PR102860]

2022-01-19 Thread Richard Biener via Gcc-patches
On Wed, 19 Jan 2022, Jakub Jelinek wrote: > Hi! > > power10 has modv4si3 expander and so vectorizes the following testcase > where Fortran modulo is FLOOR_MOD_EXPR. > optabs_for_tree_code indicates that the optab for all the *_MOD_EXPR > variants is umod_optab or smod_optab, but that isn't true,

[PATCH] testsuite: Test evrp-trans.c also with unsigned types [PR104115]

2022-01-19 Thread Jakub Jelinek via Gcc-patches
Hi! The testcase from the PR got fixed with r12-3119-g675a3e40567e1d and looks quite similar to the evrp-trans.c test, except evrp-trans.c is tested on signed integer types. I think it would be useful to test it for unsigned comparisons too. Tested on x86_64-linux (-m32/-m64), ok for trunk? 2022

Re: [PATCH] testsuite: Test evrp-trans.c also with unsigned types [PR104115]

2022-01-19 Thread Aldy Hernandez via Gcc-patches
OK. Thanks. On Wed, Jan 19, 2022, 14:58 Jakub Jelinek wrote: > Hi! > > The testcase from the PR got fixed with r12-3119-g675a3e40567e1d > and looks quite similar to the evrp-trans.c test, except evrp-trans.c > is tested on signed integer types. > I think it would be useful to test it for unsigne

Re: PING 4 [PATCH v2 2/2] add -Wdangling-pointer [PR #63272]

2022-01-19 Thread Stephan Bergmann via Gcc-patches
On 17/01/2022 20:14, Martin Sebor wrote: I tried to set up OpenOffice for testing with the latest GCC but couldn't get the build to finish (it failed downloading some unavailable prerequisites).  I don't remember what problem I ran into with LibreOffice; it was before I upgraded to Fedora 35 just

Re: [vect] PR103997: Fix epilogue mode skipping

2022-01-19 Thread Andre Vieira (lists) via Gcc-patches
On 19/01/2022 11:04, Richard Biener wrote: On Tue, 18 Jan 2022, Andre Vieira (lists) wrote: On 14/01/2022 09:57, Richard Biener wrote: The 'used_vector_modes' is also a heuristic by itself since it registers every vector type we query, not only those that are used in the end ... So it's real

Re: [PATCH, rs6000] Add a combine pattern for CA minus one [PR95737]

2022-01-19 Thread David Edelsohn via Gcc-patches
On Wed, Jan 19, 2022 at 2:12 AM HAO CHEN GUI wrote: > > Hi, >This patch adds a combine pattern for "CA minus one". As CA only has two > values (0 or 1), we could convert following pattern > (sign_extend:DI (plus:SI (reg:SI 98 ca) > (const_int -1 [0x] >

[committed] libstdc++: Fix libbacktrace build files

2022-01-19 Thread Jonathan Wakely via Gcc-patches
Tested x86_64-linux, pushed to trunk. This makes it possible to combine --enable-libstdcxx-debug with --enable-libstdcxx-backtrace, by adding a rule to src/Makefile to copy the backtrace-supported.h header into the src/debug/libbacktrace directory. Add libbacktrace path to testsuite flags so the

Re: [PATCH, rs6000] Add a combine pattern for CA minus one [PR95737]

2022-01-19 Thread David Edelsohn via Gcc-patches
On Wed, Jan 19, 2022 at 2:12 AM HAO CHEN GUI wrote: > > Hi, >This patch adds a combine pattern for "CA minus one". As CA only has two > values (0 or 1), we could convert following pattern > (sign_extend:DI (plus:SI (reg:SI 98 ca) > (const_int -1 [0x] >

Re: [PATCH] libstdc++: Fix for non-constexpr math_errhandling

2022-01-19 Thread Jonathan Wakely via Gcc-patches
On Wed, 19 Jan 2022, 12:45 Matthias Kretz, wrote: > On Wednesday, 19 January 2022 13:07:26 CET Jonathan Wakely wrote: > > On Wed, 19 Jan 2022 at 08:10, Matthias Kretz wrote: > > > Follow-up to my last patch. This one is a more thorough fix. Tested on > > > x86_64- > > > linux. OK for trunk? > > >

Re: [PATCH] libstdc++: Fix for non-constexpr math_errhandling

2022-01-19 Thread Matthias Kretz
On Wednesday, 19 January 2022 16:21:15 CET Jonathan Wakely wrote: > On Wed, 19 Jan 2022, 12:45 Matthias Kretz, wrote: > > On Wednesday, 19 January 2022 13:07:26 CET Jonathan Wakely wrote: > > > So is it a macro or not? > > > > I agree the quote I used is unclear. The complete paragraph: > > > >

Re: [committed] rs6000: Fix bootstrap

2022-01-19 Thread Segher Boessenkool
On Wed, Jan 19, 2022 at 12:27:32PM +0100, Jakub Jelinek wrote: > On Wed, Jan 19, 2022 at 07:54:19AM +0100, Sebastian Huber wrote: > > On 18/01/2022 22:42, Segher Boessenkool wrote: > > > > +default: > > > > + break; > > > Please don't do that. You can do > > > > > >default: > > >

Re: [PATCH] c++: CTAD within alias template [PR91911]

2022-01-19 Thread Patrick Palka via Gcc-patches
On Mon, Jan 3, 2022 at 10:24 AM Patrick Palka wrote: > > On Wed, 22 Dec 2021, Jason Merrill wrote: > > > On 12/21/21 14:08, Patrick Palka wrote: > > > On Tue, Dec 21, 2021 at 2:03 PM Patrick Palka wrote: > > > > > > > > On Wed, Jun 30, 2021 at 4:23 PM Jason Merrill wrote: > > > > > > > > > > On

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Martin Sebor via Gcc-patches
On 1/19/22 03:09, Richard Sandiford wrote: Richard Biener writes: On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches wrote: In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXPR(D.10779) = .MASK_LOAD_LANES (addr_5(D), 64B, _2); _7 = D.10779.

[PATCH] c++: non-dependent immediate member fn call [PR99895]

2022-01-19 Thread Patrick Palka via Gcc-patches
Here we're emitting a bogus error during ahead of time evaluation of a non-dependent immediate member function call such as a.f(args) because the defacto templated form for such a call is (a.f)(args) but we're trying to evaluate it using the intermediate CALL_EXPR built by build_over_call, which ha

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Richard Sandiford via Gcc-patches
Martin Sebor writes: > On 1/19/22 03:09, Richard Sandiford wrote: >> Richard Biener writes: >>> On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches >>> wrote: In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXPR(D.10779) = .M

Re: [PATCH] tree-optimization/104114 - avoid diagnosing V1mode lowering

2022-01-19 Thread Richard Sandiford via Gcc-patches
Richard Biener via Gcc-patches writes: > Currently we diagnose vector lowering of V1mode operations that > are not natively supported into V_C_E, scalar op plus CTOR with > -Wvector-operation-performance but that's hardly useful behavior > even though the way we lower things can be improved. > > T

[committed] Update per-file selftest and finalization hooks for .c to .cc renaming

2022-01-19 Thread David Malcolm via Gcc-patches
On Wed, 2022-01-19 at 14:46 +0100, Richard Biener wrote: > On Wed, Jan 19, 2022 at 2:45 PM David Malcolm via Gcc-patches > wrote: > > > > This is mostly a mechanical change, apart from: > > - fix the name of opt_proposer_c to match its filename > > (opt-suggestions.cc) > > - delete a bogus "modre

Re: [PATCH] libstdc++: Fix for non-constexpr math_errhandling

2022-01-19 Thread Jonathan Wakely via Gcc-patches
On Wed, 19 Jan 2022 at 15:46, Matthias Kretz wrote: > > On Wednesday, 19 January 2022 16:21:15 CET Jonathan Wakely wrote: > > On Wed, 19 Jan 2022, 12:45 Matthias Kretz, wrote: > > > On Wednesday, 19 January 2022 13:07:26 CET Jonathan Wakely wrote: > > > > So is it a macro or not? > > > > > > I ag

Re: [PATCH] libstdc++: Fix for non-constexpr math_errhandling

2022-01-19 Thread Jonathan Wakely via Gcc-patches
On Wed, 19 Jan 2022 at 16:45, Jonathan Wakely wrote: > > On Wed, 19 Jan 2022 at 15:46, Matthias Kretz wrote: > > > > On Wednesday, 19 January 2022 16:21:15 CET Jonathan Wakely wrote: > > > On Wed, 19 Jan 2022, 12:45 Matthias Kretz, wrote: > > > > On Wednesday, 19 January 2022 13:07:26 CET Jonath

RE: [PATCH v5 1/1] [ARM] Add support for TLS register based stack protector canary access

2022-01-19 Thread Kyrylo Tkachov via Gcc-patches
Hi Ard, > -Original Message- > From: Gcc-patches bounces+kyrylo.tkachov=arm@gcc.gnu.org> On Behalf Of Ard > Biesheuvel via Gcc-patches > Sent: Monday, November 15, 2021 6:04 PM > To: linux-harden...@vger.kernel.org > Cc: Richard Sandiford ; > thomas.preudho...@celest.fr; Keith Packard

Re: [RFC] Port git gcc-descr to Python

2022-01-19 Thread Eric Gallager via Gcc-patches
On Wed, Jan 19, 2022 at 8:18 AM Martin Liška wrote: > > On 1/19/22 13:49, Martin Jambor wrote: > > Hi, > > > > On Wed, Jan 19 2022, Martin Liška wrote: > >> On 10/18/21 11:01, Martin Liška wrote: > >>> On 10/12/21 10:59, Martin Liška wrote: > Hello. > > There's a complete patch that

Re: [PATCH v5 1/1] [ARM] Add support for TLS register based stack protector canary access

2022-01-19 Thread Ard Biesheuvel via Gcc-patches
On Wed, 19 Jan 2022 at 17:54, Kyrylo Tkachov wrote: > > Hi Ard, > > > -Original Message- > > From: Gcc-patches > bounces+kyrylo.tkachov=arm@gcc.gnu.org> On Behalf Of Ard > > Biesheuvel via Gcc-patches > > Sent: Monday, November 15, 2021 6:04 PM > > To: linux-harden...@vger.kernel.org

Re: [PATCH] Make `-Werror` optional in libatomic/libbacktrace/libgomp/libitm/libsanitizer

2022-01-19 Thread Eric Gallager via Gcc-patches
On Mon, Jan 17, 2022 at 3:05 PM David Seifert via Gcc-patches wrote: > > * `-Werror` can cause issues when a more recent version of GCC compiles > an older version: > - https://bugs.gentoo.org/229059 > - https://bugs.gentoo.org/475350 > - https://bugs.gentoo.org/667104 > --- > libatomic/c

Re: [PATCH] waccess: Look at calls when tracking clobbers [PR104092]

2022-01-19 Thread Martin Sebor via Gcc-patches
On 1/19/22 09:22, Richard Sandiford wrote: Martin Sebor writes: On 1/19/22 03:09, Richard Sandiford wrote: Richard Biener writes: On Tue, Jan 18, 2022 at 2:40 PM Richard Sandiford via Gcc-patches wrote: In this PR the waccess pass was fed: D.10779 ={v} {CLOBBER}; VIEW_CONVERT_EXP

[PATCH] rs6000: Fix LE code gen for vec_cnt[lt]z_lsbb [PR95082]

2022-01-19 Thread Bill Schmidt via Gcc-patches
Hi! https://gcc.gnu.org/PR95082 demonstrates that we don't generate correct code for vec_cntlz_lsbb and vec_cnttz_lsbb for little-endian targets. This patch corrects the problem by marking the built-ins as bif_is_endian and using the correct target patterns for each endianness. Note that the de

[committed] libstdc++: Include for size_t and ptrdiff_t [PR104123]

2022-01-19 Thread Jonathan Wakely via Gcc-patches
Tested x86_64-linux, pushed to trunk. libstdc++-v3/ChangeLog: PR libstdc++/104123 * testsuite/29_atomics/headers/stdatomic.h/c_compat.cc: Include . --- .../testsuite/29_atomics/headers/stdatomic.h/c_compat.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

[PATCH v6 0/1] implement TLS register based stack canary for ARM

2022-01-19 Thread Ard Biesheuvel via Gcc-patches
Bugzilla: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102352 In the Linux kernel, user processes calling into the kernel are essentially threads running in the same address space, of a program that never terminates. This means that using a global variable for the stack protector canary value is p

[PATCH v6 1/1] [ARM] Add support for TLS register based stack protector canary access

2022-01-19 Thread Ard Biesheuvel via Gcc-patches
Add support for accessing the stack canary value via the TLS register, so that multiple threads running in the same address space can use distinct canary values. This is intended for the Linux kernel running in SMP mode, where processes entering the kernel are essentially threads running the same p

[PATCH v2] rs6000: More factoring of overload processing

2022-01-19 Thread Bill Schmidt via Gcc-patches
Hi! [I'm resubmitting this because the filename changed with the recent conversion from .c to .cc.] This patch continues the refactoring started with r12-6014. I had previously noted that the resolve_vec* routines can be further simplified by processing the argument list earlier, so that all rou

Re: [PATCH v3 04/15] arm: Add GENERAL_AND_VPR_REGS regclass

2022-01-19 Thread Andre Vieira (lists) via Gcc-patches
Hi Christophe, On 13/01/2022 14:56, Christophe Lyon via Gcc-patches wrote: At some point during the development of this patch series, it appeared that in some cases the register allocator wants “VPR or general” rather than “VPR or general or FP” (which is the same thing as ALL_REGS). The series

Re: [PATCH v3 05/15] arm: Add support for VPR_REG in arm_class_likely_spilled_p

2022-01-19 Thread Andre Vieira (lists) via Gcc-patches
On 13/01/2022 14:56, Christophe Lyon via Gcc-patches wrote: VPR_REG is the only register in its class, so it should be handled by TARGET_CLASS_LIKELY_SPILLED_P, which is achieved by calling default_class_likely_spilled_p. No test fails without this patch, but it seems it should be implemented.

Re: [PATCH] tree-optimization/103721 - Only add equivalencies that are still valid.

2022-01-19 Thread Andrew MacLeod via Gcc-patches
On 1/19/22 04:33, Richard Biener wrote: On Wed, Jan 19, 2022 at 2:37 AM Andrew MacLeod via Gcc-patches wrote: OK for trunk? OK. I don't quite understand how what you describe above works, it sounds a bit odd with respect to the idea that equivalences should be transitive but The transitive c

[PATCH v3] match.pd: Simplify 1 / X for integer X [PR95424]

2022-01-19 Thread Zhao Wei Liew via Gcc-patches
This patch implements an optimization for the following C++ code: int f(int x) { return 1 / x; } int f(unsigned int x) { return 1 / x; } Before this patch, x86-64 gcc -std=c++20 -O3 produces the following assembly: f(int): xor edx, edx mov eax, 1 idiv edi ret f(unsigned

Re: [PATCH v3 06/15] arm: Fix mve_vmvnq_n_ argument mode

2022-01-19 Thread Andre Vieira (lists) via Gcc-patches
On 13/01/2022 14:56, Christophe Lyon via Gcc-patches wrote: The vmvnq_n* intrinsics and have [u]int[16|32]_t arguments, so use iterator instead of HI in mve_vmvnq_n_. 2022-01-13 Christophe Lyon gcc/ * config/arm/mve.md (mve_vmvnq_n_): Use V_elem mode for operand 1.

Re: [PATCH] c++: non-dependent immediate member fn call [PR99895]

2022-01-19 Thread Jason Merrill via Gcc-patches
On 1/19/22 11:15, Patrick Palka wrote: Here we're emitting a bogus error during ahead of time evaluation of a non-dependent immediate member function call such as a.f(args) because the defacto templated form for such a call is (a.f)(args) but we're trying to evaluate it using the intermediate CAL

Re: [PATCH] c++: CTAD within alias template [PR91911]

2022-01-19 Thread Jason Merrill via Gcc-patches
On 1/3/22 10:24, Patrick Palka wrote: On Wed, 22 Dec 2021, Jason Merrill wrote: On 12/21/21 14:08, Patrick Palka wrote: On Tue, Dec 21, 2021 at 2:03 PM Patrick Palka wrote: On Wed, Jun 30, 2021 at 4:23 PM Jason Merrill wrote: On 6/30/21 4:18 PM, Patrick Palka wrote: On Wed, Jun 30, 2021

[r12-6740 Regression] FAIL: gcc.dg/vect/slp-reduc-1.c scan-tree-dump-times vect "vectorizing stmts using SLP" 1 on Linux/x86_64

2022-01-19 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, f4ca0a53be18dfc7162fd5dcc1e73c4203805e14 is the first bad commit commit f4ca0a53be18dfc7162fd5dcc1e73c4203805e14 Author: Andre Vieira Date: Wed Jan 19 14:11:32 2022 + vect: Fix epilogue mode skipping caused FAIL: gcc.dg/vect/bb-slp-cond-1.c -flto -ffat-lto-objects s

[committed] libstdc++: Remove -gdwarf-4 from flags for debug library

2022-01-19 Thread Jonathan Wakely via Gcc-patches
Tested x86_64-linux, pushed to trunk. Worth backporting to gcc-11 too. The default is -gdwarf-5 now, so this is hurting rather than improving things. libstdc++-v3/ChangeLog: * configure.ac (GLIBCXX_ENABLE_DEBUG_FLAGS): Remove -gdwarf-4 from default flags. * configure: R

[PATCH] libstdc++: Simplify build targets for debug library

2022-01-19 Thread Jonathan Wakely via Gcc-patches
Posting for review now, but I will wait until stage 1 to push this. This rewrites the stamp-debug and build-debug targets in src/Makefile so that each generated Makefile in the debug/$(SUBDIRS) directories is a make target, instead of being created by a loop in the stamp-debug recipe. The final a

Ping^5: [PATCH, rs6000 V2] rotate and mask constants [PR94393]

2022-01-19 Thread Pat Haugen via Gcc-patches
Ping. I'll note that I recently discovered that this patch also fixes PR93176 and PR97042. -Pat On 11/22/21 1:38 PM, Pat Haugen via Gcc-patches wrote: > Updated version of the patch. Changes made from original are updated > commentary to hopefully aid readability, no functional changes. > > >

Re: [PATCH] mips: Improved RTL representation of wsbh/dsbh/dshd

2022-01-19 Thread Maciej W. Rozycki
Hi Roger, > This patch to the mips backend updates the representations used > internally for MIPS' wsbh, dsbh and dshd instructions. These were > previously described using an UNSPEC rtx, which prevents simplification > at the RTL level. In addition to now being able to eliminate rotate > instru

[PATCH] Fortran: Fix scope for OMP AFFINITY clause iterator variables [PR103695]

2022-01-19 Thread Sandra Loosemore
This patch is for PR103695, marked as a P1 regression. OK to check in? -Sandra commit 21f8ac540b73e3838b63924e3c7e6c2ad25568ee Author: Sandra Loosemore Date: Wed Jan 19 12:50:49 2022 -0800 Fortran: Fix scope for OMP AFFINITY clause iterator variables [PR103695] gfc_finish_var_de

Re: [PATCH] Fortran: Fix scope for OMP AFFINITY clause iterator variables [PR103695]

2022-01-19 Thread Thomas Koenig via Gcc-patches
Hi Sandra, This patch is for PR103695, marked as a P1 regression.  OK to check in? I'm not an OpenMP expert, but this looks straightforward enough. I assume you ran a regression-test? OK if that is the case. Thanks for the patch! Best regards Thomas

Catch 'GIMPLE_DEBUG' misbehavior in OpenACC 'kernels' decomposition [PR100400, PR103836, PR104061] (was: Decompose OpenACC 'kernels' constructs into parts, a sequence of compute constructs)

2022-01-19 Thread Thomas Schwinge
Hi! On 2020-11-13T23:22:30+0100, I wrote: > I've pushed to master branch [...] commit > e898ce7997733c29dcab9c3c62ca102c7f9fa6eb "Decompose OpenACC 'kernels' > constructs into parts, a sequence of compute constructs", see attached. > > On 2019-02-01T00:59:30+0100, I wrote: >> There's more work to

Re: [PATCH] c++: CTAD within alias template [PR91911]

2022-01-19 Thread Patrick Palka via Gcc-patches
On Wed, 19 Jan 2022, Jason Merrill wrote: > On 1/3/22 10:24, Patrick Palka wrote: > > On Wed, 22 Dec 2021, Jason Merrill wrote: > > > > > On 12/21/21 14:08, Patrick Palka wrote: > > > > On Tue, Dec 21, 2021 at 2:03 PM Patrick Palka wrote: > > > > > > > > > > On Wed, Jun 30, 2021 at 4:23 PM Jaso

Re: [PATCH v2 1/2] add -Wuse-after-free

2022-01-19 Thread Jeff Law via Gcc-patches
On 1/11/2022 3:40 PM, Jason Merrill wrote: On 11/30/21 17:32, Martin Sebor via Gcc-patches wrote: Attached is a revised patch with the following changes based on your comments: 1) Set and use statement uids to determine which statement     precedes which in the same basic block. 2) Avoid tes

Re: Catch 'GIMPLE_DEBUG' misbehavior in OpenACC 'kernels' decomposition [PR100400, PR103836, PR104061] (was: Decompose OpenACC 'kernels' constructs into parts, a sequence of compute constructs)

2022-01-19 Thread Jakub Jelinek via Gcc-patches
On Wed, Jan 19, 2022 at 11:29:18PM +0100, Thomas Schwinge wrote: > (The pass is still disabled by default, by the way.) > > We've found that 'gcc/omp-oacc-kernels-decompose.cc' is currently not at > all considerate of 'GIMPLE_DEBUG' statements -- and it's not always > straight forward how to handl

[PATCH] constrain conservative string lengths to array sizes [PR104119]

2022-01-19 Thread Martin Sebor via Gcc-patches
The attached patch suppresses a class of unexpected -Wformat-overflow (and -truncation) warnings introduced as a result of better range info with the integration of the strlen pass with Ranger. The sprintf warning code relies on the strlen pass data to determine the lengths of string arguments to

Re: [PATCH, rs6000] Add a combine pattern for CA minus one [PR95737]

2022-01-19 Thread HAO CHEN GUI via Gcc-patches
On 19/1/2022 下午 3:52, Andrew Pinski wrote: > On Tue, Jan 18, 2022 at 11:13 PM HAO CHEN GUI via Gcc-patches > wrote: >> >> Hi, >>This patch adds a combine pattern for "CA minus one". As CA only has two >> values (0 or 1), we could convert following pattern >> (sign_extend:DI (plus:SI (

Re: [PATCH, rs6000] Add a combine pattern for CA minus one [PR95737]

2022-01-19 Thread Andrew Pinski via Gcc-patches
On Wed, Jan 19, 2022 at 6:12 PM HAO CHEN GUI wrote: > > > > On 19/1/2022 下午 3:52, Andrew Pinski wrote: > > On Tue, Jan 18, 2022 at 11:13 PM HAO CHEN GUI via Gcc-patches > > wrote: > >> > >> Hi, > >>This patch adds a combine pattern for "CA minus one". As CA only has two > >> values (0 or 1),

Re: [PATCH] c++: CTAD within alias template [PR91911]

2022-01-19 Thread Jason Merrill via Gcc-patches
On 1/19/22 17:32, Patrick Palka wrote: On Wed, 19 Jan 2022, Jason Merrill wrote: On 1/3/22 10:24, Patrick Palka wrote: On Wed, 22 Dec 2021, Jason Merrill wrote: On 12/21/21 14:08, Patrick Palka wrote: On Tue, Dec 21, 2021 at 2:03 PM Patrick Palka wrote: On Wed, Jun 30, 2021 at 4:23 PM Ja

[pushed] c++: template-id with current inst qualifier [PR102300]

2022-01-19 Thread Jason Merrill via Gcc-patches
The patch for PR41723 properly changed one place to look into the current instantiation; now we need to fix this place as well. Tested x86_64-pc-linux-gnu, applying to trunk. PR c++/102300 gcc/cp/ChangeLog: * parser.cc (cp_parser_template_name): Use dependent_scope_p. gcc/tests

Re: [pushed] c++: layout of aggregate base with DMI [PR103681]

2022-01-19 Thread Hans-Peter Nilsson via Gcc-patches
> From: Jason Merrill via Gcc-patches > Date: Fri, 17 Dec 2021 04:26:55 +0100 > C++14 changed the definition of 'aggregate' to allow default member > initializers, but such classes still need to be considered "non-POD for the > purpose of layout" for ABI compatibility with C++11 code. It seems r

[PATCH v2, rs6000] Add a combine pattern for CA minus one [PR95737]

2022-01-19 Thread HAO CHEN GUI via Gcc-patches
Hi, This patch adds a combine pattern for "CA minus one". As CA only has two values (0 or 1), we could convert following pattern (sign_extend:DI (plus:SI (reg:SI 98 ca) (const_int -1 [0x] to (plus:DI (reg:DI 98 ca) (const_int -1 [0xfff