[cfe-commits] r151638 - in /cfe/trunk: include/clang/AST/Decl.h include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td include/clang/Sema/Scope.h include/clang/Sema/Sema.h lib/AST/Decl.cpp lib/AST/DumpXML.cpp lib/Parse/P...

[James Molloy]

Abramo,

Yes, this also fixes PR6298, thanks for letting me know. I think this
longstanding deficiency in Clang has been reported under many guises.

$ ./bin/clang -fsyntax-only bug.c
[18:41]
bug.c:3:15: warning: declaration of 'struct s' will not be visible outside
of this function [-Wvisibility]
void f(struct s { int y; }* v) {
              ^
1 warning generated.

Cheers,
James

-----Original Message-----
From: Abramo Bagnara [mailto:abramo.bagnara at gmail.com] 
Sent: 28 February 2012 18:36
To: James Molloy
Cc: cfe-commits at cs.uiuc.edu
Subject: Re: [cfe-commits] r151638 - in /cfe/trunk: include/clang/AST/Decl.h
include/clang/Basic/DiagnosticGroups.td
include/clang/Basic/DiagnosticSemaKinds.td include/clang/Sema/Scope.h
include/clang/Sema/Sema.h lib/AST/Decl.cpp lib/AST/DumpXML.cpp
lib/Parse/P...

Il 28/02/2012 19:12, James Molloy ha scritto:
> Author: jamesm
> Date: Tue Feb 28 12:12:11 2012
> New Revision: 151638
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=151638&view=rev
> Log:
> Correctly track tags and enum members defined in the prototype of a
function, and ensure they are properly scoped.
> 
> This fixes code such as:
> 
> enum e {x, y};
> int f(enum {y, x} n) {
>   return 0;
> }
> 
> This finally fixes PR5464 and PR5477.

Is this meant to fix also http://llvm.org/bugs/show_bug.cgi?id=6298 ?