This is the mail archive of the gcc@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: Compiling GCC with g++: a report


Andrew Pinski <pinskia@physics.uc.edu> writes:

| On May 24, 2005, at 12:01 AM, Zack Weinberg wrote:
| > Use of bare 'inline' is just plain wrong in our source code; this has
| > nothing to do with C++, no two C compilers implement bare 'inline'
| > alike.  Patches to add 'static' to such functions (AND MAKING NO OTHER
| > CHANGES) are preapproved, post-slush.
| That will not work for the cases where the bare 'inline' are used
| because they are external also in this case.  Now this is where C99 and
| C++ differs at what a bare 'inline' means so I have no idea what to
| do, except for removing the 'inline' in first place.

It suffices to make the function "static inline".  See our current
extensive usage.  That works for both C and C++.

-- Gaby


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