I'm not quite sure about this in the standards. Say I have three files like this:
foo.h
#include <iostream>
inline void foo();
void foo()
{
std::cout << "Foo" << std::endl;
}
foo.cpp:
#include "foo.h"
void baz();
int main()
{
baz();
foo();
}
bar.cpp
#include "foo.h"
void baz()
{
foo();
}
Now, the definition for foo will be compiled into both compilation units foo.o and bar.o. If I understand it correctly, having inlined functions will avoid linker collition. G++ compiled and links this just fine, but with clang++ 2.8 I get this error:
/tmp/cc-7RdmYP.o: In function `foo()':
bar.cpp:(.text+0x50): multiple definition of `foo()'
/tmp/cc-LW3id3.o:foo.cpp:(.text+0x50): first defined here
collect2: ld returned 1 exit status
It seems that clang++ does not see void foo()
as an inlined function. It does however, work fine when I add inline to the definition as well.
Do I have to add inline to void foo()
as well here to have it be seen as an inlined function, or is this a clang++ bug?
C++0X draft N3225 says in 7.1.2 Function specifiers
:
clause 2: A function declaration with an inline specifier declares an inline function
clause 4: An inline function shall be defined in every translation unit in which it is odr-used and shall have exactly the same definition in every case.
So, to me, it looks like gcc is right & clang wrong, but there's still a (slim) chance that things were (are?) different in C++03..
Odds are that your clang uses C99 inline semantics. In C99, if one of your functions doesn't use "inline" or does include "extern", then the definition is an "external definition", which can only appear once in the program. See inline in C99.
In C++, your program is fine. In Clang SVN, this bug has been fixed and your program should work fine.
I believe that the intention of the standard has always been to allow a function to be made inline
by having at least one declaration including the inline
specifier but there was some uncertainty about when it was too late to add the first inline
declaration. Was after the definition too late, or after the first call?
My reasoning for this is two fold, first the examples in 7.1.1, although non-normative and principally about storage class specifiers, suggest that inline
isn't required on every declaration.
Secondly this defect report DR 317 from 2001 (voted in in 2005) which adds the "If the definition of a function appears in a translation unit before its first declaration as inline, the program is ill-formed." sentence. It is clear from the conversation that it was intented that inline
isn't required on every declaration, specifically in the case of a member function defined explicitly inline
but outside of the class body where the original declaration didn't have an explicit inline
.
(That defect report also contains my mantra that inline
is "more than a hint".)
Of course, as soon as a function with external linkage is an inline function due to one or more declarations including the inline
specifier in one translation unit it must be declared inline
in all translation units according to the rest of paragraph 7.1.2/4.
In the example in the question I believe that the intention is that foo
is an inline function and that it is valid code although the normative text of the standard seems to me to be less clear than it could be.
You have to use inline
in both places.