This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] attribute copy, leaf, weakref and -Wmisisng-attributes (PR 88546)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Martin Sebor <msebor at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org, "Joseph S. Myers" <joseph at codesourcery dot com>, Marek Polacek <polacek at redhat dot com>, Jason Merrill <jason at redhat dot com>, Richard Biener <rguenther at suse dot de>
- Date: Sat, 22 Dec 2018 01:16:42 +0100
- Subject: Re: [PATCH] attribute copy, leaf, weakref and -Wmisisng-attributes (PR 88546)
- References: <6ff6565b-51d8-8620-f345-a0082747297b@gmail.com> <2f0b2202-ba56-0554-1cc9-f290cda2a740@gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Dec 21, 2018 at 04:50:47PM -0700, Martin Sebor wrote:
> The first revision of the patch was missing a test and didn't
> completely or completely correctly handle attribute noreturn.
> Attached is an update with the test included and the omission
> and bug fixed.
>
> I think it makes sense to consider the patch independently of
> the question whether weakrefs should be extern. That change can
Weakrefs shouldn't be extern, that is what we were using initially and
changed to static. At this point we can't change that again IMNSHO.
Jakub