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: [PATCH][LTO] Fix PR40902, ignore toplevel qualifiers when merging symbols


On Mon, 5 Oct 2009, Diego Novillo wrote:

> On Mon, Oct 5, 2009 at 09:02, Richard Guenther <rguenther@suse.de> wrote:
> >
> > This fixes PR40902, a case where we fail to merge symbols because
> > the prevailing definition is const while the extern declaration is not.
> 
> But we should at least warn if this happens.  The testcase you added
> seems to imply that we don't warn about mismatched const qualifiers.

Yes, we don't - I think we should be consistent with the linker
if it won't cause problems downstream.  At least we should add
a new warning flag for this kind of stuff?  (and I have to check
if dg-warning works with lto.exp obviously...)

Richard.


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