Re: Hoisting DFmode loads out of loops..

2020-06-25 Thread Richard Biener via Gcc
On June 26, 2020 3:24:24 AM GMT+02:00, Alan Lehotsky wrote: >On Jun 25, 2020, at 6:37 PM, Jeff Law >mailto:l...@redhat.com>> wrote: > >On Thu, 2020-06-25 at 15:46 -0400, Alan Lehotsky wrote: >I’m working on a GCC 8.3 port to a load/store architecture with a >32-bit data-path between registers and

[Bug tree-optimization/95906] New: Failure to recognize max pattern with mask

2020-06-25 Thread gabravier at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95906 Bug ID: 95906 Summary: Failure to recognize max pattern with mask Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: Hoisting DFmode loads out of loops..

2020-06-25 Thread Alan Lehotsky
On Jun 25, 2020, at 6:37 PM, Jeff Law mailto:l...@redhat.com>> wrote: On Thu, 2020-06-25 at 15:46 -0400, Alan Lehotsky wrote: I’m working on a GCC 8.3 port to a load/store architecture with a 32-bit data-path between registers and memory; looking at the gcc.dg/loop-9.c test, I fail to pass

[Bug target/95905] Failure to optimize _mm_unpacklo_epi8 with 0 as right operand to _mm_cvtepu8_epi16

2020-06-25 Thread gabravier at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95905 --- Comment #1 from Gabriel Ravier --- The same pattern with _mm_unpacklo_epi16/32 and the corresponding SSE4 intrinsics can also be optimized in the same way.

[Bug target/95905] New: Failure to optimize _mm_unpacklo_epi8 with 0 as right operand to _mm_cvtepu8_epi16

2020-06-25 Thread gabravier at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95905 Bug ID: 95905 Summary: Failure to optimize _mm_unpacklo_epi8 with 0 as right operand to _mm_cvtepu8_epi16 Product: gcc Version: 11.0 Status: UNCONFIRMED

[Bug libstdc++/95904] Improve the diagnostic for conflicting return types in std::visit

2020-06-25 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95904 Jonathan Wakely changed: What|Removed |Added Ever confirmed|0 |1 Last reconfirmed|

Re: [PATCH] rs6000: Add support for __builtin_cpu_is ("power10")

2020-06-25 Thread Peter Bergner via Gcc-patches
On 6/25/20 6:54 PM, Segher Boessenkool wrote: > Okay for trunk (maybe leave out the comma part). Thanks! Ok, I pushed the patch without the ',' change. Thanks! Peter

Re: [PATCH] rs6000: Add support for __builtin_cpu_is ("power10")

2020-06-25 Thread Segher Boessenkool
Hi! On Thu, Jun 25, 2020 at 06:36:51PM -0500, Peter Bergner wrote: > rs6000: Add support for __builtin_cpu_is ("power10") > > Add support for __builtin_cpu_is ("power10"). Also add documentation for > the recently added "arch_3_1" and "mma" __builtin_cpu_supports arguments. > gcc/ > *

[Bug libstdc++/95904] New: Improve the diagnostic for conflicting return types in std::visit

2020-06-25 Thread rs2740 at gmail dot com
: /opt/compiler-explorer/gcc-trunk-20200625/include/c++/11.0.0/variant: In instantiation of 'static constexpr auto std::__detail::__variant::__gen_vtable_impl, std::integer_sequence >::_S_apply() [with _Result_type = std::__detail::__variant::__deduce_visit_result; _Visitor = main()::&&

[PATCH] rs6000: Add support for __builtin_cpu_is ("power10")

2020-06-25 Thread Peter Bergner via Gcc-patches
rs6000: Add support for __builtin_cpu_is ("power10") Add support for __builtin_cpu_is ("power10"). Also add documentation for the recently added "arch_3_1" and "mma" __builtin_cpu_supports arguments. This passed bootstrap and regtesting with no regressions. Ok for trunk? Peter gcc/

[Bug c++/90333] [9 Regression] Can't apply attributes to lambdas with trailing returns

2020-06-25 Thread ttimo at valvesoftware dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90333 --- Comment #9 from Timothee Besset --- First test case is still a problem in 9.3 and 10: []() __attribute__((always_inline)) -> int { return 0; } Fix proposed in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95883

[Bug c++/89640] [9 Regression] g++ chokes on lambda with __attribute__

2020-06-25 Thread ttimo at valvesoftware dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89640 --- Comment #13 from Timothee Besset --- Fix proposed in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95883

[Bug c++/95883] Attributes on lambdas appear to be parsed in the wrong place

2020-06-25 Thread ttimo at valvesoftware dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95883 --- Comment #2 from Timothee Besset --- Created attachment 48787 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48787=edit Add a parse for gnu attributes before CPP_DEREF, fixes regression Proposed patch. Add a parsing for gnu attributes

[Bug middle-end/95903] New: gcc 10: wrong code with -fwrapv

2020-06-25 Thread markus at oberhumer dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95903 Bug ID: 95903 Summary: gcc 10: wrong code with -fwrapv Product: gcc Version: 10.1.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: middle-end

[Bug c++/91104] auto... in lambda expression reorders parameters

2020-06-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91104 Marek Polacek changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug c++/91104] auto... in lambda expression reorders parameters

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91104 --- Comment #3 from CVS Commits --- The master branch has been updated by Marek Polacek : https://gcc.gnu.org/g:08ca2d744d50d0d5a341530031e6bbb82831109f commit r11-1668-g08ca2d744d50d0d5a341530031e6bbb82831109f Author: Marek Polacek Date:

[committed] c++: Add test for c++/91104

2020-06-25 Thread Marek Polacek via Gcc-patches
Fixed by r271705. Tested x86_64-pc-linux-gnu, applying to trunk. gcc/testsuite/ChangeLog: PR c++/91104 * g++.dg/cpp1y/lambda-generic-variadic21.C: New test. --- .../g++.dg/cpp1y/lambda-generic-variadic21.C | 26 +++ 1 file changed, 26 insertions(+) create mode

Re: [PATCH 5/6 ver 3] rs6000, Add vector splat builtin support

2020-06-25 Thread Segher Boessenkool
Hi! On Thu, Jun 18, 2020 at 03:20:18PM -0700, Carl Love wrote: > * config/rs6000/altivec.md (UNSPEC_XXSPLTIW, UNSPEC_XXSPLTID, > UNSPEC_XXSPLTI32DX): New. > (vxxspltiw_v4si, vxxspltiw_v4sf_inst, vxxspltidp_v2df_inst, > vxxsplti32dx_v4si_inst, vxxsplti32dx_v4sf_inst): New

Re: Hoisting DFmode loads out of loops..

2020-06-25 Thread Jeff Law via Gcc
On Thu, 2020-06-25 at 15:46 -0400, Alan Lehotsky wrote: > I’m working on a GCC 8.3 port to a load/store architecture with a 32-bit > data-path between registers and memory; > > looking at the gcc.dg/loop-9.c test, I fail to pass because I have split the > move of a double constant to memory

gcc-8-20200625 is now available

2020-06-25 Thread GCC Administrator via Gcc
Snapshot gcc-8-20200625 is now available on https://gcc.gnu.org/pub/gcc/snapshots/8-20200625/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 8 git branch with the following options: git://gcc.gnu.org/git/gcc.git branch

[PATCH] libgomp: added simple functions and tests for OMPD

2020-06-25 Thread y2s1982 via Gcc-patches
From: y2s82 <1997371+y2...@users.noreply.github.com> This patch adds some unit tests for omp-tools.h header. It also adds some simple functions variables related to OMPD API versions and dll locations. It also partially defines the OMPD initialization function. More OMPD configuration is also

[Bug c++/89640] [9 Regression] g++ chokes on lambda with __attribute__

2020-06-25 Thread ttimo at valvesoftware dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89640 Timothee Besset changed: What|Removed |Added CC||ttimo at valvesoftware dot com ---

Re: [PATCH v2] c++: Fix CTAD for aggregates in template [PR95568]

2020-06-25 Thread Marek Polacek via Gcc-patches
On Wed, Jun 24, 2020 at 03:52:12PM -0400, Jason Merrill via Gcc-patches wrote: > On 6/23/20 6:58 PM, Marek Polacek wrote: > > 95568 complains that CTAD for aggregates doesn't work within > > requires-clause and it turned out that it doesn't work when we try > > the deduction in a template. The

[Bug c++/91416] ICE in cp_check_const_attributes, at cp/decl2.c:1408

2020-06-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91416 --- Comment #10 from Marek Polacek --- *** Bug 95902 has been marked as a duplicate of this bug. ***

[Bug c++/95902] [10/11 Regression] ICE: tree check: expected tree_list, have ggc_freed in get_attribute_name, at attribs.c:841

2020-06-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95902 Marek Polacek changed: What|Removed |Added Resolution|--- |DUPLICATE Status|NEW

[Bug c++/95902] [10/11 Regression] ICE: tree check: expected tree_list, have ggc_freed in get_attribute_name, at attribs.c:841

2020-06-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95902 Marek Polacek changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug c++/95902] [10/11 Regression] ICE: tree check: expected tree_list, have ggc_freed in get_attribute_name, at attribs.c:841

2020-06-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95902 Marek Polacek changed: What|Removed |Added Ever confirmed|0 |1 Keywords|

Re: [PATCH 4/6 ver 3] rs6000, Add vector shift double builtin support

2020-06-25 Thread Segher Boessenkool
Hi! On Thu, Jun 18, 2020 at 03:20:14PM -0700, Carl Love wrote: > +;; Map UNSPEC_SLDB to "l" and UNSPEC_SRDB to "r". > +(define_int_attr SLDB_LR [(UNSPEC_SLDB "l") > + (UNSPEC_SRDB "r")]) Most (all?) other attributes that always result in lower case strings are written in

[Bug fortran/93423] [8/9/10/11 Regression] ICE on invalid with argument list for module procedure

2020-06-25 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93423 anlauf at gcc dot gnu.org changed: What|Removed |Added CC||anlauf at gcc dot gnu.org

[PATCH] [8/9/10/11 Regression] ICE on invalid with argument list for module procedure

2020-06-25 Thread Harald Anlauf
A simple situation where a NULL pointer dereference occurs during error recovery. Regtested on x86_64-pc-linux-gnu. OK for master / backports? Thanks, Harald PR fortran/93423 - ICE on invalid with argument list for module procedure When recovering from an error, a NULL pointer dereference

[Bug c++/95902] New: ICE: tree check: expected tree_list, have ggc_freed in get_attribute_name, at attribs.c:841

2020-06-25 Thread p.vanhoof at oma dot be
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95902 Bug ID: 95902 Summary: ICE: tree check: expected tree_list, have ggc_freed in get_attribute_name, at attribs.c:841 Product: gcc Version: 11.0 Status: UNCONFIRMED

Re: TLS Implementation Across Architectures

2020-06-25 Thread Andrew Pinski via Gcc
On Thu, Jun 25, 2020 at 1:34 PM Joel Sherrill wrote: > > On Thu, Jun 25, 2020 at 2:54 PM Nathan Sidwell wrote: > > > On 6/25/20 2:34 PM, Joel Sherrill wrote: > > > Hi > > > > > > RTEMS supports over 15 processor architectures and we would like to > > ensure > > > that TLS is supported on all

[Bug c/95901] New: [C, C++] OpenACC, OpenMP executable directives wrongly refused after labels

2020-06-25 Thread tschwinge at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95901 Bug ID: 95901 Summary: [C, C++] OpenACC, OpenMP executable directives wrongly refused after labels Product: gcc Version: 11.0 Status: UNCONFIRMED Keywords:

Re: TLS Implementation Across Architectures

2020-06-25 Thread Joel Sherrill
On Thu, Jun 25, 2020 at 2:54 PM Nathan Sidwell wrote: > On 6/25/20 2:34 PM, Joel Sherrill wrote: > > Hi > > > > RTEMS supports over 15 processor architectures and we would like to > ensure > > that TLS is supported on all rather than just a handful of popular ones > > (arm, x86, powerpc, sparc,

[Bug fortran/95877] [8/9 regression] ICE in test case gfortran.dg/pr95689.f90 after r9-8693

2020-06-25 Thread seurer at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95877 Bill Seurer changed: What|Removed |Added Resolution|--- |FIXED Status|WAITING

[Bug fortran/93337] [9/10/11 Regression] ICE in gfc_dt_upper_string, at fortran/module.c:441

2020-06-25 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93337 anlauf at gcc dot gnu.org changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |anlauf at gcc dot

[PATCH] [9/10/11 Regression] PR fortran/93337 - ICE in gfc_dt_upper_string, at fortran/module.c:441

2020-06-25 Thread Harald Anlauf
This PR is due to a plain NULL pointer that needs to get caught in the right place. Regtested on x86_64-pc-linux-gnu. OK for master / backports where applicable? Thanks, Harald PR fortran/93337 - ICE in gfc_dt_upper_string, at fortran/module.c:441 When declaring a polymorphic variable that

[PATCH] m68k: tag floating-point ABI used

2020-06-25 Thread Pat Bernardi
Tag the floating-point calling convention used on m68k-elf (either hard-float or soft-float) through the GNU assembler attribute. The use of the tag enables the linker to ensure linked objects use a consistent floating-point ABI and allows tools like GDB to infer the ABI used from the ELF file. It

[Bug fortran/93337] [9/10/11 Regression] ICE in gfc_dt_upper_string, at fortran/module.c:441

2020-06-25 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93337 anlauf at gcc dot gnu.org changed: What|Removed |Added CC||anlauf at gcc dot gnu.org

Re: TLS Implementation Across Architectures

2020-06-25 Thread Nathan Sidwell
On 6/25/20 2:34 PM, Joel Sherrill wrote: Hi RTEMS supports over 15 processor architectures and we would like to ensure that TLS is supported on all rather than just a handful of popular ones (arm, x86, powerpc, sparc, etc). I know of Ulrich Drepper's document (

Hoisting DFmode loads out of loops..

2020-06-25 Thread Alan Lehotsky
I’m working on a GCC 8.3 port to a load/store architecture with a 32-bit data-path between registers and memory; looking at the gcc.dg/loop-9.c test, I fail to pass because I have split the move of a double constant to memory into multiple moves (4 in fact, because I only have a 16-bit

Re: [PATCH] Treat { 0 } specially for structs with the designated_init attribute.

2020-06-25 Thread Asher Gordon via Gcc-patches
Joseph Myers writes: > I think both the patches in this discussion (special { 0 } handling > and the new warning option) generally look good. I also wrote another small patch, which you might have missed since it's buried in the discussion here:

[Bug fortran/95827] ICE in gfc_get_string, at fortran/iresolve.c:70

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95827 --- Comment #4 from CVS Commits --- The releases/gcc-10 branch has been updated by Harald Anlauf : https://gcc.gnu.org/g:69240fb9428513687d451af19fb762687fe65786 commit r10-8370-g69240fb9428513687d451af19fb762687fe65786 Author: Harald Anlauf

[Bug fortran/95877] [8/9 regression] ICE in test case gfortran.dg/pr95689.f90 after r9-8693

2020-06-25 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95877 anlauf at gcc dot gnu.org changed: What|Removed |Added Status|NEW |WAITING --- Comment #2 from

[Bug fortran/95689] [8/9/10/11 Regression] ICE in check_sym_interfaces, at fortran/interface.c:2015

2020-06-25 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95689 --- Comment #11 from anlauf at gcc dot gnu.org --- Reverted on 8- and 9-branch, since not a critical regression, and a clean backport would need a manual backport of a series of other fixes.

Re: [PATCH] PR fortran/95828 - Buffer overflows with SELECT RANK

2020-06-25 Thread Harald Anlauf
Hi Thomas, > Thanks for tackling this! I haven't checked, but you are > probably setting a new record for bug fixes per second now :-) I'm trying hard, but I definitely cannot outpace Gerhard. Any attempt in this regard seems futile. |-: The good thing is that while hunting down things

[Bug fortran/95828] ICE in resolve_select_rank, at fortran/resolve.c:9774

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95828 --- Comment #4 from CVS Commits --- The master branch has been updated by Harald Anlauf : https://gcc.gnu.org/g:77d455ee81ec3a23f8b20259a31ab963716f8e82 commit r11-1667-g77d455ee81ec3a23f8b20259a31ab963716f8e82 Author: Harald Anlauf Date:

TLS Implementation Across Architectures

2020-06-25 Thread Joel Sherrill
Hi RTEMS supports over 15 processor architectures and we would like to ensure that TLS is supported on all rather than just a handful of popular ones (arm, x86, powerpc, sparc, etc). I know of Ulrich Drepper's document ( https://www.akkadia.org/drepper/tls.pdf) but it is a few years old and

[Bug fortran/95826] ICE in gfc_match_decl_type_spec, at fortran/decl.c:4290

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95826 --- Comment #3 from CVS Commits --- The master branch has been updated by Harald Anlauf : https://gcc.gnu.org/g:35a335a159216548fc77263ac5df71ff29d3f448 commit r11-1666-g35a335a159216548fc77263ac5df71ff29d3f448 Author: Harald Anlauf Date:

Re: [PATCH] Treat { 0 } specially for structs with the designated_init attribute.

2020-06-25 Thread Joseph Myers
I think both the patches in this discussion (special { 0 } handling and the new warning option) generally look good. I don't see you in the FSF copyright assignment list; could you complete https://git.savannah.gnu.org/cgit/gnulib.git/plain/doc/Copyright/request-assign.future (unless you're

Re: [PATCH] libgomp: added simple functions and tests for OMPD

2020-06-25 Thread Jakub Jelinek via Gcc-patches
On Thu, Jun 25, 2020 at 12:27:32PM -0400, y2s1982 . via Gcc-patches wrote: > > > --- /dev/null > > > +++ b/libgomp/libgompd.map > > > @@ -0,0 +1,49 @@ > > > +OMPD_5.0 { > > > + global: > > > + ompd_dll_locations_valid; > > > > ompd_dll_locations and ompd_dll_locations_valid both need to be

Re: [PATCH] libgomp: added simple functions and tests for OMPD

2020-06-25 Thread Jakub Jelinek via Gcc-patches
On Thu, Jun 25, 2020 at 12:17:05PM -0400, y2s1982 . wrote: > Ah, so code in env.c gets executed before OMPD gets started? Not all, but initialize_env is the library constructor (has constructor attribute) and is invoked during the library initialization. > > Include omp-tools.h and

Re: [PATCH] Treat { 0 } specially for structs with the designated_init attribute.

2020-06-25 Thread Joseph Myers
On Wed, 24 Jun 2020, Asher Gordon via Gcc-patches wrote: > I see. So perhaps this isn't the best way to go about implementing > attribute locations. What do you think would be a better way? Perhaps > using a DECL_MINIMAL for attributes? In general, too many things in GCC have the static type

[Bug rtl-optimization/90174] Bad register spill due to top-down allocation order

2020-06-25 Thread vmakarov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90174 --- Comment #13 from Vladimir Makarov --- (In reply to Tamar Christina from comment #12) > (In reply to Vladimir Makarov from comment #11) > > I just expressed my point of view to the bottom-up approach. If somebody > > implements any new RA

Re: [PATCH 7/7] PowerPC test: Add prefixed stack protect test

2020-06-25 Thread Segher Boessenkool
Hi! On Mon, Jun 01, 2020 at 03:53:42PM -0400, Michael Meissner wrote: > Test that stack protection generates prefixed stack instructions if you are > using large stack frame for -mcpu=future. > > 2020-06-01 Michael Meissner > > * gcc.target/powerpc/prefix-stack-protect.c: New test. >

Re: [PATCH 6/7, V2] PowerPC tests: Add PC-relative tests.

2020-06-25 Thread Segher Boessenkool
On Thu, Jun 04, 2020 at 01:05:12PM -0400, Michael Meissner wrote: > [PATCH 6/7, V2] PowerPC tests: Add PC-relative tests. > > These tests make sure that PC-relative variant is generated for -mcpu=future > on > systems that support PC-relative addressing. The only difference > is I reworded the

Re: [PATCH 5/7, V2] PowerPC tests: Prefixed insn with large offsets

2020-06-25 Thread Segher Boessenkool
Hi! On Thu, Jun 04, 2020 at 01:03:51PM -0400, Michael Meissner wrote: > [PATCH 5/7, V2] PowerPC tests: Prefixed insn with large offsets > > Add tests to make sure for -mcpu=future that prefixed load/store instructions > are generated if the offset is larger than 16 bits. The only difference >

Re: [PATCH 4/7] PowerPC test: Add prefixed no update test

2020-06-25 Thread Segher Boessenkool
On Mon, Jun 01, 2020 at 03:53:39PM -0400, Michael Meissner wrote: > This test makes sure we do not generate a prefixed instruction with an update > form. > > 2020-06-01 Michael Meissner > > * gcc.target/powerpc/prefix-no-update.c: New test. Okay like 2 and 3. Thanks! Segher

Re: [PATCH 3/7] PowerPC tests: Add prefixed vs. DS/DQ instruction tests.

2020-06-25 Thread Segher Boessenkool
On Mon, Jun 01, 2020 at 03:53:38PM -0400, Michael Meissner wrote: > Add test to make sure prefixed load/store instructions are generated if the > offset would not fit in the DS/DQ encodings. > > 2020-06-01 Michael Meissner > > * gcc.target/powerpc/prefix-ds-dq.c: New test. Okay for

Re: RFA: Remove use of register keyword in libiberty.h

2020-06-25 Thread Ian Lance Taylor
Nick Clifton writes: > Hi Nick, Hi Ian, >>> In file included from gold/archive.cc:29: >>> include/libiberty.h:646:25: error: 'register' storage class >>> specifier is deprecated and incompatible with C++17 >>> [-Werror,-Wdeprecated-register] >>> >>> So I would like to

Re: [PATCH 2/7] PowerPC tests: Add PLI/PADDI tests.

2020-06-25 Thread Segher Boessenkool
On Mon, Jun 01, 2020 at 03:53:37PM -0400, Michael Meissner wrote: > Add tests for -mcpu=future that test the generation of PADDI (and PLI which > becomes PADDI). > > 2020-06-01 Michael Meissner > > * gcc.target/powerpc/prefix-add.c: New test. > *

Re: [PATCH] libgomp: added simple functions and tests for OMPD

2020-06-25 Thread y2s1982 . via Gcc-patches
> > > > --- /dev/null > > +++ b/libgomp/libgompd.map > > @@ -0,0 +1,49 @@ > > +OMPD_5.0 { > > + global: > > + ompd_dll_locations_valid; > > ompd_dll_locations and ompd_dll_locations_valid both need to be exported, > but not from libgompd.so.1 but from libgomp.so.1, so they need to go into >

Re: [PATCH] libgomp: added simple functions and tests for OMPD

2020-06-25 Thread y2s1982 . via Gcc-patches
Hello Jakub, Thank you for the feedback. I can't believe I made some careless mistakes. I am also very interested in learning some new techniques mentioned. I had some questions about them. On Thu, Jun 25, 2020 at 4:28 AM Jakub Jelinek wrote: > > > --- /dev/null > > +++ b/libgomp/libgompd.map

PING: Re: Move simplification of statements using ranges into its own class.

2020-06-25 Thread Aldy Hernandez via Gcc-patches
PING On Thu, Jun 18, 2020, 13:24 Aldy Hernandez wrote: > Howdy. > > This moves all the simplification code from vr_values into a separate > class (simplify_using_ranges). In doing so, we get rid of a bunch of > dependencies on the internals of vr_values. The goal is to (a) remove >

[Bug c++/95888] [9/10/11 Regression] Regression in 9.3. GCC freezes when compiling code using boost::poly_collection::segment

2020-06-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95888 Marek Polacek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug testsuite/95900] New: New test case gcc.dg/vect/bb-slp-pr95866.c in r11-1647 fails

2020-06-25 Thread seurer at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95900 Bug ID: 95900 Summary: New test case gcc.dg/vect/bb-slp-pr95866.c in r11-1647 fails Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal

Re: [COMMITTED} m32r: Disable movsicc pattern

2020-06-25 Thread Jeff Law via Gcc-patches
On Thu, 2020-06-25 at 16:41 +0100, Nick Clifton via Gcc-patches wrote: > Hi Guys, > > I am checking in the patch below to fix several failures in the GCC > testsuite for the M32R target. The issue is the movsicc pattern which > is a holdover from when the port from converted from using

[Bug middle-end/95899] -funroll-loops does not duplicate accumulators when calculating reductions, failing to break up dependency chains

2020-06-25 Thread elrodc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95899 --- Comment #2 from Chris Elrod --- Interesting. Compiling with: gcc -march=native -fvariable-expansion-in-unroller -Ofast -funroll-loops -S dot.c -o dot.s Yields: ``` .L4: vmovupd (%rdi,%r11), %zmm9 vmovupd 64(%rdi,%r11),

[Bug d/95075] gcc/d/dmd/dscope.c: 2 * strange assignments ?

2020-06-25 Thread ibuclaw at gdcproject dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95075 Iain Buclaw changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[committed] d: Remove another dependency on the front-end OutBuffer type.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch removes another dependency on the front-end OutBuffer type. As the DMD front-end never frees allocated memory, the glue layer between the DMD front-end and GCC should generally avoid using DMD types and interfaces if the purpose is internal only. Bootstrapped and regression

[COMMITTED} m32r: Disable movsicc pattern

2020-06-25 Thread Nick Clifton via Gcc-patches
Hi Guys, I am checking in the patch below to fix several failures in the GCC testsuite for the M32R target. The issue is the movsicc pattern which is a holdover from when the port from converted from using cc0. The pattern was written as if a previous instruction had set the CC bits,

[committed] d: Turn on deprecation warnings by default.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch makes deprecation warnings the default in gdc, fixing both the library and any tests that are currently using deprecated styles. This is the default in the upstream reference compiler, and can reduce some confusion when comparing warning/error messages of gdc and dmd side by side.

[committed] d: Move d_signed_type and d_unsigned_type to types.cc

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch moves d_signed_type and d_unsigned_type to types.cc. These two functions are not tied to the language-specific part of the front-end in any way. Bootstrapped and regression tested on x86_64-linux-gnu, and committed to mainline. Regards Iain. --- gcc/d/ChangeLog: *

[committed] d: Merge upstream dmd 90450f3ef.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch merges the D front-end implementation with upstream dmd 90450f3ef. Fixes a regression caused by an incomplete backport of converting the Expression semantic pass to a Visitor. Bootstrapped and regression tested on x86_64-linux-gnu, and committed to mainline. Regards Iain. ---

[committed] d: Merge upstream dmd 5fc1806cd.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch merges the D front-end implementation with upstream dmd 5fc1806cd. Backports the OutBuffer interface from upstream dmd master, removing another difference this and the self-hosted D branch that is purely refactoring, and doesn't introduce any mechanical changes. Bootstrapped and

[committed] d: Merge upstream dmd 4f1046222.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch merges the D front-end implementation with upstream dmd 4f1046222. Renames OnScopeStatement to ScopeGuardStatement. Bootstrapped and regression tested on x86_64-linux-gnu, and committed to mainline. Regards Iain. --- gcc/d/ChangeLog: * dmd/MERGE: Merge upstream dmd

[committed] d: Do not implicitly set DECL_DECLARED_INLINE_P on member functions.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch removes the implicit setting of DECL_DECLARATED_INLINE_P on member function. This has been questionable behaviour since it was added, and though it has no effect on wider discussions around what should be the correct semantics of pragma(inline) within D modules, doing this

[committed] d: Merge upstream dmd 4be011355.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch merges the D front-end implementation with upstream dmd 4be011355. Fixes self-assignment warnings seen when compiling with clang. Bootstrapped and regression tested on x86_64-linux-gnu, and committed to mainline. Regards Iain. --- gcc/d/ChangeLog: PR d/95075 *

[committed] d: Fix ICE in uda_attribute_p when looking up unknown attribute

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch fixes an ICE in uda_attribute_p when looking up an unknown attribute. The target attribute table is not guaranteed to be set by all backends. Bootstrapped and regression tested on x86_64-linux-gnu, and committed to mainline. Regards Iain. --- gcc/d/ChangeLog: PR

[committed] d: Don't set DECL_INITIAL if initializer is 'void'.

2020-06-25 Thread Iain Buclaw via Gcc-patches
Hi, This patch fixes a logic bug in the code-gen for variable declarations. Declarations initialized with `= void` were being default initialized. That is not really the intent, and misses the small optimization that should have been gained from using void initializations. Bootstrapped and

[Bug d/95173] [D] ICE on some architecture targets when trying to use unknown attribute

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95173 --- Comment #1 from CVS Commits --- The master branch has been updated by Iain Buclaw : https://gcc.gnu.org/g:62e02c8729a75c4a859edc18e0bcafb87d717f46 commit r11-1658-g62e02c8729a75c4a859edc18e0bcafb87d717f46 Author: Iain Buclaw Date: Tue

[Bug d/95075] gcc/d/dmd/dscope.c: 2 * strange assignments ?

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95075 --- Comment #3 from CVS Commits --- The master branch has been updated by Iain Buclaw : https://gcc.gnu.org/g:1b7b352a8379dbdd74c962d11951982e21e20f13 commit r11-1662-g1b7b352a8379dbdd74c962d11951982e21e20f13 Author: Iain Buclaw Date: Wed

[Bug d/95250] [D] ICE instead of error when trying to use bad template type inside template

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95250 --- Comment #2 from CVS Commits --- The master branch has been updated by Iain Buclaw : https://gcc.gnu.org/g:1d19c9cd3dd99ba8885c0fc93223918399f73939 commit r11-1661-g1d19c9cd3dd99ba8885c0fc93223918399f73939 Author: Iain Buclaw Date: Tue

[Bug middle-end/95899] -funroll-loops does not duplicate accumulators when calculating reductions, failing to break up dependency chains

2020-06-25 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95899 ktkachov at gcc dot gnu.org changed: What|Removed |Added CC||ktkachov at gcc dot gnu.org

Re: [PATCH] coroutines: Generalize promise expressions [PR95519]

2020-06-25 Thread Nathan Sidwell
On 6/25/20 11:15 AM, Iain Sandoe wrote: Hi, The PR points out that the standard does not restrict promise expressions to methods, but the current implementation does. The patch factors out the building of a general promise expression, and then uses it in a fairly mechanical replacement of each

[Bug fortran/67311] ICE calling subroutine with derived type as argument within OpenMP parallel region

2020-06-25 Thread burnus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67311 --- Comment #8 from Tobias Burnus --- Submitted patch: https://gcc.gnu.org/pipermail/gcc-patches/2020-June/548920.html

[PATCH] coroutines: Generalize promise expressions [PR95519]

2020-06-25 Thread Iain Sandoe
Hi, The PR points out that the standard does not restrict promise expressions to methods, but the current implementation does. The patch factors out the building of a general promise expression, and then uses it in a fairly mechanical replacement of each case that we need such an expressions.

[Bug middle-end/95899] New: -funroll-loops does not duplicate accumulators when calculating reductions, failing to break up dependency chains

2020-06-25 Thread elrodc at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95899 Bug ID: 95899 Summary: -funroll-loops does not duplicate accumulators when calculating reductions, failing to break up dependency chains Product: gcc Version:

[Bug fortran/95837] derived-type components of character kind=4 – wrong code with component access (kind=4 ignored)

2020-06-25 Thread burnus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95837 Tobias Burnus changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug fortran/95837] derived-type components of character kind=4 – wrong code with component access (kind=4 ignored)

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95837 --- Comment #6 from CVS Commits --- The master branch has been updated by Tobias Burnus : https://gcc.gnu.org/g:f48bffe70cba310461ec19ffcd07c573a6b86575 commit r11-1654-gf48bffe70cba310461ec19ffcd07c573a6b86575 Author: Tobias Burnus Date:

[Bug tree-optimization/95839] Failure to optimize addition of vector elements to vector addition

2020-06-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95839 --- Comment #7 from Richard Biener --- OK, I have a patch doing BIT_FIELD_REFs.

[Bug debug/95898] New: Wrong backtrace infromation at Og

2020-06-25 Thread massarelli at diag dot uniroma1.it
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95898 Bug ID: 95898 Summary: Wrong backtrace infromation at Og Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: debug

Re: [Patch][gcn, nvptx, offloading] mkoffload – handle -fpic/-fPIC

2020-06-25 Thread Thomas Schwinge
Hi Tobias! On 2020-06-24T15:36:25+0200, Tobias Burnus wrote: > [...] who knows > what users are doing... Indeed. ;-) > On 6/23/20 9:36 PM, Thomas Schwinge wrote: >> I don't think I can approve, but seems fine if this works (as you've >> confirmed) -- it's one incremental step forward!

[Bug fortran/95869] ICE when "target parallel" construct used with "if" clause in Fortran

2020-06-25 Thread kcy at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95869 Kwok Yeung changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

Re: Fortran: Fix character-kind=4 substring resolution (PR95837)

2020-06-25 Thread Thomas Koenig via Gcc-patches
Hi Tobias, OK – with assert or without? I don't think the assert is needed - if things go wrong there, then I am quite sure that we will get all sorts of ICEs downstream if this is not set correctly somewhere after all, and your testing indicates that it is not. Besides, I like the elegance

[Bug c++/95895] internal compiler error: in captures_temporary, at cp/coroutines.cc:2717

2020-06-25 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95895 Iain Sandoe changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |iains at gcc dot gnu.org

[Bug tree-optimization/95897] New: [11 regression] gfortran.dg/graphite/pr40982.f90 fails with ICE starting with r11-1649

2020-06-25 Thread seurer at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95897 Bug ID: 95897 Summary: [11 regression] gfortran.dg/graphite/pr40982.f90 fails with ICE starting with r11-1649 Product: gcc Version: 11.0 Status: UNCONFIRMED

[Patch][Fortran, OpenMP] Fix allocatable-components check (PR67311)

2020-06-25 Thread Tobias Burnus
The test case was giving an ICE as it recursively called itself. Rather obvious fix. OK for the trunk? Tobias - Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Alexander Walter

[Bug tree-optimization/95839] Failure to optimize addition of vector elements to vector addition

2020-06-25 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95839 --- Comment #6 from CVS Commits --- The master branch has been updated by Richard Biener : https://gcc.gnu.org/g:628b78f9794a2eefcbc578011806bfa8e09b9ef7 commit r11-1653-g628b78f9794a2eefcbc578011806bfa8e09b9ef7 Author: Richard Biener Date:

[Bug c++/95809] GCC treats inline namespace declaration as "ambiguous"

2020-06-25 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95809 --- Comment #2 from Nathan Sidwell --- yup, dr2061 made that ill-formed. p1701 (wg21.link/p1701) documents the behaviour and it appears EWG is exploring another avenue to resolve the underlying problem 2061 was attempting to fix.

  1   2   >