Go patch commited: Fix initialization ordering

2022-04-15 Thread Ian Lance Taylor via Gcc-patches
compiler: revert `for package-scope "a = b; b = x" just set "a = x"` This patch to the Go frontend reverts https://go.dev/cl/245098, which introduced incorrect initialization ordering. This patch adjusts the runtime package to work even with that change reverted. Original description of change 2

Re: [PATCH] libstdc++: Avoid double-deref of __first in ranges::minmax [PR104858]

2022-04-15 Thread Patrick Palka via Gcc-patches
On Thu, 14 Apr 2022, Jonathan Wakely wrote: > On Thu, 14 Apr 2022 at 16:21, Patrick Palka via Libstdc++ > wrote: > > > > Tested on x86_64-pc-linux-gnu, does this look OK for trunk and 11/10 > > once the branch is unfrozen? > > > > PR libstdc++/104858 > > > > libstdc++-v3/ChangeLog: > > >

Re: [PATCH] testsuite: Skip pr105250.c for powerpc and s390 [PR105266]

2022-04-15 Thread Segher Boessenkool
On Thu, Apr 14, 2022 at 11:10:43AM +0800, Kewen.Lin wrote: > Hi, > > The test case pr105250.c is like its related pr105140.c, which > suffers the error with message like "{AltiVec,vector} argument > passed to unprototyped" on powerpc and s390. So like commits > r12-8025 and r12-8039, this fix is

[PATCH] PR target/89125

2022-04-15 Thread Steve Kargl via Gcc-patches
Can someone, anyone, please commit the attach patch, which is also attached to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89125 where one can read the audit trail. The original patch was submitted 2 years ago, and required manual intervention due to the recent *.c to *.cc rename. Back story:

Re: [wwwdocs] Add Ada's changelog entry

2022-04-15 Thread Fernando Oleo Blanco via Gcc-patches
Am Wed, 13 Apr 2022 23:04:17 -0600 (MDT) schrieb "Gerald Pfeifer" : > On Wed, 13 Apr 2022, Gerald Pfeifer wrote: > > This is the first batch of fixes. Pushed. > > Hmm, there is one issue that confused me a bit. What did you want to > relay by the following? > >+Pattern matching >+

Re: [PATCH] i386: Correct target attribute for crc32 intrinsics

2022-04-15 Thread Uros Bizjak via Gcc-patches
On Fri, Apr 15, 2022 at 10:34 AM Hongyu Wang wrote: > > > This test should not be changed, it correctly reports ISA mismatch. It > > even passes -mno-crc32. > > The error message changes from "needs isa option -mcrc32" to "target > specific option mismatch" with the #pragma change. > I see many of

[PATCH] c++: Fix up CONSTRUCTOR_PLACEHOLDER_BOUNDARY handling [PR105256]

2022-04-15 Thread Jakub Jelinek via Gcc-patches
Hi! The CONSTRUCTOR_PLACEHOLDER_BOUNDARY bit is supposed to separate PLACEHOLDER_EXPRs that should be replaced by one object or subobjects of it (variable, TARGET_EXPR slot, ...) from other PLACEHOLDER_EXPRs that should be replaced by different objects or subobjects. The bit is set when finding PL

Re: [PING] AArch64: add R30_REGNUM into shrink-wrapping separate

2022-04-15 Thread Dan Li via Gcc-patches
On 4/12/22 06:05, Richard Sandiford wrote: Dan Li writes: Gentile ping for this :), thanks. Link: https://gcc.gnu.org/pipermail/gcc-patches/2022-February/590906.html Sorry, I should have realised this at the time, but I don't think we can do this after all. The ABI requires us to set up

Re: [PATCH] i386: Correct target attribute for crc32 intrinsics

2022-04-15 Thread Hongyu Wang via Gcc-patches
> This test should not be changed, it correctly reports ISA mismatch. It > even passes -mno-crc32. The error message changes from "needs isa option -mcrc32" to "target specific option mismatch" with the #pragma change. I see many of our intrinsic would throw such error, it has been a long term iss

[PATCH] rs6000: Move V2DI vec_neg under power8-vector [PR105271]

2022-04-15 Thread Kewen.Lin via Gcc-patches
Hi, As PR105271 shows, __builtin_altivec_neg_v2di requires option -mpower8-vector as its pattern expansion relies on subv2di which has guard VECTOR_UNIT_P8_VECTOR_P (V2DImode). This fix is to move the related lines for __builtin_altivec_neg_v2di to the section of stanza power8-vector. Bootstrapp

Re: [PATCH] i386: Correct target attribute for crc32 intrinsics

2022-04-15 Thread Uros Bizjak via Gcc-patches
On Fri, Apr 15, 2022 at 6:30 AM Hongyu Wang wrote: > > Hi, > > Complile _mm_crc32_u8/16/32/64 intrinsics with -mcrc32 > would meet target specific option mismatch. Correct target pragma > to fix. > > Bootstrapped/regtest on x86_64-pc-linux-gnu{-m32,}. > > Ok for master and backport to GCC 11? > >

[PATCH] gcov-profile: Allow negavive counts of indirect calls [PR105282]

2022-04-15 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich TOPN metrics are histograms that contain overall count and per-bucket count. Overall count can be nevative when two profiles merge and some of per-bucket metrics are dropped. Noticed as an ICE on python PGO build where gcc crashes as: during IPA pass: modref a.c

Re: [PATCH v1] LoongArch: Fix bug for tmpdir-g++.dg-struct-layout-1/t033.

2022-04-15 Thread Lulu Cheng
Hi,all: This patch has been merged into the master branch. Thanks! Lulu Cheng 在 2022/4/11 下午3:08, Lulu Cheng 写道: From: chenglulu gcc/ChangeLog: * config/loongarch/loongarch.cc: Fix bug for tmpdir-g++.dg-struct-layout-1/t033. --- gcc/config/loongarch/loongarch.cc | 8 +--