This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: RFA: PATCH to ipa-devirt for c++/58678

From: Jason Merrill <jason at redhat dot com>
To: Jan Hubicka <hubicka at ucw dot cz>
Cc: gcc-patches List <gcc-patches at gcc dot gnu dot org>
Date: Wed, 10 Sep 2014 22:06:56 -0400
Subject: Re: RFA: PATCH to ipa-devirt for c++/58678
Authentication-results: sourceware.org; auth=none
References: <54108DA3 dot 9060806 at redhat dot com> <20140910224743 dot GA28486 at kam dot mff dot cuni dot cz>

On 09/10/2014 06:47 PM, Jan Hubicka wrote:

Again - I think main problem is that we provide a middle end a function body
that it is not allowed to use.  We do not really have a concept of function
definitions that we may not use and worse yet, we are preventing that just in
one special case - i.e. in the speculative devirtualization.

Because that special case is the only time we want to prevent it, as Iargued in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58678#c19

My patch shouldn't change any behavior except for fixing this LTO bug,since implicitly-declared member functions always have vague linkage.It seems to me a simple, safe fix.


Jason

Follow-Ups:
- Re: RFA: PATCH to ipa-devirt for c++/58678
  - From: Jan Hubicka

References:
- RFA: PATCH to ipa-devirt for c++/58678
  - From: Jason Merrill
- Re: RFA: PATCH to ipa-devirt for c++/58678
  - From: Jan Hubicka

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]