Use the most recent previous decl to check if inline is added after a definition
authorReid Kleckner <reid@kleckner.net>
Wed, 8 Apr 2015 00:04:47 +0000 (00:04 +0000)
committerReid Kleckner <reid@kleckner.net>
Wed, 8 Apr 2015 00:04:47 +0000 (00:04 +0000)
commit93864170479ae974fc2865c395048829163dfca8
tree6bbc542fa116a6b4848480b04b6286a3bf46e0a4
parentafbae16179f3867e34284de9ef55e971c070cb44
Use the most recent previous decl to check if inline is added after a definition

This affects this test case:
  void foo();
  template <typename T> class C {
    friend inline void foo();
  };
  inline void foo() {}
  C<int> c;

Here, we instantiate the foo friend decl and add it to foo's redecl
chain. However, our previous decl pointer happens to reference the first
declaration of foo, which is not marked inline. When we check to see if
foo was already defined, we implicitly search all previous decls. We
should do the same for the inline check, instead of just checking this
particular previous decl.

Reviewers: rsmith

Differential Revision: http://reviews.llvm.org/D8872

llvm-svn: 234374
clang/lib/Sema/SemaDeclCXX.cpp
clang/test/SemaTemplate/friend.cpp