[PATCH] D13925: Implement __attribute__((internal_linkage))

[Evgeniy Stepanov via cfe-commits]

eugenis added a comment.

Hm, the current implementation allows all of the following:

void f(int a [[clang::internal_linkage]]) {  // 1

  int b [[clang::internal_linkage]];   // 2
  static int c [[clang::internal_linkage]];  // 3

}

I'll fix (1). Is it OK to allow (2) and (3)? The attribute has no effect because the declarations already have internal linkage, so I'd say it behaves as documented.


Repository:
  rL LLVM

http://reviews.llvm.org/D13925