Hi Steve,

Steve Naroff wrote:-

> @@ -594,8 +602,7 @@
>    Identifier.setKind(II.getTokenID());
>      
>    // If this is an extension token, diagnose its use.
> -  // FIXME: tried (unsuccesfully) to shut this up when compiling with gnu99
> -  // For now, I'm just commenting it out (while I work on attributes).
> -  if (II.isExtensionToken() && Features.C99) 
> +  // We avoid diagnosing tokens that originate from macro definitions.
> +  if (II.isExtensionToken() && Features.C99 && !DisableMacroExpansion)
>      Diag(Identifier, diag::ext_token_used);
>  }

Is that the correct test?  It looks like its testing whether expansion
is enabled rather than what the source of the token is.

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

Reply via email to