Hi Eli,

On Fri, Jan 13, 2012 at 15:41, Eli Friedman <[email protected]> wrote:
> Modified: cfe/trunk/test/SemaCXX/linkage.cpp
> URL: 
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/linkage.cpp?rev=148158&r1=148157&r2=148158&view=diff
> ==============================================================================
> --- cfe/trunk/test/SemaCXX/linkage.cpp (original)
> +++ cfe/trunk/test/SemaCXX/linkage.cpp Fri Jan 13 17:41:25 2012
> @@ -76,13 +76,15 @@
>     struct X {
>       int f() {
>         extern int g();
> -        extern int a;
> +        // FIXME: We don't compute the correct linkage for this variable
> +        // at the moment
> +        // extern int a;

Are you planning on addressing this FIXME soon? We have some code
which looks roughly like this, and now Clang is producing a warning
"variable 'a' has internal linkage but is not defined".

-Matt

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to