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: [lto] make more things go


>>>>> "Nathan" == Nathan Froyd <froydnj@codesourcery.com> writes:

Nathan> This patch doesn't produce any spectacular results on testsuite
Nathan> numbers--it may actually regress slightly due to the builtin change--
Nathan> but (with minor tweaks to the SPEC code to properly declare
Nathan> functions[*]) 

Nathan> [*] Yes, I know you're not allowed to do this.  What do people think is
Nathan> the right thing to do here?  Entirely ignore function types for the
Nathan> purposes of merging decls?

IMO, emit a warning, but have a way to disable it.  And, regardless of
the warning, do whatever the linker does now.  Maybe don't consider
"bad" functions as inline candidates across object files.

My rationale for this is that real programs do this dumb stuff, and
the above seems to yield the least surprising result.

It would be interesting to know whether anything in gcc triggers this
warning.  But I suppose we could find out with --combine.

Tom


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