[cfe-commits] r67336 - in /cfe/trunk: include/clang/AST/Decl.h lib/Sema/SemaDecl.cpp test/Sema/tentative-decls.c test/Sema/var-redecl.c

Douglas Gregor dgregor at apple.com
Mon Mar 23 09:17:05 PDT 2009


On Mar 20, 2009, at 4:58 PM, Eli Friedman wrote:

> On Thu, Mar 19, 2009 at 3:01 PM, Douglas Gregor <dgregor at apple.com>  
> wrote:
>> URL: http://llvm.org/viewvc/llvm-project?rev=67336&view=rev
>> Log:
>> Variables marked as "extern" can actually have internal linkage if
>> there is a previous declaration marked "static". This fixes PR3645.
>
> Minor issue: the following should print a diagnostic (see 6.2.2p5):
>
> static int x; int x;

Thanks, Eli. I willfully misread the first sentence of that paragraph,  
thinking it applied to objects. I'll commit the fix in a moment.

	- Doug



More information about the cfe-commits mailing list