[pushed] c++: ICE with anonymous union [PR97974]

2021-04-10 Thread Jason Merrill via Gcc-patches
Here lookup got confused by finding a conversion operator from lookup_anon_field. Let's avoid this by pruning functions from CLASSTYPE_MEMBER_VEC as well as TYPE_FIELDS. Tested x86_64-pc-linux-gnu, applying to trunk. gcc/cp/ChangeLog: PR c++/97974 * decl.c (fixup_anonymous_aggr)

[Patch, fortran] PR fortran/84006, PR fortran/100027 - ICE on storage_size with polymorphic argument

2021-04-10 Thread José Rui Faustino de Sousa via Gcc-patches
Hi All! Proposed patch to: PR84006 - [8/9/10/11 Regression] ICE in storage_size() with CLASS entity PR100027 - ICE on storage_size with polymorphic argument Patch tested only on x86_64-pc-linux-gnu. Add branch to if clause to handle polymorphic objects, not sure if I got all possible variation

[Patch, fortran] PR fortran/100024 PR fortran/100025 ICE on subroutine missing explicit interface

2021-04-10 Thread José Rui Faustino de Sousa via Gcc-patches
Hi all! Proposed patch to PR100024 & PR100025 - ICE on missing polymorphic argument. Patch tested only on x86_64-pc-linux-gnu. Remove assertion checking for possible assumed rank arrays and added an explicit error message. Change if clause to allow the handling of assumed-rank arrays as arra

Re: [Patch, fortran] PR fortran/100018 - ICE on missing polymorphic argument

2021-04-10 Thread José Rui Faustino de Sousa via Gcc-patches
On 10/04/21 17:37, Tobias Burnus wrote: And you need an additional single-line summary for git – which should be part of the patch submission. Fortran: Fix ICE due to referencing a NULL pointer [PR100018] gcc/fortran/ChangeLog: PR fortran/100018 * resolve.c: Add association check be

[pushed] c++: ICE with invalid use of 'this' with static memfn [PR98800]

2021-04-10 Thread Jason Merrill via Gcc-patches
Here instantiation of the fake 'this' parameter we used when parsing the trailing return type of func() was failing because there is no actual 'this' parameter in the instantiation. For PR97399 I told Patrick to do the 'this' injection even for statics, but now I think I was wrong; the out-of-clas

[committed] analyzer: fix ICE on assignment from STRING_CST when building path [PR100011]

2021-04-10 Thread David Malcolm via Gcc-patches
Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. Pushed to trunk as r11-8117-gec633d3777bd71f7bde5e671b61ec18e5b7b43ea. gcc/analyzer/ChangeLog: PR analyzer/100011 * region-model.cc (region_model::on_assignment): Avoid NULL dereference if ctxt is NULL when assi

[PATCH] c++: Reject alias CTAD in C++17 [PR99008]

2021-04-10 Thread Patrick Palka via Gcc-patches
Here, in C++17 mode, we only pedwarn about the use of alias CTAD and then later ICE from alias_ctad_tweaks when attempting to add a constraint to one of the guides. Since the construction of the guides of an alias template effectively relies on concepts, we shouldn't be permissive about alias CTAD

Re: [Patch, fortran] 99307 - FAIL: gfortran.dg/class_assign_4.f90 execution test

2021-04-10 Thread Tobias Burnus
Dear Paul, sorry for the belate reply. I think you forgot to attach the patch. Tobias On 06.04.21 19:08, Paul Richard Thomas via Fortran wrote: Hi Tobias, I believe that the attached fixes the problems that you found with gfc_find_and_cut_at_last_class_ref. I will test: type1%type%array_

Re: [Patch, fortran] PR fortran/100018 - ICE on missing polymorphic argument

2021-04-10 Thread Tobias Burnus
Hi José, On 10.04.21 18:58, José Rui Faustino de Sousa via Fortran wrote: Proposed patch to PR100018 - ICE on missing polymorphic argument. Patch tested only on x86_64-pc-linux-gnu. LGTM – Thanks for the patch. Two nits: If you don't want to rely on the author field of git and specify an ext

[Patch, fortran] PR fortran/100018 - ICE on missing polymorphic argument

2021-04-10 Thread José Rui Faustino de Sousa via Gcc-patches
Hi all! Proposed patch to PR100018 - ICE on missing polymorphic argument. Patch tested only on x86_64-pc-linux-gnu. Add association check before de-referencing pointer in order to avoid ICE. Thank you very much. Best regards, José Rui 2021-4-10 José Rui Faustino de Sousa gcc/fortran

[GCC 12] [PATCH] Add inline_ignore_target function attribute

2021-04-10 Thread H.J. Lu via Gcc-patches
Add inline_ignore_target function attribute to inform the compiler that target specific option mismatch on functions with the always_inline attribute may be ignored. On x86 targets, this attribute can be used on integer functions to ignore target non-integer option mismatch. gcc/c-family/

Re: [PATCH] c: Avoid clobbering TREE_TYPE (error_mark_node) [PR99990]

2021-04-10 Thread Marek Polacek via Gcc-patches
On Sat, Apr 10, 2021 at 08:13:08AM +0200, Jakub Jelinek wrote: > Hi! > > The following testcase ICEs during error recovery, because finish_decl > overwrites TREE_TYPE (error_mark_node), which better should stay always > to be error_mark_node. > > Fixed thusly, bootstrapped/regtested on x86_64-lin

[committed] d: Merge upstream dmd 0450061c8

2021-04-10 Thread Iain Buclaw via Gcc-patches
Hi, This patch merges the D front-end implementation with upstream dmd 0450061c8. D front-end changes: - Fix ICE in forward referenced type members of structs. - Fix ICE passing a member template mixin identifier as alias argument. - Fix ICE when `__traits' prints error involving a Paramete

[committed] libphobos: Add section support code for MACHO and PE/COFF

2021-04-10 Thread Iain Buclaw via Gcc-patches
Hi, This replaces the original and untested support for Windows and OSX, and is the 90% of the work needed to support libphobos on those targets. The core.thread interface has been updated to accomodate for the same function might be implemented by any of the platform-dependent modules. Bootstra

[committed] libphobos: Build runtime library with -ffunction-sections -fdata-sections

2021-04-10 Thread Iain Buclaw via Gcc-patches
Hi, This patch updates libphobs build files to compile with `-ffunction-sections -fdata-sections' where supported, and sets SECTION_FLAGS accordingly, to take advantage of the smaller executables that can be had with `--gc-sections'. Bootstrapped and regression tested on x86_64-linux-gnu/-m32/-mx

[committed] libphobos: Explicitly use -static-libphobos in druntime and phobos tests

2021-04-10 Thread Iain Buclaw via Gcc-patches
Hi, This patch updates the default flags to explicitly use `-static-libphobos' in the druntime.exp and phobos.exp test scripts. Linking to libphobos statically is the default in the driver, however this may change in future. Be explicit that the static libphobos is what's being tested. Bootstra

[committed] libphobos: Remove is-effective-target static from druntime and phobos tests

2021-04-10 Thread Iain Buclaw via Gcc-patches
Hi, This patch removes is-effective-target from the druntime.exp and phobos.exp test scripts. These tests aren't compiling with `-static', they're there to verify that the libphobos is functional when linked in statically. Bootstrapped and regression tested on x86_64-linux-gnu/-m32/-mx32, and co

[committed] libphobos: Re-add -fno-moduleinfo flag to dg-runtest [PR99812]

2021-04-10 Thread Iain Buclaw via Gcc-patches
Hi, This patch partially reverts a previous change that removed these flags, just to satisfy running tests on Solaris. The fix isn't really correct, what should really be done is that the test modules are compiled in a way that doesn't conflict with the release library, which would also allow mos

Re: [PATCH] expand: Fix up LTO ICE with COMPOUND_LITERAL_EXPR [PR99849]

2021-04-10 Thread Richard Biener
On April 10, 2021 8:16:16 AM GMT+02:00, Jakub Jelinek wrote: >Hi! > >The gimplifier optimizes away COMPOUND_LITERAL_EXPRs, but they can >remain >in the form of ADDR_EXPR of COMPOUND_LITERAL_EXPRs in static >initializers. >By the TREE_STATIC check I meant to check that the underlying decl of >the c

Re: [PATCH] gimple-ssa-warn-alloca: Always initialize limit [PR99989]

2021-04-10 Thread Richard Biener
On April 10, 2021 8:11:33 AM GMT+02:00, Jakub Jelinek wrote: >Hi! > >This PR is about a -W*uninitialized warning on riscv64. >alloca_type_and_limit is documented to have limit member only defined >when type is ALLOCA_BOUND_MAYBE_LARGE or ALLOCA_BOUND_DEFINITELY_LARGE >and otherwise just default c

Re: [PATCH] rtlanal: Another fix for VOIDmode MEMs [PR98601]

2021-04-10 Thread Richard Biener
On April 10, 2021 8:07:57 AM GMT+02:00, Jakub Jelinek wrote: >Hi! > >This is a sequel to the PR85022 changes, inline-asm can (unfortunately) >introduce VOIDmode MEMs and in PR85022 they have been changed so that >we don't pretend we know their size (as opposed to assuming they have >zero size). >

For contributing into GCC website or wwwdocs

2021-04-10 Thread Veerpal Kaur via Gcc-patches
Dear mam/sir. I am Veerpal Kaur, I have good knowledge of HTML, CSS, Bootstrap, JavaScript, Reactjs, SQL, MySQL, MongoDB, etc. Now I want to contribute to the GCC Web site and want to make it more user-friendly. I have one problem that I am unable to find a mentor for this project. Please mam/sir

Re: [PATCH] x86: Define _serialize as macro

2021-04-10 Thread Uros Bizjak via Gcc-patches
On Fri, Apr 9, 2021 at 8:58 PM H.J. Lu wrote: > > On Tue, Mar 30, 2021 at 4:40 AM Uros Bizjak wrote: > > > > On Tue, Mar 30, 2021 at 1:03 PM Jakub Jelinek wrote: > > > > > > On Tue, Mar 30, 2021 at 12:59:16PM +0200, Richard Biener wrote: > > > > > > > So yes, a better solution would be nice but

Do not release body of declare_variant_alt

2021-04-10 Thread Jan Hubicka
Hi, the libgomp.c/../libgomp.c-c++-common/task-detach-6.c introduced by my release_body refactoring turns out to be relatively funny interaction between free_leang_data and openmp clones production. Openmp creates special functions with declare_variant_alt which eventualy becomes real functions, b

Re: [PATCH] rtlanal: Another fix for VOIDmode MEMs [PR98601]

2021-04-10 Thread Richard Sandiford via Gcc-patches
Richard Sandiford via Gcc-patches writes: > Jakub Jelinek via Gcc-patches writes: >> Hi! >> >> This is a sequel to the PR85022 changes, inline-asm can (unfortunately) >> introduce VOIDmode MEMs and in PR85022 they have been changed so that >> we don't pretend we know their size (as opposed to ass

Re: [PATCH] rtlanal: Another fix for VOIDmode MEMs [PR98601]

2021-04-10 Thread Richard Sandiford via Gcc-patches
Jakub Jelinek via Gcc-patches writes: > Hi! > > This is a sequel to the PR85022 changes, inline-asm can (unfortunately) > introduce VOIDmode MEMs and in PR85022 they have been changed so that > we don't pretend we know their size (as opposed to assuming they have > zero size). > > This time we ICE